trim values for search, fixes #1765

Closes #1765

Merge request studip/studip!1142

1 files changed, 13 insertions, 2 deletions

diff --git a/app/controllers/admin/user.php b/app/controllers/admin/user.php
index 72a42b4..dd875dc 100644
--- a/app/controllers/admin/user.php
+++ b/app/controllers/admin/user.php
@@ -117,8 +117,10 @@ class Admin_UserController extends AuthenticatedController
                 $this->order = $this->order == 'desc' ? 'asc' : 'desc';
             }
 
-            $request['vorname']    = $request['vorname'] ?? null;
-            $request['nachname']   = $request['nachname'] ?? null;
+            $request['username']   = $this->getStringValueFromRequest($request, 'username');
+            $request['email']      = $this->getStringValueFromRequest($request, 'email');
+            $request['vorname']    = $this->getStringValueFromRequest($request, 'vorname');
+            $request['nachname']   = $this->getStringValueFromRequest($request, 'nachname');
             $request['inaktiv']    = $inaktiv;
             $request['datafields'] = $search_datafields;
             $request['sort']       = $this->sortby;
@@ -233,6 +235,15 @@ class Admin_UserController extends AuthenticatedController
         }
     }
 
+    private function getStringValueFromRequest(array $request, string $key): ?string
+    {
+        if (!isset($request[$key])) {
+            return null;
+        }
+
+        return trim($request[$key]) ?: null;
+    }
+
     /**
      * Bulk action (delete users or send message to all)
      */