* @ingroup Maintenance
*/
class WatchlistCleanup extends TableCleanup {
- protected $defaultParams = array(
+ protected $defaultParams = [
'table' => 'watchlist',
- 'index' => array( 'wl_user', 'wl_namespace', 'wl_title' ),
- 'conds' => array(),
+ 'index' => [ 'wl_user', 'wl_namespace', 'wl_title' ],
+ 'conds' => [],
'callback' => 'processRow'
- );
+ ];
public function __construct() {
parent::__construct();
- $this->mDescription = "Script to remove broken, unparseable titles in the Watchlist";
+ $this->addDescription( 'Script to remove broken, unparseable titles in the Watchlist' );
$this->addOption( 'fix', 'Actually remove entries; without will only report.' );
}
$title = Title::newFromText( $verified );
if ( $row->wl_user == 0 || is_null( $title ) || !$title->equals( $current ) ) {
- $this->output( "invalid watch by {$row->wl_user} for ({$row->wl_namespace}, \"{$row->wl_title}\")\n" );
+ $this->output( "invalid watch by {$row->wl_user} for "
+ . "({$row->wl_namespace}, \"{$row->wl_title}\")\n" );
$updated = $this->removeWatch( $row );
$this->progress( $updated );
+
return;
}
$this->progress( 0 );
private function removeWatch( $row ) {
if ( !$this->dryrun && $this->hasOption( 'fix' ) ) {
- $dbw = wfGetDB( DB_MASTER );
- $dbw->delete( 'watchlist', array(
+ $dbw = $this->getDB( DB_MASTER );
+ $dbw->delete(
+ 'watchlist', [
'wl_user' => $row->wl_user,
'wl_namespace' => $row->wl_namespace,
- 'wl_title' => $row->wl_title ),
- __METHOD__ );
+ 'wl_title' => $row->wl_title ],
+ __METHOD__
+ );
+
$this->output( "- removed\n" );
+
return 1;
} else {
return 0;