Bug #56353

SQL Error in sys_refindex on saving records with tca/flexform type group internal_type file

Added by Marc Bastian Heinrichs over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-02-26
Priority:Should have Due date:
Assigned To:Marc Bastian Heinrichs % Done:

100%

Category:DataHandler aka TCEmain Spent time: -
Target version:next-patchlevel
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:

Description

caller     TYPO3\CMS\Core\Database\DatabaseConnection::exec_INSERTquery
ERROR     Column 'ref_table' cannot be null
lastBuiltQuery     INSERT INTO sys_refindex (tablename,recuid,field,flexpointer,softref_key,softref_id,sorting,deleted,ref_table,ref_uid,ref_string,hash) VALUES ('tx_impexpgroupfiles_item','1','images','','','','0','0',NULL,NULL,'','bd4e53ee65b4927172175e806f7f389a')
debug_backtrace     TYPO3\CMS\Backend\Controller\EditDocumentController->processData#48 // TYPO3\CMS\Core\DataHandling\DataHandler->process_datamap#516 // TYPO3\CMS\Core\DataHandling\DataHandler->updateDB#1222 // TYPO3\CMS\Core\DataHandling\DataHandler->updateRefIndex#5851 // TYPO3\CMS\Core\Database\ReferenceIndex->updateRefIndexTable#6008 // TYPO3\CMS\Core\Database\DatabaseConnection->exec_INSERTquery#120 // TYPO3\CMS\Core\Database\DatabaseConnection->debug#203

Related issues

related to Grid Elements - Bug #55380: SQL Error when choosing flexform file Closed 2014-01-28
related to Core - Task #56352: Impexp: Files of TCA type "group" internal_type "file" ar... Resolved 2014-02-26
related to Core - Task #56991: RefIndex is broken for FlexForm fields group file_referen... Resolved 2014-03-17

Associated revisions

Revision 53c6adc8
Added by Marc Bastian Heinrichs over 1 year ago

[BUGFIX] Fields of type group file are not properly indexed

If an record with a field of type "group", internal_type "file"
should be indexed in the ReferenceIndex an SQL error exception
gets thrown. So the relation to the file isn't represented in the
refindex. Thus those files are also missing in exports created
with system extension impexp.

This is caused by wrong array keys and incorrectly storing a file
relation as DB relation since merge of FAL.

Resolves: #56353
Resolves: #56352
Releases: 6.2, 6.1, 6.0
Change-Id: I78211efcb6bf3032811ef0b0e20ed7f98f9fa8fb
Reviewed-on: https://review.typo3.org/27310
Tested-by: Sebastian Fischer
Tested-by: Christian Zenker
Reviewed-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

Revision 1dbfe75e
Added by Marc Bastian Heinrichs over 1 year ago

[BUGFIX] Fields of type group file are not properly indexed

If an record with a field of type "group", internal_type "file"
should be indexed in the ReferenceIndex an SQL error exception
gets thrown. So the relation to the file isn't represented in the
refindex. Thus those files are also missing in exports created
with system extension impexp.

This is caused by wrong array keys and incorrectly storing a file
relation as DB relation since merge of FAL.

Resolves: #56353
Resolves: #56352
Releases: 6.2, 6.1, 6.0
Change-Id: I78211efcb6bf3032811ef0b0e20ed7f98f9fa8fb
Reviewed-on: https://review.typo3.org/28795
Reviewed-by: Marc Bastian Heinrichs
Tested-by: Marc Bastian Heinrichs

Revision 75f6b1b6
Added by Marc Bastian Heinrichs over 1 year ago

[BUGFIX] Fields of type group file are not properly indexed

If an record with a field of type "group", internal_type "file"
should be indexed in the ReferenceIndex an SQL error exception
gets thrown. So the relation to the file isn't represented in the
refindex. Thus those files are also missing in exports created
with system extension impexp.

This is caused by wrong array keys and incorrectly storing a file
relation as DB relation since merge of FAL.

Resolves: #56353
Resolves: #56352
Releases: 6.2, 6.1, 6.0
Change-Id: I78211efcb6bf3032811ef0b0e20ed7f98f9fa8fb
Reviewed-on: https://review.typo3.org/28796
Reviewed-by: Marc Bastian Heinrichs
Tested-by: Marc Bastian Heinrichs

History

#1 Updated by Bernhard Kraft over 1 year ago

Fixing this bug is somehow prohibited by the following bug:

http://i.imgur.com/3MgE6bi.png

Which results in:

http://i.imgur.com/xWynWgs.png

... and both of those bugs inhibit properly testing this issue (Can't upload images)

Sorry! The problem was that I did not run the upgrade wizard.
False alarm.

#2 Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27310

#3 Updated by Gerrit Code Review over 1 year ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27310

#4 Updated by Gerrit Code Review over 1 year ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27310

#5 Updated by Gerrit Code Review over 1 year ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27310

#6 Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28021

#7 Updated by Gerrit Code Review over 1 year ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27310

#8 Updated by Marc Bastian Heinrichs over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#9 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28795

#10 Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28796

#11 Updated by Marc Bastian Heinrichs over 1 year ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF