Skip to content
Snippets Groups Projects
Commit 7cf4260c authored by Jan-Hendrik Willms's avatar Jan-Hendrik Willms Committed by David Siegfried
Browse files

trim values for search, fixes #1765

Closes #1765

Merge request studip/studip!1142
parent a2a499c2
No related branches found
No related tags found
No related merge requests found
...@@ -117,8 +117,10 @@ class Admin_UserController extends AuthenticatedController ...@@ -117,8 +117,10 @@ class Admin_UserController extends AuthenticatedController
$this->order = $this->order == 'desc' ? 'asc' : 'desc'; $this->order = $this->order == 'desc' ? 'asc' : 'desc';
} }
$request['vorname'] = $request['vorname'] ?? null; $request['username'] = $this->getStringValueFromRequest($request, 'username');
$request['nachname'] = $request['nachname'] ?? null; $request['email'] = $this->getStringValueFromRequest($request, 'email');
$request['vorname'] = $this->getStringValueFromRequest($request, 'vorname');
$request['nachname'] = $this->getStringValueFromRequest($request, 'nachname');
$request['inaktiv'] = $inaktiv; $request['inaktiv'] = $inaktiv;
$request['datafields'] = $search_datafields; $request['datafields'] = $search_datafields;
$request['sort'] = $this->sortby; $request['sort'] = $this->sortby;
...@@ -233,6 +235,15 @@ class Admin_UserController extends AuthenticatedController ...@@ -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) * Bulk action (delete users or send message to all)
*/ */
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment