Ruby 2.7: Fix RuboCop offense Performance/MapCompact (#38134).

git-svn-id: https://svn.redmine.org/redmine/trunk@22055 e93f8b46-1217-0410-a6f0-8f06a7374b81

7 files changed, 10 insertions, 11 deletions

diff --git a/app/models/member.rb b/app/models/member.rb
index 46a81afc0..21c623408 100644
--- a/app/models/member.rb
+++ b/app/models/member.rb
@@ -120,8 +120,7 @@ class Member < ActiveRecord::Base
   def role_inheritance(role)
     member_roles.
       select {|mr| mr.role_id == role.id && mr.inherited_from.present?}.
-      map {|mr| mr.inherited_from_member_role.try(:member)}.
-      compact.
+      filter_map {|mr| mr.inherited_from_member_role.try(:member)}.
       map {|m| m.project == project ? m.principal : m.project}
   end
 
diff --git a/app/models/query.rb b/app/models/query.rb
index e25824bd0..bd670205b 100644
--- a/app/models/query.rb
+++ b/app/models/query.rb
@@ -787,9 +787,9 @@ class Query < ActiveRecord::Base
     return [] if available_columns.empty?
 
     # preserve the column_names order
-    cols = (has_default_columns? ? default_columns_names : column_names).collect do |name|
+    cols = (has_default_columns? ? default_columns_names : column_names).filter_map do |name|
       available_columns.find {|col| col.name == name}
-    end.compact
+    end
     available_columns.select(&:frozen?) | cols
   end
 
diff --git a/app/models/repository.rb b/app/models/repository.rb
index dc7783aff..be10fa3cf 100644
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -436,7 +436,7 @@ class Repository < ActiveRecord::Base
     # commits.to_a.sort! {|x, y| x.last <=> y.last}
     changes = Change.joins(:changeset).where("#{Changeset.table_name}.repository_id = ?", id).
                 select("committer, user_id, count(*) as count").group("committer, user_id")
-    user_ids = changesets.map(&:user_id).compact.uniq
+    user_ids = changesets.filter_map(&:user_id).uniq
     authors_names = User.where(:id => user_ids).inject({}) do |memo, user|
       memo[user.id] = user.to_s
       memo
diff --git a/app/models/repository/git.rb b/app/models/repository/git.rb
index 2463ab80a..2c77c173d 100644
--- a/app/models/repository/git.rb
+++ b/app/models/repository/git.rb
@@ -213,7 +213,7 @@ class Repository::Git < Repository
   private :save_revisions
 
   def save_revision(rev)
-    parents = (rev.parents || []).collect{|rp| find_changeset_by_name(rp)}.compact
+    parents = (rev.parents || []).filter_map{|rp| find_changeset_by_name(rp)}
     changeset =
       Changeset.create(
         :repository   => self,
diff --git a/app/models/repository/mercurial.rb b/app/models/repository/mercurial.rb
index 30cd59fd5..a152950c9 100644
--- a/app/models/repository/mercurial.rb
+++ b/app/models/repository/mercurial.rb
@@ -193,9 +193,9 @@ class Repository::Mercurial < Repository
     (db_rev + 1).step(scm_rev, FETCH_AT_ONCE) do |i|
       scm.each_revision('', i, [i + FETCH_AT_ONCE - 1, scm_rev].min) do |re|
         transaction do
-          parents = (re.parents || []).collect do |rp|
+          parents = (re.parents || []).filter_map do |rp|
             find_changeset_by_name(scmid_for_inserting_db(rp))
-          end.compact
+          end
           cs = Changeset.create(:repository   => self,
                                 :revision     => re.revision,
                                 :scmid        => scmid_for_inserting_db(re.scmid),
diff --git a/app/models/repository/subversion.rb b/app/models/repository/subversion.rb
index c6355cff1..373d250bc 100644
--- a/app/models/repository/subversion.rb
+++ b/app/models/repository/subversion.rb
@@ -42,7 +42,7 @@ class Repository::Subversion < Repository
   def latest_changesets(path, rev, limit=10)
     revisions = scm.revisions(path, rev, nil, :limit => limit)
     if revisions
-      identifiers = revisions.collect(&:identifier).compact
+      identifiers = revisions.filter_map(&:identifier)
       changesets.where(:revision => identifiers).reorder("committed_on DESC").includes(:repository, :user).to_a
     else
       []
@@ -97,7 +97,7 @@ class Repository::Subversion < Repository
 
     entries_with_identifier =
       entries.select {|entry| entry.lastrev && entry.lastrev.identifier.present?}
-    identifiers = entries_with_identifier.map {|entry| entry.lastrev.identifier}.compact.uniq
+    identifiers = entries_with_identifier.filter_map {|entry| entry.lastrev.identifier}.uniq
     if identifiers.any?
       changesets_by_identifier =
         changesets.where(:revision => identifiers).
diff --git a/app/models/role.rb b/app/models/role.rb
index b73f97d4c..e67f30054 100644
--- a/app/models/role.rb
+++ b/app/models/role.rb
@@ -122,7 +122,7 @@ class Role < ActiveRecord::Base
   end
 
   def permissions=(perms)
-    perms = perms.collect {|p| p.to_sym unless p.blank?}.compact.uniq if perms
+    perms = perms.filter_map {|p| p.to_sym unless p.blank?}.uniq if perms
     write_attribute(:permissions, perms)
   end