fix sorting, closes #1781

Closes #1781

Merge request studip/studip!1163

app/controllers/admin/user.php

@@ -123,6 +123,9 @@ class Admin_UserController extends AuthenticatedController
                 $this->order = $this->order == 'desc' ? 'asc' : 'desc';
             }
 
+            $request['sortby'] = $this->sortby;
+            $request['order'] = $this->order;
+
             $empty_search          = $request['perm'] === 'alle';
 
             $values = [

lib/models/User.class.php

@@ -448,8 +448,9 @@ class User extends AuthUserMd5 implements Range, PrivacyObject
         $query .= implode(' AND ', $where);
         $query .= " GROUP BY au.`user_id` ";
 
+        if (!empty($attributes['sortby'])) {
             //sortieren
-        switch ($attributes['sort']) {
+            switch ($attributes['sortby']) {
                 case "perms":
                     $query .= "ORDER BY au.`perms` {$attributes['order']}, au.`username`";
                     break;
@@ -474,6 +475,7 @@ class User extends AuthUserMd5 implements Range, PrivacyObject
                 default:
                     $query .= " ORDER BY au.`username` {$attributes['order']}";
             }
+        }
 
         return DBManager::get()->fetchAll($query, $params, __CLASS__ . '::buildExisting');
     }