diff options
author | Mark Côté <mcote@alumni.uwaterloo.ca> | 2015-05-05 02:29:09 +0200 |
---|---|---|
committer | Mark Côté <mcote@alumni.uwaterloo.ca> | 2015-05-05 17:23:39 +0200 |
commit | 63b0ecb066a2bfde2828d77c0c414c57d4b7b1ce (patch) | |
tree | b081a1be30a94ba88a1bcf526d73e87210a27f34 /extensions/MozReview/Extension.pm | |
parent | 27db7bb9aabe7a411d701ffbd9989e1ef11a9651 (diff) | |
download | bugzilla-63b0ecb066a2bfde2828d77c0c414c57d4b7b1ce.tar.gz bugzilla-63b0ecb066a2bfde2828d77c0c414c57d4b7b1ce.tar.xz |
Bug 1158516 - Fetch MozReview summaries by bug. r=glob
Instead of issuing separate calls for every MozReview
request associated with a given bug, call the new
MozReview summaries-by-bug API exactly once.
Diffstat (limited to 'extensions/MozReview/Extension.pm')
-rw-r--r-- | extensions/MozReview/Extension.pm | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/extensions/MozReview/Extension.pm b/extensions/MozReview/Extension.pm index 296240b49..1f2758f9f 100644 --- a/extensions/MozReview/Extension.pm +++ b/extensions/MozReview/Extension.pm @@ -36,19 +36,19 @@ sub template_before_process { $vars->{'mozreview_enabled'} = 1; } } else { - my @rrids; + my $has_mozreview = 0; my $attachments = Bugzilla::Attachment->get_attachments_by_bug($bug); foreach my $attachment (@$attachments) { if ($attachment->contenttype eq 'text/x-review-board-request' && !$attachment->isobsolete) { - push @rrids, ($attachment->data =~ m#/r/(\d+)/?$#); + $has_mozreview = 1; + last; } } - if (scalar @rrids) { + if ($has_mozreview) { $vars->{'mozreview'} = 1; - $vars->{'review_request_ids'} = \@rrids; } } } |