From 7cf4260c0ecc4fc39e4871ae03e426f7b1c2512b Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+studip@gmail.com>
Date: Tue, 8 Nov 2022 15:30:54 +0000
Subject: [PATCH] trim values for search, fixes #1765
Closes #1765
Merge request studip/studip!1142
---
app/controllers/admin/user.php | 15 +++++++++++++--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/app/controllers/admin/user.php b/app/controllers/admin/user.php
index 72a42b4c57c..dd875dc0f06 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)
*/
--
GitLab