Bug 896320: increase the visibility of review suggestions (report, webservice)

3 files changed, 98 insertions, 0 deletions

diff --git a/extensions/Review/template/en/default/hook/global/user-error-errors.html.tmpl b/extensions/Review/template/en/default/hook/global/user-error-errors.html.tmpl
index 4265f4b32..856ff3c75 100644
--- a/extensions/Review/template/en/default/hook/global/user-error-errors.html.tmpl
+++ b/extensions/Review/template/en/default/hook/global/user-error-errors.html.tmpl
@@ -9,4 +9,10 @@
 [% IF error == "reviewer_required" %]
   [% title = "Reviewer Required" %]
   You must provide a reviewer for review requests.
+
+[% ELSIF error == "reviewer_suggestions_param_required" %]
+  [% title = "Parameter Required" %]
+  You must provide either a bug_id, or a product (and optionally a
+  component).
+
 [% END %]
diff --git a/extensions/Review/template/en/default/hook/reports/menu-end.html.tmpl b/extensions/Review/template/en/default/hook/reports/menu-end.html.tmpl
new file mode 100644
index 000000000..7396e4fcb
--- /dev/null
+++ b/extensions/Review/template/en/default/hook/reports/menu-end.html.tmpl
@@ -0,0 +1,16 @@
+[%# This Source Code Form is subject to the terms of the Mozilla Public
+  # License, v. 2.0. If a copy of the MPL was not distributed with this
+  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
+  #
+  # This Source Code Form is "Incompatible With Secondary Licenses", as
+  # defined by the Mozilla Public License, v. 2.0.
+  #%]
+
+<ul>
+  <li>
+    <strong>
+      <a href="[% urlbase FILTER none %]page.cgi?id=review_suggestions.html">Suggested Reviers</a>
+    </strong> - All suggestions for the "review" flag.
+  </li>
+</ul>
+
diff --git a/extensions/Review/template/en/default/pages/review_suggestions.html.tmpl b/extensions/Review/template/en/default/pages/review_suggestions.html.tmpl
new file mode 100644
index 000000000..5d9132e40
--- /dev/null
+++ b/extensions/Review/template/en/default/pages/review_suggestions.html.tmpl
@@ -0,0 +1,76 @@
+[%# This Source Code Form is subject to the terms of the Mozilla Public
+  # License, v. 2.0. If a copy of the MPL was not distributed with this
+  # file, You can obtain one at http://mozilla.org/MPL/2.0/.
+  #
+  # This Source Code Form is "Incompatible With Secondary Licenses", as
+  # defined by the Mozilla Public License, v. 2.0.
+  #%]
+
+[% INCLUDE global/header.html.tmpl
+  title = "Suggested Reviewers Report"
+  style_urls = [ "extensions/BMO/web/styles/reports.css",
+                 "extensions/Review/web/styles/reports.css" ]
+%]
+
+Products:
+<ul>
+  [% FOREACH product = products %]
+    <li>
+      <a href="#[% product.name FILTER uri %]">
+        [% product.name FILTER html %]
+      </a>
+    </li>
+  [% END %]
+</ul>
+
+<a href="enter_bug.cgi?product=bugzilla.mozilla.org&amp;component=Administration&amp;format=__default__">Request a change</a>
+
+<table id="report" class="hover" cellspacing="0">
+
+<tr id="report-header">
+  <th>Product/Component</th>
+  <th>Suggested Reviewers</th>
+</tr>
+
+[% FOREACH product = products %]
+  <tr class="report_subheader">
+    <td class="product_name">
+      <a name="[% product.name FILTER html %]">
+        [% product.name FILTER html %]
+      </a>
+    </td>
+    <td>
+    </td>
+  </tr>
+  [% row_class = "report_row_even" %]
+  [% FOREACH component = product.components %]
+    <tr class="[% row_class FILTER none %]">
+      <td class="component_name">[% component.name FILTER html %]</td>
+      <td class="reviewers">
+        [% FOREACH reviewer = component.reviewers %]
+          <span title="[% reviewer.name FILTER html %]">
+          [% reviewer.email FILTER html %]</span>
+          [% ", " UNLESS loop.last %]
+        [% END %]
+      </td>
+    </tr>
+    [% row_class = row_class == "report_row_even" ? "report_row_odd" : "report_row_even" %]
+  [% END %]
+  [% IF product.reviewers.size %]
+    <tr class="[% row_class FILTER none %]">
+      <td class="other_components">All [% product.components.size ? "other" : "" %] components</td>
+      <td class="reviewers">
+        [% FOREACH reviewer = product.reviewers %]
+          <span title="[% reviewer.name FILTER html %]">
+          [% reviewer.email FILTER html %]</span>
+          [% ", " UNLESS loop.last %]
+        [% END %]
+      </td>
+    </tr>
+    [% row_class = row_class == "report_row_even" ? "report_row_odd" : "report_row_even" %]
+  [% END %]
+[% END %]
+
+</table>
+
+[% INCLUDE global/footer.html.tmpl %]