diff options
| author | Sean McGivern <sean@gitlab.com> | 2019-08-20 12:58:03 +0000 |
|---|---|---|
| committer | Sean McGivern <sean@gitlab.com> | 2019-08-20 12:58:03 +0000 |
| commit | 57ec78d53066cf9184859a0202609eb01567eab9 (patch) | |
| tree | 0fec8bd0238520e11cb226f118ab6590e29f44fb /lib/api/entities.rb | |
| parent | 9e6a8bb93b6fbcbe5657079108332b99fc8749cc (diff) | |
| parent | ba7c501fef5976ea7a1cc4212e84742246fed781 (diff) | |
| download | gitlab-ce-57ec78d53066cf9184859a0202609eb01567eab9.tar.gz | |
Merge branch 'sh-fix-issues-api-gitaly-nplusone' into 'master'
Fix Gitaly N+1 calls with listing issues/MRs via API
Closes #66202
See merge request gitlab-org/gitlab-ce!31938
Diffstat (limited to 'lib/api/entities.rb')
| -rw-r--r-- | lib/api/entities.rb | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 09253ab6b0e..5e66b4e76a5 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -645,7 +645,10 @@ module API end end - expose :subscribed do |issue, options| + # Calculating the value of subscribed field triggers Markdown + # processing. We can't do that for multiple issues / merge + # requests in a single API request. + expose :subscribed, if: -> (_, options) { options.fetch(:include_subscribed, true) } do |issue, options| issue.subscribed?(options[:current_user], options[:project] || issue.project) end end |