Bug #27519

getProcessedValue ignores sorting of select field values

Added by Joh. Feustel over 8 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2011-06-17
Due date:
% Done:

100%

TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

t3lib_BEfunc::getProcessedValue looses sorting of select field values by using "uid IN (' . implode(',', $selectUids) . ')"
The patch sorts the array using the keys from $selectUids afterwards.

How to reproduce: Select field like tt_news categories in list module. Sorting of categories in the records has no effect in the list moduel.

27519.diff View (863 Bytes) Joh. Feustel, 2011-06-17 16:42

27519_reproduce_1.jpg View (33.3 KB) Joh. Feustel, 2012-11-11 12:07

27519_reproduce_2.jpg View (28.6 KB) Joh. Feustel, 2012-11-11 12:07

Associated revisions

Revision b2f438a8 (diff)
Added by Joh. Feustel over 6 years ago

[BUGFIX] getProcessedValue ignores sorting of select field values

Keep sorting of select field values which is lost during database select.

Change-Id: If0347f6c92e71a3855a00fd24bbbada7c15af386
Fixes: #27519
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/2882
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

History

#1 Updated by Joh. Feustel over 8 years ago

#2 Updated by Mr. Hudson over 8 years ago

Patch set 1 of change If0347f6c92e71a3855a00fd24bbbada7c15af386 has been pushed to the review server.
It is available at http://review.typo3.org/2882

#3 Updated by Gerrit Code Review about 8 years ago

  • Status changed from New to Under Review

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#4 Updated by Gerrit Code Review about 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#5 Updated by Gerrit Code Review about 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#6 Updated by Gerrit Code Review over 7 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#7 Updated by Gerrit Code Review over 7 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#8 Updated by Joh. Feustel over 7 years ago

Happens only for select field with "MM" table not used in current core extensions. Example is categories in EXT:tt_news or EXT:news. See attached screenshots.

#10 Updated by Gerrit Code Review over 7 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2882

#11 Updated by Gerrit Code Review about 7 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/2882

#12 Updated by Gerrit Code Review over 6 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/2882

#13 Updated by Gerrit Code Review over 6 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/2882

#14 Updated by Gerrit Code Review over 6 years ago

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

#15 Updated by Joh. Feustel over 6 years ago

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

#16 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF