Bug #67187

recursiveFileListSortingHelper natural sorting isn't locale aware

Added by Alexander Opitz over 3 years ago. Updated 9 months ago.

Status:
Under Review
Priority:
Should have
Category:
-
Start date:
2015-05-29
Due date:
% Done:

0%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
hard
Is Regression:
No
Sprint Focus:

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 TYPO3 Core - Bug #65319: ResourceUtilityTest::recursiveFileListSortingHelperCorrectlySorts Closed 2015-02-25
Related to TYPO3 Core - Bug #20612: scandinavian letters are translittered wrong Needs Feedback 2009-06-12

History

#2 Updated by Alexander Opitz over 3 years 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 over 3 years ago

  • Description updated (diff)

#4 Updated by Gerrit Code Review over 3 years 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 over 3 years ago

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

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

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

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

#9 Updated by Gerrit Code Review over 3 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/39801

#10 Updated by Gerrit Code Review about 3 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/39801

#11 Updated by Gerrit Code Review about 3 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/39801

#12 Updated by Gerrit Code Review about 3 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/39801

#13 Updated by Gerrit Code Review about 3 years ago

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

#14 Updated by Benni Mack over 1 year ago

  • Target version changed from 8 LTS to next-patchlevel

#15 Updated by Benni Mack about 1 year ago

  • Sprint Focus set to On Location Sprint

#16 Updated by Gerrit Code Review about 1 year ago

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

#17 Updated by Alexander Opitz 12 months ago

  • Target version changed from next-patchlevel to Candidate for Major Version

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

#19 Updated by Gerrit Code Review 12 months ago

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

#20 Updated by Gerrit Code Review 12 months ago

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

#21 Updated by Gerrit Code Review 12 months ago

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

#22 Updated by Mathias Brodala 9 months ago

  • Sprint Focus deleted (On Location Sprint)

Also available in: Atom PDF