diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-12-02 20:23:48 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-12-02 20:23:48 +0000 |
commit | 536747b74708188848d85d8de5df34e7aa762006 (patch) | |
tree | c7cfe03314c047dff313825c8548d056e5e1af29 /app/models/group.rb | |
parent | 96fca0b08f6d60e8736e8e3f95a5d0ae33779068 (diff) | |
download | redmine-536747b74708188848d85d8de5df34e7aa762006.tar.gz redmine-536747b74708188848d85d8de5df34e7aa762006.zip |
Replaces find(:all) calls.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10917 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app/models/group.rb')
-rw-r--r-- | app/models/group.rb | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/app/models/group.rb b/app/models/group.rb index 0eba591b5..f8b1ee639 100644 --- a/app/models/group.rb +++ b/app/models/group.rb @@ -62,8 +62,11 @@ class Group < Principal def user_removed(user) members.each do |member| - MemberRole.find(:all, :include => :member, - :conditions => ["#{Member.table_name}.user_id = ? AND #{MemberRole.table_name}.inherited_from IN (?)", user.id, member.member_role_ids]).each(&:destroy) + MemberRole. + includes(:member). + where("#{Member.table_name}.user_id = ? AND #{MemberRole.table_name}.inherited_from IN (?)", user.id, member.member_role_ids). + all. + each(&:destroy) end end |