From 0a5f752f24ab8c8305bfcb05d10cefe511588db9 Mon Sep 17 00:00:00 2001
From: Jason Tibbitts <tibbs@math.uh.edu>
Date: Tue, 12 Jun 2012 21:04:09 -0500
Subject: [PATCH] Add currently under review report

---
 scripts/review-stats/review-stats.py          | 27 +++++++++++++++++++
 .../review-stats/review-templates/index.html  |  4 ++-
 2 files changed, 30 insertions(+), 1 deletion(-)

diff --git a/scripts/review-stats/review-stats.py b/scripts/review-stats/review-stats.py
index d405a49..90f4ec3 100755
--- a/scripts/review-stats/review-stats.py
+++ b/scripts/review-stats/review-stats.py
@@ -234,6 +234,11 @@ def select_needsponsor(bug, bugd):
         return 1
     return 0
 
+def select_review(bug, bugd):
+    if bugd['reviewflag'] == '?':
+        return 1
+    return 0
+
 def select_trivial(bug, bugd):
     if (bugd['reviewflag'] == ' '
             and bug.bug_status != 'CLOSED'
@@ -299,6 +304,24 @@ def report_hidden(bugs, bugdata, loader, tmpdir, subs):
 
     return data['count']
 
+def report_review(bugs, bugdata, loader, tmpdir, subs):
+    data = deepcopy(subs)
+    data['description'] = 'This page lists tickets currently under review'
+    data['title'] = 'Tickets under review'
+
+    for i in bugs:
+        if select_review(i, bugdata[i.bug_id]):
+            rowclass = 'bz_row_even'
+            if data['count'] % 2 == 1:
+                rowclass = 'bz_row_odd'
+
+            data['bugs'].append(std_row(i, rowclass))
+            data['count'] +=1
+
+    write_html(loader, 'plain.html', data, tmpdir, 'REVIEW.html')
+
+    return data['count']
+
 def report_trivial(bugs, bugdata, loader, tmpdir, subs):
     data = deepcopy(subs)
     data['description'] = 'This page lists review tickets marked as trivial'
@@ -465,11 +488,14 @@ if __name__ == '__main__':
             }
     args = {'bugs':bugs, 'bugdata':bugdata, 'loader':loader, 'tmpdir':tmpdir, 'subs':subs}
 
+    t = time.time()
+
     subs['new'] =         report_new(**args)
     subs['epel'] =        report_epel(**args)
     subs['hidden'] =      report_hidden(**args)
     subs['merge'] =       report_merge(**args)
     subs['needsponsor'] = report_needsponsor(**args)
+    subs['review'] =      report_review(**args)
     subs['trivial'] =     report_trivial(**args)
 #    data['accepted_closed'] = report_accepted_closed(bugs, bugdata, loader, tmpdir)
 #    data['accepted_open'] = report_accepted_open(bugs, bugdata, loader, tmpdir)
@@ -477,6 +503,7 @@ if __name__ == '__main__':
 #    data['rejected_open'] = report_rejected_open(bugs, bugdata, loader, tmpdir)
 #    data['review_closed'] = report_review_closed(bugs, bugdata, loader, tmpdir)
 #    data['review_open'] = report_review_open(bugs, bugdata, loader, tmpdir)
+    subs['outputtime'] = time.time() - t
     write_html(loader, 'index.html', subs, tmpdir, 'index.html')
 
     for filename in glob.glob(os.path.join(tmpdir, '*')):
diff --git a/scripts/review-stats/review-templates/index.html b/scripts/review-stats/review-templates/index.html
index 817ba9b..435e3c7 100644
--- a/scripts/review-stats/review-templates/index.html
+++ b/scripts/review-stats/review-templates/index.html
@@ -47,8 +47,10 @@ available:
     <td>All merge review tickets.  Please see <a href="https://fedoraproject.org/wiki/Merge_Reviews">this page</a> for more information on merge reviews.</td></tr>
 <tr><td style="white-space: nowrap"><a href="HIDDEN.html">Hidden tickets</a> ($hidden)</td>
     <td>Tickets which have been hidden for some reason.  These tickets either depend on other review tickets which have not yet been closed, or are unreviewable for some reason.  See <a href="https://fedoraproject.org/wiki/Package_Review_Process#Special_blocker_tickets">this page</a> for more information on the various states a review ticket can have.</td></tr>
+    <tr><td style="white-space: nowrap"><a href="REVIEW.html">Tickets under review</a> ($review)</td>
+    <td>All tickets currently under review.</td></tr>
 </table>
-Last updated: ${update}, query time: ${'{:0.3f}'.format(querytime)}, version: ${version}.
+Last updated: ${update}, query time: ${'{:0.3f}'.format(querytime)}s, output time: ${'{:0.3f}'.format(outputtime)}s, version: ${version}.
 <br/>
     <h2>Quick Review Search</h2>
     <form name="queryform" action="http://bugzilla.redhat.com/buglist.cgi" method="get"