Revision: 48542
Author:   werdna
Date:     2009-03-18 23:24:10 +0000 (Wed, 18 Mar 2009)

Log Message:
-----------
Report createaccounts as 'user creating own accounts', rather than as 'IP 
creating user'

Modified Paths:
--------------
    trunk/extensions/AbuseFilter/AbuseFilter.class.php

Modified: trunk/extensions/AbuseFilter/AbuseFilter.class.php
===================================================================
--- trunk/extensions/AbuseFilter/AbuseFilter.class.php  2009-03-18 21:32:58 UTC 
(rev 48541)
+++ trunk/extensions/AbuseFilter/AbuseFilter.class.php  2009-03-18 23:24:10 UTC 
(rev 48542)
@@ -524,6 +524,7 @@
                        
                $var_dump = self::storeVarDump( $vars );
                $var_dump = "stored-text:$var_dump"; // To distinguish from 
stuff stored directly
+               $action = $vars->getVar( 'ACTION' )->toString();
 
                // Create a template
                $log_template = array(
@@ -534,8 +535,13 @@
                        'afl_namespace' => $title->getNamespace(),
                        'afl_title' => $title->getDBKey(),
                        'afl_ip' => wfGetIp() );
+               
+               // Hack to avoid revealing IPs of people creating accounts
+               if (!$wgUser->getId() && $action == 'createaccount') {
+                       $log_template['afl_user_text'] = $vars->getVar( 
'accountname' )->toString();
+               }
 
-               self::addLogEntries( $actions_taken, $log_template, 
$vars->getVar( 'ACTION' )->toString() );
+               self::addLogEntries( $actions_taken, $log_template, $action );
 
                $error_msg = $error_msg == '' ? true : $error_msg;
                



_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to