/**
* Clean up broken, unparseable upload filenames.
*
- * Usage: php cleanupImages.php [--fix]
- * Options:
- * --fix Actually clean up titles; otherwise just checks for them
- *
* Copyright © 2005-2006 Brion Vibber <brion@pobox.com>
* https://www.mediawiki.org/
*
* @ingroup Maintenance
*/
class ImageCleanup extends TableCleanup {
- protected $defaultParams = array(
+ protected $defaultParams = [
'table' => 'image',
- 'conds' => array(),
+ 'conds' => [],
'index' => 'img_name',
'callback' => 'processRow',
- );
+ ];
public function __construct() {
parent::__construct();
- $this->mDescription = "Script to clean up broken, unparseable upload filenames";
+ $this->addDescription( 'Script to clean up broken, unparseable upload filenames' );
}
protected function processRow( $row ) {
if ( $source == '' ) {
// Ye olde empty rows. Just kill them.
$this->killRow( $source );
+
return $this->progress( 1 );
}
return $this->progress( 0 );
}
$this->pokeFile( $source, $safe );
+
return $this->progress( 1 );
}
$munged = $title->getDBkey();
$this->output( "page $source ($munged) doesn't match self.\n" );
$this->pokeFile( $source, $munged );
+
return $this->progress( 1 );
}
$this->output( "DRY RUN: would delete bogus row '$name'\n" );
} else {
$this->output( "deleting bogus row '$name'\n" );
- $db = wfGetDB( DB_MASTER );
+ $db = $this->getDB( DB_MASTER );
$db->delete( 'image',
- array( 'img_name' => $name ),
+ [ 'img_name' => $name ],
__METHOD__ );
}
}
if ( !isset( $this->repo ) ) {
$this->repo = RepoGroup::singleton()->getLocalRepo();
}
+
return $this->repo->getRootDirectory() . '/' . $this->repo->getHashPath( $name ) . $name;
}
private function imageExists( $name, $db ) {
- return $db->selectField( 'image', '1', array( 'img_name' => $name ), __METHOD__ );
+ return $db->selectField( 'image', '1', [ 'img_name' => $name ], __METHOD__ );
}
private function pageExists( $name, $db ) {
return $db->selectField(
'page',
'1',
- array( 'page_namespace' => NS_FILE, 'page_title' => $name ),
+ [ 'page_namespace' => NS_FILE, 'page_title' => $name ],
__METHOD__
);
}
if ( !file_exists( $path ) ) {
$this->output( "missing file: $path\n" );
$this->killRow( $orig );
+
return;
}
- $db = wfGetDB( DB_MASTER );
+ $db = $this->getDB( DB_MASTER );
/*
* To prevent key collisions in the update() statements below,
* if the target title exists in the image table, or if both the
* original and target titles exist in the page table, append
* increasing version numbers until the target title exists in
- * neither. (See also bug 16916.)
+ * neither. (See also T18916.)
*/
$version = 0;
$final = $new;
$conflict = ( $this->imageExists( $final, $db ) ||
- ( $this->pageExists( $orig, $db ) && $this->pageExists( $final, $db ) ) );
+ ( $this->pageExists( $orig, $db ) && $this->pageExists( $final, $db ) ) );
while ( $conflict ) {
$this->output( "Rename conflicts with '$final'...\n" );
} else {
$this->output( "renaming $path to $finalPath\n" );
// @todo FIXME: Should this use File::move()?
- $db->begin( __METHOD__ );
+ $this->beginTransaction( $db, __METHOD__ );
$db->update( 'image',
- array( 'img_name' => $final ),
- array( 'img_name' => $orig ),
+ [ 'img_name' => $final ],
+ [ 'img_name' => $orig ],
__METHOD__ );
$db->update( 'oldimage',
- array( 'oi_name' => $final ),
- array( 'oi_name' => $orig ),
+ [ 'oi_name' => $final ],
+ [ 'oi_name' => $orig ],
__METHOD__ );
$db->update( 'page',
- array( 'page_title' => $final ),
- array( 'page_title' => $orig, 'page_namespace' => NS_FILE ),
+ [ 'page_title' => $final ],
+ [ 'page_title' => $orig, 'page_namespace' => NS_FILE ],
__METHOD__ );
$dir = dirname( $finalPath );
if ( !file_exists( $dir ) ) {
if ( !wfMkdirParents( $dir, null, __METHOD__ ) ) {
$this->output( "RENAME FAILED, COULD NOT CREATE $dir" );
- $db->rollback( __METHOD__ );
+ $this->rollbackTransaction( $db, __METHOD__ );
+
return;
}
}
if ( rename( $path, $finalPath ) ) {
- $db->commit( __METHOD__ );
+ $this->commitTransaction( $db, __METHOD__ );
} else {
$this->error( "RENAME FAILED" );
- $db->rollback( __METHOD__ );
+ $this->rollbackTransaction( $db, __METHOD__ );
}
}
}
private function buildSafeTitle( $name ) {
$x = preg_replace_callback(
'/([^' . Title::legalChars() . ']|~)/',
- array( $this, 'hexChar' ),
+ [ $this, 'hexChar' ],
$name );
$test = Title::makeTitleSafe( NS_FILE, $x );
if ( is_null( $test ) || $test->getDBkey() !== $x ) {
$this->error( "Unable to generate safe title from '$name', got '$x'" );
+
return false;
}
}
}
-$maintClass = "ImageCleanup";
+$maintClass = ImageCleanup::class;
require_once RUN_MAINTENANCE_IF_MAIN;