Bug #63557
closedBackendUtility::getProcessedValue does not include check for foreign_match_fields and foreign_table_field (at least if not MM table like sys_file_reference)
100%
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?)
Updated by Andreas Allacher almost 10 years ago
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, ...
Updated by Stephan Jorek almost 10 years ago
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.
Updated by Gerrit Code Review almost 10 years ago
- 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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review almost 10 years ago
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
Updated by Gerrit Code Review over 9 years ago
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
Updated by Nicole Cordes over 9 years ago
- Target version set to 7.2 (Frontend)
- Sprint Focus set to Stabilization Sprint
Updated by Gerrit Code Review over 9 years ago
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
Updated by Andreas Allacher over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 710d064921457c4313f77f9c21ff8afc0b268c5a.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed