summaryrefslogtreecommitdiff
path: root/maintenance/removeUnusedAccounts.php
diff options
context:
space:
mode:
Diffstat (limited to 'maintenance/removeUnusedAccounts.php')
-rw-r--r--maintenance/removeUnusedAccounts.php16
1 files changed, 11 insertions, 5 deletions
diff --git a/maintenance/removeUnusedAccounts.php b/maintenance/removeUnusedAccounts.php
index 16cb17ab..90dc6220 100644
--- a/maintenance/removeUnusedAccounts.php
+++ b/maintenance/removeUnusedAccounts.php
@@ -58,12 +58,13 @@ class RemoveUnusedAccounts extends Maintenance {
}
$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.
+ # 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" );
@@ -85,7 +86,12 @@ class RemoveUnusedAccounts extends Maintenance {
$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__ );
+ $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" );
}
@@ -96,8 +102,8 @@ class RemoveUnusedAccounts extends Maintenance {
* 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 bool Perform checking on the master
+ * @param int $id User's ID
+ * @param bool $master Perform checking on the master
* @return bool
*/
private function isInactiveAccount( $id, $master = false ) {