Project

General

Profile

Actions

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 almost 10 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2014-12-04
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
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?)

Actions #1

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, ...

Actions #2

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.
Actions #3

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

Actions #4

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

Actions #5

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

Actions #6

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

Actions #7

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

Actions #8

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

Actions #9

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

Actions #10

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

Actions #11

Updated by Nicole Cordes over 9 years ago

  • Target version set to 7.2 (Frontend)
  • Sprint Focus set to Stabilization Sprint
Actions #12

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

Actions #13

Updated by Andreas Allacher over 9 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF