openproject icon indicating copy to clipboard operation
openproject copied to clipboard

[#68702] Project user filter also includes groups and placeholder users

Open EinLama opened this issue 1 month ago • 0 comments

Ticket

https://community.openproject.org/wp/68702

What are you trying to accomplish?

Remove the type = User restriction on the filter component, so that groups and placeholder users can also be found.

Included some minor drive-by rubocop fixes for neighboring specs.

Screenshots

image

What approach did you choose and why?

Simply removed the restriction from the base class that only matched on principals of type user. Apart from the project filter, the meetings filter is inheriting from the base class, too. Therefore, this change alters meetings filters – in theory. In practice, there is no user custom field filter for meetings, so this change has no effect on meetings.

Merge checklist

  • [x] Added/updated tests
  • [ ] Added/updated documentation in Lookbook (patterns, previews, etc)
  • [x] Tested major browsers (Chrome, Firefox, Edge, ...)

EinLama avatar Jan 08 '26 10:01 EinLama