Bug #70677
closed
Sorting of foreign records in TCA type select
100%
Description
Currently records of a foreign table won't be sorted in a select field if foreign_table_where doesn't contain an "ORDER BY ..." statement.
I'd suggest the following solution:- Introduce a new option 'foreign_table_sortby'
- If 'foreign_table_sortby' is defined, use this statement for sorting
- Backward compatibility: If 'foreign_table_where' contains an 'ORDER BY ...' statement and 'foreign_table_sortby' is undefined, use the 'ORDER BY ...' statement as 'foreign_table_sortby'. If 'foreign_table_sortby' is defined, ignore 'ORDER BY ...' statements in 'foreign_table_where'
- If 'foreign_table_sortby' is not defined, use 'default_sortby' of the foreign table
In the long term this is a cleaner solution and if no sorting is defined, the records are sorted according to the default sorting.
Updated by Gone With the Wind over 6 years ago
- Tracker changed from Feature to Bug
- Category set to Backend API
- Target version set to next-patchlevel
- TYPO3 Version set to 8
- PHP Version set to 7.1
I just came over a similar problem: the default_sortby is a date field (start_date) and it works fine, if it is used as the standard sorting mechanism of the table. But as soon as it is used as the right table of an m:m relation, things don't work as nicely any more, when the TCA does NOT define an ORDER BY. As soon as there is an "ORDER BY start_date" it works as expected - but it is "double the trouble".
I would say: if there is no ORDER BY, then the standard sorting mechanism of the table should be used.
(I am not so sure if the intrdocution of a new sorting mechanism is necessary...)
TYPO3 8.7.1, php 7.1.13, MAMP with opcache
Updated by Benni Mack over 5 years ago
- Target version changed from next-patchlevel to Candidate for patchlevel
Updated by Gerrit Code Review over 5 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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/61474
Updated by Christoph Lehmann over 5 years ago
- Related to Task #88933: Cleanup foreign_table_where directives added
Updated by Christoph Lehmann over 5 years ago
- Related to deleted (Task #88933: Cleanup foreign_table_where directives)
Updated by Christoph Lehmann over 5 years ago
- Blocks Task #88933: Cleanup foreign_table_where directives added
Updated by Gerrit Code Review over 5 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61474
Updated by Gerrit Code Review over 5 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61474
Updated by Gerrit Code Review about 5 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61474
Updated by Gerrit Code Review almost 5 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62906
Updated by Christoph Lehmann almost 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1d604c7ed800c22b12202a91309aefa723ac3e59.
Updated by Benni Mack almost 5 years ago
- Related to Bug #90127: ambiguous column name title when editing content elements added