X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=maintenance%2FdeleteOldRevisions.php;h=56576d908f6741aa4f07863887dc94b8f55defa3;hb=ae031e237eab8e2023d0fe128f9749a8a43ea439;hp=123163fb2b2286f1452c9cefde0da2568595d6fe;hpb=74174fca253b9a759d7402f09b7fc37d5de0e9be;p=lhc%2Fweb%2Fwiklou.git

diff --git a/maintenance/deleteOldRevisions.php b/maintenance/deleteOldRevisions.php
index 123163fb2b..56576d908f 100644
--- a/maintenance/deleteOldRevisions.php
+++ b/maintenance/deleteOldRevisions.php
@@ -42,14 +42,13 @@ class DeleteOldRevisions extends Maintenance {
 		$this->doDelete( $this->hasOption( 'delete' ), $this->mArgs );
 	}
 
-	function doDelete( $delete = false, $args = array() ) {
-
+	function doDelete( $delete = false, $args = [] ) {
 		# Data should come off the master, wrapped in a transaction
 		$dbw = $this->getDB( DB_MASTER );
 		$this->beginTransaction( $dbw, __METHOD__ );
 
-		$pageConds = array();
-		$revConds = array();
+		$pageConds = [];
+		$revConds = [];
 
 		# If a list of page_ids was provided, limit results to that set of page_ids
 		if ( count( $args ) > 0 ) {
@@ -61,7 +60,7 @@ class DeleteOldRevisions extends Maintenance {
 		# Get "active" revisions from the page table
 		$this->output( "Searching for active revisions..." );
 		$res = $dbw->select( 'page', 'page_latest', $pageConds, __METHOD__ );
-		$latestRevs = array();
+		$latestRevs = [];
 		foreach ( $res as $row ) {
 			$latestRevs[] = $row->page_latest;
 		}
@@ -73,7 +72,7 @@ class DeleteOldRevisions extends Maintenance {
 			$revConds[] = 'rev_id NOT IN (' . $dbw->makeList( $latestRevs ) . ')';
 		}
 		$res = $dbw->select( 'revision', 'rev_id', $revConds, __METHOD__ );
-		$oldRevs = array();
+		$oldRevs = [];
 		foreach ( $res as $row ) {
 			$oldRevs[] = $row->rev_id;
 		}
@@ -86,11 +85,11 @@ class DeleteOldRevisions extends Maintenance {
 		# Delete as appropriate
 		if ( $delete && $count ) {
 			$this->output( "Deleting..." );
-			$dbw->delete( 'revision', array( 'rev_id' => $oldRevs ), __METHOD__ );
+			$dbw->delete( 'revision', [ 'rev_id' => $oldRevs ], __METHOD__ );
+			$dbw->delete( 'ip_changes', [ 'ipc_rev_id' => $oldRevs ], __METHOD__ );
 			$this->output( "done.\n" );
 		}
 
-		# This bit's done
 		# Purge redundant text records
 		$this->commitTransaction( $dbw, __METHOD__ );
 		if ( $delete ) {
@@ -99,5 +98,5 @@ class DeleteOldRevisions extends Maintenance {
 	}
 }
 
-$maintClass = "DeleteOldRevisions";
+$maintClass = DeleteOldRevisions::class;
 require_once RUN_MAINTENANCE_IF_MAIN;