summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorRobert Speicher <robert@gitlab.com>2017-03-17 00:13:31 +0000
committerRobert Speicher <robert@gitlab.com>2017-03-17 00:13:31 +0000
commit13b1e4fe33f25b5f03a9e3cc164c75ef9a6ad59f (patch)
tree75bb6720483e38c23132f7a4672cd0b168044948 /app
parente5dfcdf26875a2294453cc58d8bbe79f69f37a53 (diff)
parent871bed7ac0a777d6187cbf09362c5fae441a8607 (diff)
downloadgitlab-ce-13b1e4fe33f25b5f03a9e3cc164c75ef9a6ad59f.tar.gz
Merge branch 'dm-use-index-by' into 'master'
Use Enumerable#index_by where possible See merge request !10011
Diffstat (limited to 'app')
-rw-r--r--app/services/merge_requests/get_urls_service.rb5
-rw-r--r--app/services/users/refresh_authorized_projects_service.rb4
2 files changed, 2 insertions, 7 deletions
diff --git a/app/services/merge_requests/get_urls_service.rb b/app/services/merge_requests/get_urls_service.rb
index 1262ecbc29a..ae386b53f42 100644
--- a/app/services/merge_requests/get_urls_service.rb
+++ b/app/services/merge_requests/get_urls_service.rb
@@ -23,10 +23,7 @@ module MergeRequests
def opened_merge_requests_from_source_branches(branches)
merge_requests = MergeRequest.from_project(project).opened.from_source_branches(branches)
- merge_requests.inject({}) do |hash, mr|
- hash[mr.source_branch] = mr
- hash
- end
+ merge_requests.index_by(&:source_branch)
end
def get_branches(changes)
diff --git a/app/services/users/refresh_authorized_projects_service.rb b/app/services/users/refresh_authorized_projects_service.rb
index d9370bbb598..8f6f5b937c4 100644
--- a/app/services/users/refresh_authorized_projects_service.rb
+++ b/app/services/users/refresh_authorized_projects_service.rb
@@ -93,9 +93,7 @@ module Users
end
def current_authorizations_per_project
- current_authorizations.each_with_object({}) do |row, hash|
- hash[row.project_id] = row
- end
+ current_authorizations.index_by(&:project_id)
end
def current_authorizations