From 73061bdba98c73d31b1e8aa9e6ab54a235db3945 Mon Sep 17 00:00:00 2001 From: etiennevvv <59622352+etiennevvv@users.noreply.github.com> Date: Thu, 19 Mar 2020 11:17:33 -0400 Subject: [PATCH] GRN2-284: Fixed bug where admin rights emails was sent multiple times (#996) Co-authored-by: Jesus Federico --- app/controllers/concerns/rolify.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/concerns/rolify.rb b/app/controllers/concerns/rolify.rb index e2affed9..dbfc9fb0 100644 --- 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