- 🇩🇪Germany Anybody Porta Westfalica
@dawid_cyron that's fixed in 🐛 Query condition 'ur.roles_target_id IN ()' cannot be empty Fixed
@immaculatexavier could you please provide your patch as MR to mark it RTBC?
- Status changed to Needs work
about 2 years ago 4:48pm 9 May 2023 - First commit to issue fork.
- Status changed to Postponed
about 2 years ago 10:45am 12 May 2023 - 🇩🇪Germany Grevil
Let's postpone this for now.
For me at least, the expected result of not including nor excluding a user role would suggest deleting all users, which are targeted by the purge.
- We should make sure, that in this case, users with the admin role are excluded
- Check what the outcome of "$exist_query->condition('ur.roles_target_id',[], 'IN');" will be, as the fallback value of the user roles is currently an empty array. And I am in doubt that conditioning an empty array, will lead to selecting all user roles.
- 🇫🇷France mably
Could you give a try to the new policy feature in this issue's MR: ✨ Use plugins and condition to create advanced policies Active
Custom conditions have been created for role inclusion and exclusion.
They are not mandatory any more. For now at least.
Automatically closed - issue fixed for 2 weeks with no activity.