Changeset 44626

Show
Ignore:
Timestamp:
05/07/08 05:14:24 (2 months ago)
Author:
ear1grey
Message:

Improved the form content so that only items that are above a threshold* get deleted in the "All" screen. Additional items may be visible in each tab to allow the user to inspect spam that's not necessarily a worst offender, but that looks like it might be. e.g. the multilink threshold for automatic deletion might be 7 links, but comments with more than 4 links could be shown, and thus any entries with 4, 5 or 6 links would not be automatically ticked.
(*threshold is not yet UI configurable)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • worst-offenders/trunk/classes/litmus.php

    r43791 r44626  
    7171 
    7272      public static function wo3_show_row($num, $reason_or_offender, $ids) { 
     73            $preselect_threshold = 3; // should be an option 
     74            $checked = ($num >= $preselect_threshold ? "checked" : ""); 
    7375            if (AllLitmus::isActive()) { 
    74                   echo("<input id='$hids' type='hidden' name='worst[]' checked value='$ids' />"); 
     76                  echo("<input style='visibility:hidden;display:none;' id='$hids' type='checkbox' name='worst[]' $checked value='$ids' />"); 
    7577            } else { 
    76                   $preselect_threshold = 3; // should be an option 
    77                   $checked = ($num >= $preselect_threshold ? "checked" : ""); 
    7878                  echo("<tr>"); 
    7979                  echo("<td><input id='$hids' type='checkbox' name='worst[]' $checked value='$ids' /></td>");