Omit the `all` call after Project#project_group_links to avoid unnecessary loads

39d293ab · Stan Hu · Francisco Lopez · 88e3ce30 · 39d293ab
Commit 39d293ab authored Nov 19, 2017 by Stan Hu Committed by Francisco Lopez Dec 01, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

lib/api/entities.rb lib/api/entities.rb +1 -1

No files found.
--- a/lib/api/entities.rb
+++ b/lib/api/entities.rb
@@ -156,7 +156,7 @@ module API
      expose :public_builds, as: :public_jobs
      expose :ci_config_path
      expose :shared_with_groups do |project, options|
-        SharedGroup.represent(project.project_group_links.all, options)
+        SharedGroup.represent(project.project_group_links, options)
      end
      expose :only_allow_merge_if_pipeline_succeeds
      expose :request_access_enabled