From d35063273c264d9991e0cc47decf6976ca5f2844 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Tue, 22 Apr 2014 11:44:47 -0700 Subject: [PATCH] Added --multiversiononly option to image SHA1 population script * A portion of files have bad SHA1 values due to re-upload bugs. This adds an option to just force updates for files with re-uploads. Change-Id: I24bd9a8f19bafc3e794cf32c78ce7f720f302627 --- maintenance/populateImageSha1.php | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/maintenance/populateImageSha1.php b/maintenance/populateImageSha1.php index 126d22d986..4964bf116c 100644 --- a/maintenance/populateImageSha1.php +++ b/maintenance/populateImageSha1.php @@ -33,6 +33,7 @@ class PopulateImageSha1 extends LoggedUpdateMaintenance { parent::__construct(); $this->mDescription = "Populate the img_sha1 field"; $this->addOption( 'force', "Recalculate sha1 for rows that already have a value" ); + $this->addOption( 'multiversiononly', "Calculate only for files with several versions" ); $this->addOption( 'method', "Use 'pipe' to pipe to mysql command line,\n" . "\t\tdefault uses Database class", false, true ); $this->addOption( 'file', 'Fix for a specific file, without File: namespace prefixed', false, true ); @@ -47,7 +48,7 @@ class PopulateImageSha1 extends LoggedUpdateMaintenance { } public function execute() { - if ( $this->getOption( 'file' ) ) { + if ( $this->getOption( 'file' ) || $this->hasOption( 'multiversiononly' ) ) { $this->doDBUpdates(); // skip update log checks/saves } else { parent::execute(); @@ -82,7 +83,12 @@ class PopulateImageSha1 extends LoggedUpdateMaintenance { $conds = array( 'img_sha1' => '' ); $this->output( "Populating img_sha1 field\n" ); } - $res = $dbw->select( 'image', array( 'img_name' ), $conds, __METHOD__ ); + if ( $this->hasOption( 'multiversiononly' ) ) { + $res = $dbw->select( 'oldimage', + array( 'img_name' => 'DISTINCT(oi_name)' ), $conds, __METHOD__ ); + } else { + $res = $dbw->select( 'image', array( 'img_name' ), $conds, __METHOD__ ); + } } $imageTable = $dbw->tableName( 'image' ); @@ -109,10 +115,12 @@ class PopulateImageSha1 extends LoggedUpdateMaintenance { "Done %d of %d, %5.3f%% \r", $i, $numRows, $i / $numRows * 100 ) ); wfWaitForSlaves(); } + $file = wfLocalFile( $row->img_name ); if ( !$file ) { continue; } + // Upgrade the current file version... $sha1 = $file->getRepo()->getFileSha1( $file->getPath() ); if ( strval( $sha1 ) !== '' ) { // file on disk and hashed properly -- 2.20.1