Bug #67187

recursiveFileListSortingHelper natural sorting isn't locale aware

Added by Alexander Opitz 2 months ago. Updated about 1 month ago.

Status:Under Review Start date:2015-05-29
Priority:Should have Due date:
Assigned To:Alexander Opitz % Done:

0%

Category:- Spent time: -
Target version:8
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:hard

Description

recursiveFileListSortingHelper uses strnatcasecmp which isn't locale aware so Umlauts (and others) gets sorted to the end. Which, depending on locale, isn't correct handling.

=> Request change in PHP, as this wouldn't be backported to older versions this won't happen for 7 LTS.
=> Use intl Extensions Collator


Related issues

related to Core - Bug #65319: ResourceUtilityTest::recursiveFileListSortingHelperCorrec... Resolved 2015-02-25

History

#2 Updated by Alexander Opitz 2 months ago

There is a way, by using intl Extension.

See http://3v4l.org/RHPkY

So the question, can we raise the dependencies?

#3 Updated by Alexander Opitz 2 months ago

  • Description updated (diff)

#4 Updated by Gerrit Code Review 2 months ago

  • Status changed from In Progress 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/39801

#5 Updated by Alexander Opitz 2 months ago

The solution with Collator doesn't need the locale installed on system to work.

#6 Updated by Gerrit Code Review 2 months 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/39801

#7 Updated by Gerrit Code Review 2 months 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/39801

#8 Updated by Gerrit Code Review 2 months 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/39801

#9 Updated by Gerrit Code Review about 1 month 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/39801

Also available in: Atom PDF