U4-10058 - Wire up user state filter

Created by Mads Rasmussen 26 Jun 2017, 07:22:48 Updated by Claus Jensen 06 Jul 2017, 13:11:04

Subtask of: U4-9776

Comments

Shannon Deminick 04 Jul 2017, 02:15:40

I got this working now. To test:

  • In the user list select All - it will show all
  • You can now not un-check All - because it's the same thing as unchecked (default)
  • If you select another filter, All will be unselected and the users will be filtered
  • If you select yet another filter the users will be filtered again, NOTE: in many cases multiple filters will just yield zero users since it's rare a user will show up in multiple filters
  • If you select All again, it will uncheck all other filters


Claus Jensen 05 Jul 2017, 09:47:23

Hmm.. I've just clicked through this and tested with Sebastiaan just to make sure it wasn't just me.

It doesn't really make sense that the 'Status' filter, filters the way it does.

  • If you select 'All': all users should appear (correct)
  • If you select 'Active': active users should appear (correct)
  • If you select 'Active' and 'Disabled': you see only users who are both active and disabled .. For this type of filter option you would expect to see a combined list of users who are in either one of these two options.

Actually - for the 'Status' filter, it might even be a better idea to go with a radio select instead of allowing multiple checkboxes as this is basically just confusing how it works. It would be better to simply show users matching either one of these states when that specific state is selected - and only allow 'All' or one single state to be selected at a time.

If we don't go with the radio button, the 'Status' filter should actually just work like the 'Groups' filter, which shows all users matching any (not all) of the selected filters.


Shannon Deminick 05 Jul 2017, 10:18:58

Yup that's what I thought too but I didn't design it :)

But perhaps the way to 'solve' the issue is to do an OR instead of an AND


Claus Jensen 05 Jul 2017, 10:40:35

Haha :D oh well.. I agree on the OR. It should be a fairly simple way to fix most of it and then whoever designed this can consider if it should be changed to a single select radio button.


Shannon Deminick 06 Jul 2017, 09:52:16

I've changed this to an OR now, seems better at least


Claus Jensen 06 Jul 2017, 13:10:59

Yep agree - it works the way you would expect it to work now.


Priority: Normal

Type: Task

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions: 7.7.0

Due in version: 7.7.0

Sprint: Sprint 62

Story Points: 1

Cycle: