(bug 40829) Show cascading protection info on action=info

[lhc/web/wiklou.git] / maintenance / removeUnusedAccounts.php

diff --git a/maintenance/removeUnusedAccounts.php b/maintenance/removeUnusedAccounts.php
index 7b0535b..8bc27c1 100644 (file)
--- a/maintenance/removeUnusedAccounts.php
+++ b/maintenance/removeUnusedAccounts.php
@@ -18,12 +18,18 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  * http://www.gnu.org/copyleft/gpl.html
  *
+ * @file
  * @ingroup Maintenance
  * @author Rob Church <robchur@gmail.com>
  */
 
-require_once( dirname(__FILE__) . '/Maintenance.php' );
+require_once( __DIR__ . '/Maintenance.php' );
 
+/**
+ * Maintenance script that removes unused user accounts from the database.
+ *
+ * @ingroup Maintenance
+ */
 class RemoveUnusedAccounts extends Maintenance {
        public function __construct() {
                parent::__construct();
@@ -35,26 +41,26 @@ class RemoveUnusedAccounts extends Maintenance {
        public function execute() {
 
                $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 { 
+               if ( $this->hasOption( 'ignore-groups' ) ) {
+                       $excludedGroups = explode( ',', $this->getOption( 'ignore-groups' ) );
+               } else {
                        $excludedGroups = array();
                }
                $touched = $this->getOption( 'ignore-touched', "1" );
-               if( !ctype_digit( $touched ) ) {
+               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 ) {
+               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
+                       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 )
                                ) {
@@ -65,9 +71,9 @@ class RemoveUnusedAccounts extends Maintenance {
                }
                $count = count( $del );
                $this->output( "...found {$count}.\n" );
-       
+
                # If required, go back and delete each marked account
-               if( $count > 0 && $this->hasOption('delete') ) {
+               if ( $count > 0 && $this->hasOption( 'delete' ) ) {
                        $this->output( "\nDeleting inactive accounts..." );
                        $dbw = wfGetDB( DB_MASTER );
                        $dbw->delete( 'user', array( 'user_id' => $del ), __METHOD__ );
@@ -75,36 +81,36 @@ class RemoveUnusedAccounts extends Maintenance {
                        # 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 ) {
+               } elseif ( $count > 0 ) {
                        $this->output( "\nRun the script again with --delete to remove them from the database.\n" );
                }
                $this->output( "\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
+        * @param $master bool 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' );
+                                                'image' => 'img', 'oldimage' => 'oi', 'filearchive' => 'fa' );
                $count = 0;
-       
-               $dbo->begin();
-               foreach( $checks as $table => $fprefix ) {
+
+               $dbo->begin( __METHOD__ );
+               foreach ( $checks as $table => $fprefix ) {
                        $conds = array( $fprefix . '_user' => $id );
                        $count += (int)$dbo->selectField( $table, 'COUNT(*)', $conds, __METHOD__ );
                }
-               $dbo->commit();
-       
+               $dbo->commit( __METHOD__ );
+
                return $count == 0;
        }
 }
 
 $maintClass = "RemoveUnusedAccounts";
-require_once( DO_MAINTENANCE );
+require_once( RUN_MAINTENANCE_IF_MAIN );