fixes #3888

Closes #3888

Merge request studip/studip!2740

app/controllers/admin/courses.php

@@ -302,7 +302,7 @@ class Admin_CoursesController extends AuthenticatedController
     public function index_action()
     {
         $this->fields = $this->getViewFilters();
-        $this->sortby = $GLOBALS['user']->cfg->MEINE_SEMINARE_SORT ?? 'name';
+        $this->sortby = $GLOBALS['user']->cfg->MEINE_SEMINARE_SORT ?? (Config::get()->IMPORTANT_SEMNUMBER ? 'number' : 'name');
         $this->sortflag = $GLOBALS['user']->cfg->MEINE_SEMINARE_SORT_FLAG ?? 'ASC';
 
         $this->buildSidebar();
@@ -788,11 +788,6 @@ class Admin_CoursesController extends AuthenticatedController
                 $template->course = $course;
                 $d['action'] = $template->render();
                 break;
-            case 22: //Masssenexport Teilnehmendendaten
-                $template = $tf->open('admin/courses/batch_export_members');
-                $template->course = $course;
-                $d['action'] = $template->render();
-                break;
             default:
                 foreach (PluginManager::getInstance()->getPlugins('AdminCourseAction') as $plugin) {
                     if ($GLOBALS['user']->cfg->MY_COURSES_ACTION_AREA === get_class($plugin)) {

resources/vue/components/AdminCourses.vue

@@ -248,6 +248,16 @@ export default {
 
             if (sortby === 'last_activity') {
                 sortFunction = (a, b) => a.last_activity_raw - b.last_activity_raw;
+            } else if (sortby === 'name') {
+                sortFunction = (a, b) => {
+                    return collator.compare(striptags(a.name), striptags(b.name))
+                        || collator.compare(striptags(a.number), striptags(b.number));
+                };
+            } else if (sortby === 'number') {
+                sortFunction = (a, b) => {
+                    return collator.compare(striptags(a.number), striptags(b.number))
+                        || collator.compare(striptags(a.name), striptags(b.name));
+                };
             } else {
                 let is_numeric = true;
                 for (let i in array) {