File Listings in Alphabetical Order

Applied patch. Works for me.

Subscribing for integration into filedepot

A variable_del needs to be added in filedepot_uninstall() for filedepot_override_folderorder.

Works as expected.

Here's an updated patch with the variable_del.

Works fine; please apply to the dev.

It does not work for files in subfolders.

I've uploaded a couple of files into a subfolder and they do not display in alphabetical order -- see the attachment.

I've duplicated this with the patch filedepot-alpha-file-order-2.patch. It seems to only happen when the option 'Display the index numbers in the listing of folders and files' is enabled.

This patch should fix the problem in comment #6. I didn't understand the changes to filedepotAjaxServer_loadFileDetails in the original patch (the changes there should do nothing anyway, I hope), so I removed them from this patch.

Was eager to try it, but unfortunately this patch fails against the current 6.x-1.x-dev. Could you please check and re-roll?

Solved the patching temporarily by moving the module to folder "b" and then applied the patch.

However, looking at a subfolder, the files are not sorted by file names with or without "Display the index numbers..." option. See the attached screenshots.

Try "patch -p1 < issue-1263826-3.patch". I tried creating a folder with the same files you did and they are sorted for me.

Thanks for the tip, yes, of course, that obviates the need for creating a temp folder.

The effect was the same, though. After applying the patch (and I did it again, very carefully, just now), and despite clearing all caches, despite trying in both the latest FF and Chrome, the files are not sorted by name and look the same
.
But that applies to an existing folder structure.

When I created a new folder with files, or a subfolder with the same files, they are ordered by file name even though I was uploading them deliberately in another order:

Is there something that prevents the existing structure to be re-ordered after applying the patch?

The patch can definitely be applied now since it does work for freshly added files and folders.

Bump... can somebody please apply the latest patch?

I'd change one thing from the posted patch at #comment-5491614. I'd change the default value from 1 to 0, as shown in the code snippet below:

@@ -108,6 +108,11 @@ function filedepot_admin_settings() {
   '#default_value' => variable_get('filedepot_show_index_enabled', 1),

to be

@@ -108,6 +108,11 @@ function filedepot_admin_settings() {
   '#default_value' => variable_get('filedepot_show_index_enabled', 0),

The reason is, when you first bring up the admin page with the first version set, it looks like you have already enabled alphabetical listing. By defaulting to off, users aren't accidentally misled. They *have* to click Save for the setting to take effect.

Of course, you could always add stuff in the .install to preset the file listing to alphabetical order, but then you're changing the default functionality away from what over a thousand users are already using and expecting.

If you agree, please re-roll the patch with the change described above.

Updated dev - thanks for the patch. I had to modify it as the code structure has changed since it was made, but the general idea works.