GRN2-284: Fixed bug where admin rights emails was sent multiple times (#996)

Co-authored-by: Jesus Federico <jesus@123it.ca>
2020-03-19 11:17:33 -04:00 · 2020-03-19 11:17:33 -04:00 · 73061bdba9
parent 3eead129c9
commit 73061bdba9
1 changed files with 1 additions and 1 deletions
--- a/app/controllers/concerns/rolify.rb
+++ b/app/controllers/concerns/rolify.rb
@ -51,7 +51,7 @@ module Rolify
    return true unless current_user.highest_priority_role.get_permission("can_manage_users")

    new_roles = roles.split(' ').map(&:to_i)
-    old_roles = @user.roles.pluck(:id)
+    old_roles = @user.roles.distinct.pluck(:id)

    added_role_ids = new_roles - old_roles
    removed_role_ids = old_roles - new_roles