class DeleteOldRevisions extends Maintenance {
public function __construct() {
parent::__construct();
- $this->mDescription = "Delete old (non-current) revisions from the database";
+ $this->addDescription( 'Delete old (non-current) revisions from the database' );
$this->addOption( 'delete', 'Actually perform the deletion' );
$this->addOption( 'page_id', 'List of page ids to work on', false );
}
$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 = wfGetDB( DB_MASTER );
- $dbw->begin( __METHOD__ );
+ $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 ) {
# 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;
}
$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;
}
# 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
- $dbw->commit( __METHOD__ );
+ $this->commitTransaction( $dbw, __METHOD__ );
if ( $delete ) {
$this->purgeRedundantText( true );
}