make table of unregistered plugins sortable, fixes #4927

Closes #4927

Merge request studip/studip!3698

app/controllers/admin/plugin.php

@@ -509,7 +509,11 @@ class Admin_PluginController extends AuthenticatedController
 
     public function unregistered_action()
     {
-        $this->unknown_plugins = $this->plugin_admin->scanPluginDirectory();
+        $plugins = $this->plugin_admin->scanPluginDirectory();
+        usort($plugins, function ($a, $b) {
+            return $a['pluginname'] <=> $b['pluginname'];
+        });
+        $this->unknown_plugins = $plugins;
     }
 
     /**

app/views/admin/plugin/unregistered.php

@@ -4,16 +4,16 @@
  * @var array $unknown_plugins
  */
 ?>
-<table class="default">
+<table class="default sortable-table" data-sortlist="[[0, 0]]">
     <caption>
         <?= _('Im Pluginverzeichnis vorhandene Plugins registrieren') ?>
     </caption>
     <thead>
         <tr>
-            <th><?= _('Name') ?></th>
-            <th><?= _('Pluginklasse') ?></th>
-            <th><?= _('Version') ?></th>
-            <th><?= _('Ursprung') ?></th>
+            <th data-sort="text"><?= _('Name') ?></th>
+            <th data-sort="text"><?= _('Pluginklasse') ?></th>
+            <th data-sort="digit"><?= _('Version') ?></th>
+            <th data-sort="text"><?= _('Ursprung') ?></th>
             <th><?= _('Registrieren') ?></th>
         </tr>
     </thead>

lib/classes/PluginAdministration.php

@@ -405,9 +405,12 @@ class PluginAdministration
         $basepath = Config::get()->PLUGINS_PATH;
         $plugin_manager = PluginManager::getInstance();
         $iterator = new RegexIterator(
-                        new RecursiveIteratorIterator(
-                            new RecursiveDirectoryIterator($basepath, FilesystemIterator::FOLLOW_SYMLINKS | FilesystemIterator::UNIX_PATHS)),
-                        '/plugin\.manifest$/', RecursiveRegexIterator::MATCH);
+            new RecursiveIteratorIterator(
+                new RecursiveDirectoryIterator($basepath, FilesystemIterator::FOLLOW_SYMLINKS | FilesystemIterator::UNIX_PATHS)
+            ),
+            '/plugin\.manifest$/',
+            RegexIterator::MATCH
+        );
         foreach ($iterator as $manifest_file) {
             $manifest = $plugin_manager->getPluginManifest($manifest_file->getPath());
             if (!isset($manifest['pluginclassname'])) {