ensure the user doesn't already have the role before adding it.

refinery · Jul 2, 2010 · 6fa7872 · 6fa7872
1 parent 5158967
commit 6fa7872
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/vendor/plugins/authentication/app/models/user.rb b/vendor/plugins/authentication/app/models/user.rb
@@ -59,7 +59,9 @@ def can_delete?(user_to_delete = self)
   end
 
   def add_role(role_to_add)
-    roles << Role[role_to_add]
+    unless self.has_role?(role = Role[role_to_add])
+      roles << role
+    end
   end
 
   def has_role?(role)