From 64e818b7f831e2df8bc5638a977b5e34e3e31a1d Mon Sep 17 00:00:00 2001
From: Byron Jones <bjones@mozilla.com>
Date: Fri, 16 Aug 2013 13:15:07 +0800
Subject: Bug 905904: List "away" reviewers in reviewer report

---
 extensions/Review/Extension.pm                     | 27 +++++++++++-----------
 .../admin/components/edit-common-rows.html.tmpl    |  4 ++--
 .../hook/admin/products/edit-common-rows.html.tmpl |  2 +-
 3 files changed, 17 insertions(+), 16 deletions(-)

(limited to 'extensions/Review')

diff --git a/extensions/Review/Extension.pm b/extensions/Review/Extension.pm
index ccbb4203e..568a16a30 100644
--- a/extensions/Review/Extension.pm
+++ b/extensions/Review/Extension.pm
@@ -38,18 +38,18 @@ BEGIN {
 # reviewers
 #
 
-sub _product_reviewers         { _reviewers($_[0],      'product')   }
-sub _product_reviewers_objs    { _reviewers_objs($_[0], 'product')   }
-sub _component_reviewers       { _reviewers($_[0],      'component') }
-sub _component_reviewers_objs  { _reviewers_objs($_[0], 'component') }
+sub _product_reviewers         { _reviewers($_[0],      'product',   $_[1])   }
+sub _product_reviewers_objs    { _reviewers_objs($_[0], 'product',   $_[1])   }
+sub _component_reviewers       { _reviewers($_[0],      'component', $_[1]) }
+sub _component_reviewers_objs  { _reviewers_objs($_[0], 'component', $_[1]) }
 
 sub _reviewers {
-    my ($object, $type) = @_;
-    return join(', ', map { $_->login } @{ _reviewers_objs($object, $type) });
+    my ($object, $type, $include_disabled) = @_;
+    return join(', ', map { $_->login } @{ _reviewers_objs($object, $type, $include_disabled) });
 }
 
 sub _reviewers_objs {
-    my ($object, $type) = @_;
+    my ($object, $type, $include_disabled) = @_;
     if (!$object->{reviewers}) {
         my $dbh = Bugzilla->dbh;
         my $user_ids = $dbh->selectcol_arrayref(
@@ -61,9 +61,10 @@ sub _reviewers_objs {
         # so we have to reorder the list
         my $users = Bugzilla::User->new_from_list($user_ids);
         my %user_map = map { $_->id => $_ } @$users;
-        my @reviewers =
-            grep { $_->name !~ UNAVAILABLE_RE }
-            map { $user_map{$_} } @$user_ids;
+        my @reviewers = map { $user_map{$_} } @$user_ids;
+        if (!$include_disabled) {
+            @reviewers = grep { $_->name !~ UNAVAILABLE_RE } @reviewers;
+        }
         $object->{reviewers} = \@reviewers;
     }
     return $object->{reviewers};
@@ -169,7 +170,7 @@ sub object_end_of_update {
     my ($new, $new_users) = _new_reviewers_from_input();
     my $old = $old_object->reviewers;
     if ($old ne $new) {
-        _update_reviewers($object, $old_object->reviewers_objs, $new_users);
+        _update_reviewers($object, $old_object->reviewers_objs(1), $new_users);
         $changes->{reviewers} = [ $old ? $old : undef, $new ? $new : undef ];
     }
 }
@@ -304,14 +305,14 @@ sub page_before_template {
         my $product = {
             name       => $product_obj->name,
             components => [],
-            reviewers  => $product_obj->reviewers_objs,
+            reviewers  => $product_obj->reviewers_objs(1),
         };
         $has_reviewers = scalar @{ $product->{reviewers} };
 
         foreach my $component_obj (@{ $product_obj->components }) {
             my $component = {
                 name       => $component_obj->name,
-                reviewers  => $component_obj->reviewers_objs,
+                reviewers  => $component_obj->reviewers_objs(1),
             };
             if (@{ $component->{reviewers} }) {
                 push @{ $product->{components} }, $component;
diff --git a/extensions/Review/template/en/default/hook/admin/components/edit-common-rows.html.tmpl b/extensions/Review/template/en/default/hook/admin/components/edit-common-rows.html.tmpl
index befb183b2..297b08506 100644
--- a/extensions/Review/template/en/default/hook/admin/components/edit-common-rows.html.tmpl
+++ b/extensions/Review/template/en/default/hook/admin/components/edit-common-rows.html.tmpl
@@ -12,11 +12,11 @@
     [% INCLUDE global/userselect.html.tmpl
        id => "reviewers"
        name => "reviewers"
-       value => comp.reviewers
+       value => comp.reviewers(1)
        size => 64
        emptyok => 1
        title => "One or more email address (comma delimited)"
-       placeholder => product.reviewers
+       placeholder => product.reviewers(1)
     %]
   </td>
 </tr>
diff --git a/extensions/Review/template/en/default/hook/admin/products/edit-common-rows.html.tmpl b/extensions/Review/template/en/default/hook/admin/products/edit-common-rows.html.tmpl
index da41f7dde..61a275e72 100644
--- a/extensions/Review/template/en/default/hook/admin/products/edit-common-rows.html.tmpl
+++ b/extensions/Review/template/en/default/hook/admin/products/edit-common-rows.html.tmpl
@@ -19,7 +19,7 @@
     [% INCLUDE global/userselect.html.tmpl
        id => "reviewers"
        name => "reviewers"
-       value => product.reviewers
+       value => product.reviewers(1)
        size => 64
        emptyok => 1
        title => "One or more email address (comma delimited)"
-- 
cgit v1.2.3-24-g4f1b