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

Hi,

I would like to propose the following patch for
archive-spam-removals/cgi-bin/review/review2.pl, please.

It seems there is an off-by-one error with the 'a/s/d/f' keyboard
shortcuts.

Without the patch, I can't use the shortcuts on the last message of the
list. On the other items but the last, the shortcuts modify the status
of the previous message instead of the current one.

Thank you.
--- review2.pl.old	2016-10-07 17:17:48.000000000 -0300
+++ review2.pl	2016-10-07 17:18:07.000000000 -0300
@@ -86,7 +86,7 @@
   }
 
   if (elPrefix != '') {
-    document.getElementById(elPrefix+pos).checked = true;
+    document.getElementById(elPrefix+(pos-1)).checked = true;
   }
 
   if (fullPrefix != '') {

Attachment: signature.asc
Description: Digital signature

Reply via email to