diff options
author | Robert Speicher <robert@gitlab.com> | 2017-03-17 00:13:31 +0000 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2017-03-17 00:13:31 +0000 |
commit | 13b1e4fe33f25b5f03a9e3cc164c75ef9a6ad59f (patch) | |
tree | 75bb6720483e38c23132f7a4672cd0b168044948 /app | |
parent | e5dfcdf26875a2294453cc58d8bbe79f69f37a53 (diff) | |
parent | 871bed7ac0a777d6187cbf09362c5fae441a8607 (diff) | |
download | gitlab-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.rb | 5 | ||||
-rw-r--r-- | app/services/users/refresh_authorized_projects_service.rb | 4 |
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 |