X-Git-Url: https://git.cyclocoop.org/?a=blobdiff_plain;f=maintenance%2FremoveUnusedAccounts.php;h=1215a4541e4e391dabf92765b63eb4099db91757;hb=5cd45f79d1b9164e87040da8d9a204e19fa2938b;hp=9166a40441efbd178877c7f8ef51c663bcbc3081;hpb=971d902bd482bcafc3376c529552f3ee25787175;p=lhc%2Fweb%2Fwiklou.git

diff --git a/maintenance/removeUnusedAccounts.php b/maintenance/removeUnusedAccounts.php
index 9166a40441..1215a4541e 100644
--- a/maintenance/removeUnusedAccounts.php
+++ b/maintenance/removeUnusedAccounts.php
@@ -1,60 +1,110 @@
 <?php
-
 /**
  * Remove unused user accounts from the database
  * An unused account is one which has made no edits
  *
- * @package MediaWiki
- * @subpackage Maintenance
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
+ * @ingroup Maintenance
  * @author Rob Church <robchur@gmail.com>
  */
 
-define( 'ACTION_REPORT', 0 );
-define( 'ACTION_DELETE', 1 );
-$options = array( 'delete','help' );
-require_once( 'commandLine.inc' );
-require_once( 'userFunctions.inc' );
+require_once( dirname( __FILE__ ) . '/Maintenance.php' );
 
-echo( "Remove Unused Accounts\nThis script will delete all users who have made no edits and uploaded no files.\n\n" );
-
-# Check parameters
-if( @$options['help'] ) {
-	echo( "USAGE: removeUnusedAccounts.php [--help|--delete]\n\nThe first (default) account is ignored.\n\n" );
-	wfDie();
-} else {
-	$delete = @$options['delete'] ? true : false ;
-}
+class RemoveUnusedAccounts extends Maintenance {
+	public function __construct() {
+		parent::__construct();
+		$this->addOption( 'delete', 'Actually delete the account' );
+		$this->addOption( 'ignore-groups', 'List of comma-separated groups to exclude', false, true );
+		$this->addOption( 'ignore-touched', 'Skip accounts touched in last N days', false, true );
+	}
 
-$count = 0;
-$del = array();
+	public function execute() {
 
-# Right, who needs deleting?
-$users = GetUsers();
-echo( "Found " . count( $users ) . " accounts.\n\n" );
-echo( "Locating inactive users..." );
-foreach( $users as $user ) {
-	if( $user != 1 ) {	# Don't *touch* the first user account, ever
-		if( CountEdits( $user, false ) == 0 && CountImages( $user, false ) == 0 && CountLogs( $user, false ) == 0 ) {
-			# User has no edits or images, mark them for deletion
-			$del[] = $user;
-			$count++;
+		$this->output( "Remove unused accounts\n\n" );
+		
+		# Do an initial scan for inactive accounts and report the result
+		$this->output( "Checking for unused user accounts...\n" );
+		$del = array();
+		$dbr = wfGetDB( DB_SLAVE );
+		$res = $dbr->select( 'user', array( 'user_id', 'user_name', 'user_touched' ), '', __METHOD__ );
+		if ( $this->hasOption( 'ignore-groups' ) ) {
+			$excludedGroups = explode( ',', $this->getOption( 'ignore-groups' ) );
+		} else {
+			$excludedGroups = array();
 		}
+		$touched = $this->getOption( 'ignore-touched', "1" );
+		if ( !ctype_digit( $touched ) ) {
+			$this->error( "Please put a valid positive integer on the --ignore-touched parameter.", true );
+		}
+		$touchedSeconds = 86400 * $touched;
+		foreach ( $res as $row ) {
+			# Check the account, but ignore it if it's within a $excludedGroups group or if it's touched within the $touchedSeconds seconds.
+			$instance = User::newFromId( $row->user_id );
+			if ( count( array_intersect( $instance->getEffectiveGroups(), $excludedGroups ) ) == 0
+				&& $this->isInactiveAccount( $row->user_id, true )
+				&& wfTimestamp( TS_UNIX, $row->user_touched ) < wfTimestamp( TS_UNIX, time() - $touchedSeconds )
+				) {
+				# Inactive; print out the name and flag it
+				$del[] = $row->user_id;
+				$this->output( $row->user_name . "\n" );
+			}
+		}
+		$count = count( $del );
+		$this->output( "...found {$count}.\n" );
+	
+		# If required, go back and delete each marked account
+		if ( $count > 0 && $this->hasOption( 'delete' ) ) {
+			$this->output( "\nDeleting inactive accounts..." );
+			$dbw = wfGetDB( DB_MASTER );
+			$dbw->delete( 'user', array( 'user_id' => $del ), __METHOD__ );
+			$this->output( "done.\n" );
+			# Update the site_stats.ss_users field
+			$users = $dbw->selectField( 'user', 'COUNT(*)', array(), __METHOD__ );
+			$dbw->update( 'site_stats', array( 'ss_users' => $users ), array( 'ss_row_id' => 1 ), __METHOD__ );
+		} elseif ( $count > 0 ) {
+			$this->output( "\nRun the script again with --delete to remove them from the database.\n" );
+		}
+		$this->output( "\n" );
 	}
-}
-echo( "done.\n" );
-
-# Purge the inactive accounts we found
-echo( $count . " inactive accounts found.\n" );
-if( $count > 0 ) {
-	if( $delete ) {
-		echo( "Deleting..." );
-		DeleteUsers( $del );
-		echo( "done.\n" );
-	} else {
-		echo "Run the script with the --delete option to remove them from the database.\n";
+	
+	/**
+	 * Could the specified user account be deemed inactive?
+	 * (No edits, no deleted edits, no log entries, no current/old uploads)
+	 *
+	 * @param $id User's ID
+	 * @param $master Perform checking on the master
+	 * @return bool
+	 */
+	private function isInactiveAccount( $id, $master = false ) {
+		$dbo = wfGetDB( $master ? DB_MASTER : DB_SLAVE );
+		$checks = array( 'revision' => 'rev', 'archive' => 'ar', 'logging' => 'log',
+						 'image' => 'img', 'oldimage' => 'oi' );
+		$count = 0;
+	
+		$dbo->begin();
+		foreach ( $checks as $table => $fprefix ) {
+			$conds = array( $fprefix . '_user' => $id );
+			$count += (int)$dbo->selectField( $table, 'COUNT(*)', $conds, __METHOD__ );
+		}
+		$dbo->commit();
+	
+		return $count == 0;
 	}
 }
-		
-echo( "\n" );
 
-?>
+$maintClass = "RemoveUnusedAccounts";
+require_once( DO_MAINTENANCE );