chiark / gitweb /
Remove the blacklist of posters
authorMatthew Vernon <matthew@debian.org>
Sun, 5 Jan 2014 13:28:25 +0000 (13:28 +0000)
committerMatthew Vernon <matthew@debian.org>
Sun, 5 Jan 2014 13:28:25 +0000 (13:28 +0000)
The ULM mods want a watchlist for posters, not a blacklist. So remove
the blacklisted posters list, and replace the option to blacklist a
poster with the option to add them to the watch list.

webstump/doc/help/filter-lists.html
webstump/scripts/filter.lib.pl
webstump/scripts/html_output.pl
webstump/scripts/webstump.lib.pl

index fe7fad74165650b2abeb989de14870a4a5beb8f1..6a1199cc7bf4276fd2badefa87dd413988137931 100644 (file)
@@ -15,9 +15,6 @@ The algorithm for every incoming article is as follows:
 
 <OL>
 
-  <LI> Reject message if the author is listed in <A
-       HREF=##bad.posters.list>the List of Banned Posters.</A>.
-
   <LI> File message for human review if the author is listed in <A
        HREF=##watch.posters.list>the List of Untrusted Posters</A>.
 
index b82a55a52e836b46979827242141b983c6ca66e8..891f7c926f7a17b4774db47f19c57b26f123342a 100644 (file)
@@ -92,11 +92,6 @@ sub review_incoming_message { # Newsgroup, From, Subject, RealSubject, Message,
   my $from = pop( @_ );
   my $newsgroup = pop( @_ );
 
-  if( &name_is_in_list( $from, "bad.posters.list" ) ) {
-    &process_approval_decision( $subject, $newsgroup, $dir, "reject blocklist", "", "auto bad poster" );
-    return;
-  }
-
   my $warning_file = &article_file_name( $dir ) . "/stump-warning.txt";
   my $match;
 
index 505ff3251d3920d07797411c9418e70e10fa0d5e..61f2b13817dbeb1275c21b5551711982d10b5817 100644 (file)
@@ -258,20 +258,18 @@ sub html_moderate_article {
 <INPUT TYPE=radio NAME=poster_decision VALUE=nothing CHECKED>Don't change poster's status</INPUT>
 <INPUT TYPE=radio NAME=poster_decision VALUE=preapprove 
 >White-list poster</INPUT>
-<INPUT TYPE=radio NAME=poster_decision VALUE=ban 
-  ONCLICK=\"alert( 'Banning a poster is a controversial practice'); \"
-> Ban All Posts by this Person (Careful!)</INPUT>
+<INPUT TYPE=radio NAME=poster_decision VALUE=suspicious>
+Add poster to watch list</INPUT>
 
 <BR><BR>
 <I>
-NOTE: Decisions to ban and whitelist posters can be reversed by 
-logging in as \"admin\" and editing respective lists of whitelisted
-and banned threads.
+NOTE: Decisions to watchlist and whitelist posters can be reversed by 
+editing the respective lists of whitelisted and watchlisted posters.
 ";
 
   &link_to_help( "filter-lists", "automatic filtering and filter lists, blacklisting and preapproved threads." );
 
-  print "Be really careful about blacklisting of everyone except spammers.</I><BR><BR>
+  print "</I><BR><BR>
 
 <INPUT TYPE=radio NAME=next_screen VALUE=single CHECKED> 
        Review ONE article in next screen
@@ -755,7 +753,6 @@ sub html_newsgroup_management {
 
     <OPTION VALUE=good.posters.list>Good Posters List
     <OPTION VALUE=watch.posters.list>Suspicious Posters List
-    <OPTION VALUE=bad.posters.list>Banned Posters List
     <OPTION VALUE=watch.words.list>Suspicious Words List
 
   </SELECT>
index 379cf0602bbf87c61b2c393d2e5ac02195baa4d2..45ff3d7dae41c4d56a95f5d67a570e23884411bb 100644 (file)
@@ -398,7 +398,6 @@ sub check_config_list {
  &user_error( "invalid list name $list_to_edit" )
     if( $list_to_edit ne "good.posters.list"
         && $list_to_edit ne "watch.posters.list"
-        && $list_to_edit ne "bad.posters.list"
         && $list_to_edit ne "watch.words.list" );
 
   return &untaint( $list_to_edit );