Bug #63557
closed
BackendUtility::getProcessedValue does not include check for foreign_match_fields and foreign_table_field (at least if not MM table like sys_file_reference)
Added by Andreas Allacher over 9 years ago.
Updated over 6 years ago.
Sprint Focus:
Stabilization Sprint
Description
BackendUtility::getProcessedValue does not include checks for foreign_match_fields and foreign_table_field which results in e.g. referenced files of other data records to be listed in List module (if one includes a FAL field)
Furthermore, it would also be nice to e.g. see the combined identifier instead of the sys_file_reference UID or at least the filename (e.g. by checking for foreign_label?)
Also I think the hooks for getProcessedValue should pass on the tablename, the col itself (both in pre and post process)
preProcessValue I think could also include the value as one could use that info, maybe even to bypass the rest of the method if everything can be done in preProcessValue
And also certain method parameters should be passed on like noRecordLookup, ...
I vote for raising this issue's priority to a “must have”, as BackendUtility::getProcessedValue does not consider any additional foreign_table_field and foreign_match_fields constraints for “select“ and “inline” relations using foreign_field-constructs which imitate “MM”-relations. The current 6.x TYPO3 Core uses this for “FAL” relations.
Related use-cases:
- Running a fresh TYPO3 v6.x with some custom models using custom “FAL”-relations, shows wrong details for those relations in the “web_list”-Module, if there are “sys_file_reference”-records matching the currently displayed record's “uid” with their “uid_foreign”-column but with different “tablename” and “fieldname” values.
- Migrating TYPO3 v4.x with “DAM”-Extension's relations to TYPO3 v6.x with “FAL”-relations, shows wrong details for those relations in the “web_list”-Module, if there are “sys_file_reference”-records matching the currently displayed record's “uid” with their “uid_foreign”-column but with different “tablename” and “fieldname” values.
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/37035
- Target version set to 7.2 (Frontend)
- Sprint Focus set to Stabilization Sprint
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38960
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by 
Updated by