summaryrefslogtreecommitdiffstats
path: root/extensions/MozReview/Extension.pm
diff options
context:
space:
mode:
authorMark Côté <mcote@alumni.uwaterloo.ca>2015-05-05 02:29:09 +0200
committerMark Côté <mcote@alumni.uwaterloo.ca>2015-05-05 17:23:39 +0200
commit63b0ecb066a2bfde2828d77c0c414c57d4b7b1ce (patch)
treeb081a1be30a94ba88a1bcf526d73e87210a27f34 /extensions/MozReview/Extension.pm
parent27db7bb9aabe7a411d701ffbd9989e1ef11a9651 (diff)
downloadbugzilla-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.pm8
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;
}
}
}