owncloud / web

:dragon_face: Next generation frontend for ownCloud Infinite Scale
https://owncloud.dev/clients/web/
GNU Affero General Public License v3.0
443 stars 157 forks source link

[admin-settings] User: Filter by multiple groups, unexpected outcome #8469

Closed hurradieweltgehtunter closed 1 year ago

hurradieweltgehtunter commented 1 year ago

Steps to reproduce

  1. Open admin settings, “Users” Tab
  2. Click on groups filter, select multiple groups.

Expected behaviour

My expectation was to get a result with users, who belong to one of the selected groups (OR).

Actual behaviour

Users got filtered by groups with AND => Only users who belong to all selected groups are displayed.

Not a bug, but unexpected outcome. If this is not done on purpose, it should be considered what the general expectation of this filter is. OR Add a selection “Show users belonging to ONE of the selected groups” / “Show users belonging to all selected groups”

https://ocis.ocis-wopi.latest.owncloud.works

screenshot_000447

tbsbdr commented 1 year ago

added image to initial comment

tbsbdr commented 1 year ago

Should be changed to "OR" concatenation (as this logic meets most usecases)

JammingBen commented 1 year ago

An or filter needs to be implemented on the server first, currently it only supports and.

cc @micbar @rhafer

rhafer commented 1 year ago

Ok. or support for groupMemberShip should be easy enough to add.

@tbsbdr Probably a bit off topic for this issue, but one question regarding the roleAssignment filtering (which will get introduced soon): Which behavior do you envision if a group and a role is picked for filtering in the admin UI. I guess that should then have AND behaviour, right? So it should only display users that are member of one of the selected groups AND have the selected role assigned, correct?

Also do we need to allow selecting multiple roles for the roles filter as well? It's possible, but it would need some extra implementation effort in the backend, that I'd rather defer for now.

tbsbdr commented 1 year ago

hey @rhafer 1) filter concatenation: correct, filters should be and-concatenated amongst each other (show people who are in group XY AND have the role guest) 2) Single select for role selection is sufficient for the first step