Merge commit 'fa89acd'

# Conflicts:
#	.gitignore
#	extensions/ArchInterWiki.sql

1 files changed, 15 insertions, 13 deletions

diff --git a/maintenance/deleteArchivedRevisions.php b/maintenance/deleteArchivedRevisions.php
index 30883ba4..9924eb0c 100644
--- a/maintenance/deleteArchivedRevisions.php
+++ b/maintenance/deleteArchivedRevisions.php
@@ -25,7 +25,6 @@
  */
 
 require_once __DIR__ . '/Maintenance.php';
-require_once __DIR__ . '/deleteArchivedRevisions.inc';
 
 /**
  * Maintenance script to delete archived (deleted from public) revisions
@@ -41,21 +40,24 @@ class DeleteArchivedRevisions extends Maintenance {
 		$this->addOption( 'delete', 'Performs the deletion' );
 	}
 
-	public function handleOutput( $str ) {
-		$this->output( $str );
-	}
-
 	public function execute() {
-		$this->output( "Delete archived revisions\n\n" );
-		# Data should come off the master, wrapped in a transaction
-		if ( $this->hasOption( 'delete' ) ) {
-			DeleteArchivedRevisionsImplementation::doDelete( $this );
-		} else {
-			$dbw = wfGetDB( DB_MASTER );
-			$res = $dbw->selectRow( 'archive', 'COUNT(*) as count', array(), __FUNCTION__ );
-			$this->output( "Found {$res->count} revisions to delete.\n" );
+		$dbw = $this->getDB( DB_MASTER );
+
+		if ( !$this->hasOption( 'delete' ) ) {
+			$count = $dbw->selectField( 'archive', 'COUNT(*)', '', __METHOD__ );
+			$this->output( "Found $count revisions to delete.\n" );
 			$this->output( "Please run the script again with the --delete option "
 				. "to really delete the revisions.\n" );
+			return;
+		}
+
+		$this->output( "Deleting archived revisions... " );
+		$dbw->delete( 'archive', '*', __METHOD__ );
+		$count = $dbw->affectedRows();
+		$this->output( "done. $count revisions deleted.\n" );
+
+		if ( $count ) {
+			$this->purgeRedundantText( true );
 		}
 	}
 }