From f8a258de8911f3e1035739c105bec1fa94a6d7db Mon Sep 17 00:00:00 2001
From: Byron Jones <glob@mozilla.com>
Date: Wed, 2 Apr 2014 17:11:44 +0800
Subject: Bug 986590: Confusing error message when not finding reviewer

---
 extensions/Review/Extension.pm | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'extensions')

diff --git a/extensions/Review/Extension.pm b/extensions/Review/Extension.pm
index 634e7c9cd..16429ccdd 100644
--- a/extensions/Review/Extension.pm
+++ b/extensions/Review/Extension.pm
@@ -364,6 +364,16 @@ sub _update_reviewers {
 # wrong, or matches too many, default to empty", which breaks mandatory
 # reviewer requirements.  instead we just throw an error.
 sub post_bug_attachment_flags {
+    my ($self, $args) = @_;
+    $self->_check_review_flag($args);
+}
+
+sub create_attachment_flags {
+    my ($self, $args) = @_;
+    $self->_check_review_flag($args);
+}
+
+sub _check_review_flag {
     my ($self, $args) = @_;
     my $bug = $args->{bug};
     my $cgi = Bugzilla->cgi;
-- 
cgit v1.2.3-24-g4f1b