Skip to content

Handle nil user in EE version of Group#member?

Peter Leitzen requested to merge pl-ee-group-member-nilable into master

What does this MR do and why?

Previously, we had to check if user is present before passing it to Group#member? because it'd fail due to the EE override.

This commit handles the case where user is nil and returns false.

The previous nil checks are no longer necessary.

See !117694 (7fec75ee, comment 1357379029)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Peter Leitzen

Merge request reports