]> source.dussan.org Git - redmine.git/commitdiff
Use pluck(:id) instead of collect(&:id) in app/models/issue.rb (#26711).
authorGo MAEDA <maeda@farend.jp>
Sun, 29 Jul 2018 05:45:40 +0000 (05:45 +0000)
committerGo MAEDA <maeda@farend.jp>
Sun, 29 Jul 2018 05:45:40 +0000 (05:45 +0000)
git-svn-id: http://svn.redmine.org/redmine/trunk@17455 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/models/issue.rb

index d04705d587bff1b871021de6685ef87baed9a4aa..0906771536f6b2aa57401e812eba7db621db552c 100644 (file)
@@ -124,10 +124,10 @@ class Issue < ActiveRecord::Base
         when 'all'
           '1=1'
         when 'default'
-          user_ids = [user.id] + user.groups.map(&:id).compact
+          user_ids = [user.id] + user.groups.pluck(:id).compact
           "(#{table_name}.is_private = #{connection.quoted_false} OR #{table_name}.author_id = #{user.id} OR #{table_name}.assigned_to_id IN (#{user_ids.join(',')}))"
         when 'own'
-          user_ids = [user.id] + user.groups.map(&:id).compact
+          user_ids = [user.id] + user.groups.pluck(:id).compact
           "(#{table_name}.author_id = #{user.id} OR #{table_name}.assigned_to_id IN (#{user_ids.join(',')}))"
         else
           '1=0'