Make Plugin Editor's file list alphabetical by full path

[Daedalon]

The files under Plugin Editor's (plugin-editor.php) Plugin Files header should be shown in alphabetical order of their full path. Currently their order seems completely random, which makes the list cumbersome to use.

[Daedalon]

Sorts file list alphabetically by full path and improves performance of get_plugin_files() by calling plugin_basename() only once

[MattyRob]

I have tested this patch and it works fine for me. Commit?

[godhulii_1985]

Please commit this patch. It works...

[MattyRob]

Sorry to report on further testing that this doesn't work fully.

By returning a sorted list of plugin files the 'first' entry returned may not be the main plugin file and it may also not be a file supported in the editor - resulting in unusual behaviour.

The sort needs to return the array with the main file at the start of the array or somehow that file needs to be 'selected' by default.

[MattyRob]

Attached patch resolves the issued stated above by placing the main plugin file name at the start of the returned array.

I've also dropped the root file sorting section with the preg_match - it doesn't seem to change the returned values and also causes errors as the regex is not properly contained - I was getting lots of errors log about invalid operators.

[johnbillion]

If you select a file in a subdirectory for editing, the list of files in the sidebar completely breaks, and the current plugin is not correctly selected. I can't believe there isn't an existing ticket for this.

I think that's closely related to this and they should be fixed together. get_plugin_files() should probably always be passed the plugin's root file, not the file being currently edited.

We should also get some unit tests on this.

Related: #24048

[dd32]

Vaguely related: #17552

[SergeyBiryukov]

This appears to be fixed in [41806], I can't reproduce any issues with file order in current trunk. The list includes:

• The main plugin file.
• The rest of the files, in alphabetical order.