Skip to content

Admin: Allow searching users with their full name#1230

Merged
ZakarFin merged 1 commit intooskariorg:developfrom
jampukka:fix/admin-user-search
Jan 7, 2026
Merged

Admin: Allow searching users with their full name#1230
ZakarFin merged 1 commit intooskariorg:developfrom
jampukka:fix/admin-user-search

Conversation

@jampukka
Copy link
Copy Markdown
Member

@jampukka jampukka commented Dec 22, 2025

Allow Admin to find users when searching with their whole name.

oskaridb=# select user_name, first_name, last_name from oskari_users;
 user_name  | first_name | last_name
------------+------------+-----------
 user       | Oskari     | Demo user
 testirobot | Test       | Robot
 admin      | Oskari     | Admin
(3 rows)

Searching with search=Oskari Admin:

oskaridb=# SELECT first_name, last_name, user_name
FROM oskari_users
WHERE
  user_name ilike '%' || 'Oskari Admin' || '%'
  OR first_name ilike '%' || 'Oskari Admin' || '%'
  OR last_name ilike '%' || 'Oskari Admin' || '%'
  OR email ilike '%' || 'Oskari Admin' || '%';
 first_name | last_name | user_name
------------+-----------+-----------
(0 rows)

With the code change:

oskaridb=# SELECT first_name, last_name, user_name
FROM oskari_users
WHERE
  user_name ilike '%' || 'Oskari Admin' || '%'
  OR first_name ilike '%' || 'Oskari Admin' || '%'
  OR last_name ilike '%' || 'Oskari Admin' || '%'
  OR email ilike '%' || 'Oskari Admin' || '%'
  OR replace(first_name || last_name, ' ', '') ilike '%' || replace('Oskari Admin', ' ', '') || '%'
  OR replace(last_name || first_name, ' ', '') ilike '%' || replace('Oskari Admin', ' ', '') || '%';
 first_name | last_name | user_name
------------+-----------+-----------
 Oskari     | Admin     | admin

The second OR handles searching with {lastName} {firstName} order, not uncommon to do with actual human names.

@ZakarFin ZakarFin added this to the 3.3.0 milestone Jan 7, 2026
@ZakarFin ZakarFin merged commit d3f075d into oskariorg:develop Jan 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants