From 63b0ecb066a2bfde2828d77c0c414c57d4b7b1ce Mon Sep 17 00:00:00 2001
From: Mark Côté <mcote@alumni.uwaterloo.ca>
Date: Mon, 4 May 2015 20:29:09 -0400
Subject: 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.
---
 extensions/MozReview/Extension.pm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'extensions/MozReview/Extension.pm')

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;
             }
         }
     }
-- 
cgit v1.2.3-24-g4f1b