Project

General

Profile

Actions
Copy link

Bug #67187

closed

recursiveFileListSortingHelper natural sorting isn't locale aware

Added by Alexander Opitz almost 9 years ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Candidate for Major Version
Start date:
2015-05-29
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
12
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 3 (1 open — 2 closed)

Related to TYPO3 Core - Bug #65319: ResourceUtilityTest::recursiveFileListSortingHelperCorrectlySortsClosedAlexander Opitz2015-02-25

Actions
Related to TYPO3 Core - Bug #20612: scandinavian letters are translittered wrongNeeds Feedback2009-06-12

Actions
Related to TYPO3 Core - Task #100231: Improve performance of FilelistClosed2023-03-20

Actions
Actions
Copy link
 #2

Updated by Alexander Opitz almost 9 years ago

There is a way, by using intl Extension.

See http://3v4l.org/RHPkY

So the question, can we raise the dependencies?

Actions
Copy link
 #3

Updated by Alexander Opitz almost 9 years ago

  • Description updated (diff)
Actions
Copy link
 #4

Updated by Gerrit Code Review almost 9 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

Actions
Copy link
 #5

Updated by Alexander Opitz almost 9 years ago

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

Actions
Copy link
 #6

Updated by Gerrit Code Review almost 9 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

Actions
Copy link
 #7

Updated by Gerrit Code Review almost 9 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

Actions
Copy link
 #8

Updated by Gerrit Code Review almost 9 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

Actions
Copy link
 #9

Updated by Gerrit Code Review almost 9 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

Actions
Copy link
 #10

Updated by Gerrit Code Review over 8 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

Actions
Copy link
 #11

Updated by Gerrit Code Review over 8 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

Actions
Copy link
 #12

Updated by Gerrit Code Review over 8 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

Actions
Copy link
 #13

Updated by Gerrit Code Review over 8 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

Actions
Copy link
 #14

Updated by Benni Mack almost 7 years ago

  • Target version changed from 8 LTS to next-patchlevel
Actions
Copy link
 #15

Updated by Benni Mack over 6 years ago

  • Sprint Focus set to On Location Sprint
Actions
Copy link
 #16

Updated by Gerrit Code Review over 6 years 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

Actions
Copy link
 #17

Updated by Alexander Opitz over 6 years ago

  • Target version changed from next-patchlevel to Candidate for Major Version
Actions
Copy link
 #18

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/39801

Actions
Copy link
 #19

Updated by Gerrit Code Review over 6 years 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

Actions
Copy link
 #20

Updated by Gerrit Code Review over 6 years 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

Actions
Copy link
 #21

Updated by Gerrit Code Review over 6 years 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

Actions
Copy link
 #22

Updated by Mathias Brodala about 6 years ago

  • Sprint Focus deleted (On Location Sprint)
Actions
Copy link
 #23

Updated by Susanne Moog about 5 years ago

  • Status changed from Under Review to New

Patch has been abandoned.

Actions
Copy link
 #24

Updated by Susanne Moog about 4 years ago

  • Category set to File Abstraction Layer (FAL)
  • Assignee deleted (Alexander Opitz)
Actions
Copy link
 #25

Updated by Tymoteusz Motylewski about 1 year ago

  • Related to Task #100231: Improve performance of Filelist added
Actions
Copy link
 #26

Updated by Tymoteusz Motylewski about 1 year ago

  • Status changed from New to Resolved
  • TYPO3 Version changed from 6.2 to 12
Actions
Copy link

Also available in: Atom PDF