Package: lists.debian.org
Severity: wishlist
Tags: patch

As a follow-up for #563289, I propose a patch which allows marking all
nominations displayed in a page as unsure/ham/inappropriate/spam.

I find this quite useful nowadays, as almost every message I encounter
is spam, and marking them all as such speeds up my reviews.

My patch is based on review2.pl source found in bug #563289, I'm willing
to adjust patch in case script isn't accurate anymore.

-- 
  .''`.
 :  :' :   Luca Falavigna <[email protected]>
 `.  `'
   `-
--- review2.pl.orig	2011-09-04 11:27:10.000000000 +0200
+++ review2.pl	2011-09-04 11:30:16.685162636 +0200
@@ -64,6 +64,18 @@
     case 70: // f
       elPrefix = 's';
       break;
+    case 90: // z
+      fullPrefix = 'u';
+      break;
+    case 88: // x
+      fullPrefix = 'h';
+      break;
+    case 67: // c
+      fullPrefix = 'i';
+      break;
+    case 86: // v
+      fullPrefix = 's';
+      break;
   }
 
   if (1 <= pos && pos <= $count_out) {
@@ -74,6 +86,15 @@
     document.getElementById(elPrefix+pos).checked = true;
   }
 
+  if (fullPrefix != '') {
+    for (i = 0; i < 10; i++) {
+      id = document.getElementById(fullPrefix+i);
+      if (id) {
+        id.checked = true;
+      }
+    }
+  }
+
   return document.defaultAction;
 }
 // -->
@@ -109,7 +130,7 @@
 
   print "
 	<p><small>
-		Keys: <b>j</b>/<b>k</b> move down/up, <b>a</b>/<b>s</b>/<b>d</b>/<b>f</b> mark current entry as Unsure/Ham/Inappropriate/Spam respectively
+		Keys: <b>j</b>/<b>k</b> move down/up, <b>a</b>/<b>s</b>/<b>d</b>/<b>f</b> mark current entry as Unsure/Ham/Inappropriate/Spam respectively, <b>z</b>/<b>x</b>/<b>c</b>/<b>v</b> mark all entries as Unsure/Ham/Inappropriate/Spam respectively
 	</small></p>\n";
 
   print "<form action=\"/cgi-bin/review/review3.pl\" method=\"GET\">\n";

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to