Merge pull request RolifyCommunity#216 from RallyPointNetworks/speedu…

…p_roles_removal Significant speed improvement when removing roles using limit(1)
zlotnika · Jan 27, 2014 · 9cb745c · 9cb745c
2 parents 0c01a28 + 73e1795
commit 9cb745c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/rolify/adapters/active_record/role_adapter.rb b/lib/rolify/adapters/active_record/role_adapter.rb
@@ -24,7 +24,7 @@ def remove(relation, role_name, resource = nil)
         if roles
           relation.roles.delete(roles)
           roles.each do |role|
-            role.destroy if role.send(ActiveSupport::Inflector.demodulize(user_class).tableize.to_sym).empty?
+            role.destroy if role.send(ActiveSupport::Inflector.demodulize(user_class).tableize.to_sym).limit(1).empty?
           end
         end
         roles