Bug #56824

FileStorages aren't sorted

Added by Christoph Lehmann over 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-03-12
Due date:
% Done:

100%

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

Description

In the filelist modul the file storages are listed by uid order. There is no sorting field in the table sys_file_storage.

I think the best would be to sort it alphabetically since it's very confusing when you have hundred of them.

Thanks,
Christoph

FileStorages.png View (24.9 KB) Christoph Lehmann, 2014-03-17 08:29

Associated revisions

Revision 6a7cd68b (diff)
Added by Frans Saris over 5 years ago

[BUGFIX] Order storages in treeview by name

The storages can not be ordered manualy and in list view they
are by default ordered by name.
But in the tree views (file module, element browser) there is
no ordering set.

This change sets the default ordering to name when the storageRowCache
is initialised.

Resolves: #56824
Releases: 6.2
Change-Id: I8e879920bc6da14156ce486b8c25b1b8820f896b
Reviewed-on: https://review.typo3.org/28402
Reviewed-by: Marcin SÄ…gol
Reviewed-by: Stefan Neufeind
Reviewed-by: Christoph Lehmann
Tested-by: Christoph Lehmann
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

History

#1 Updated by Frans Saris over 5 years ago

Not sure if 100 storages is a real usecase but sorting by name makes sense. Could you provide a patch?

#2 Updated by Christoph Lehmann over 5 years ago

I have no patch, sorry.

#3 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/28402

#4 Updated by Christoph Lehmann over 5 years ago

The patch works, thank you!

#5 Updated by Frans Saris over 5 years ago

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

#6 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF