diff options
Diffstat (limited to 'maintenance/deleteEqualMessages.php')
| -rw-r--r-- | maintenance/deleteEqualMessages.php | 25 |
1 files changed, 16 insertions, 9 deletions
diff --git a/maintenance/deleteEqualMessages.php b/maintenance/deleteEqualMessages.php index 81758913..dbe96982 100644 --- a/maintenance/deleteEqualMessages.php +++ b/maintenance/deleteEqualMessages.php @@ -30,15 +30,18 @@ require_once __DIR__ . '/Maintenance.php'; class DeleteEqualMessages extends Maintenance { public function __construct() { parent::__construct(); - $this->mDescription = "Deletes all pages in the MediaWiki namespace that are equal to the default message"; + $this->mDescription = 'Deletes all pages in the MediaWiki namespace that are equal to ' + . 'the default message'; $this->addOption( 'delete', 'Actually delete the pages (default: dry run)' ); $this->addOption( 'delete-talk', 'Don\'t leave orphaned talk pages behind during deletion' ); - $this->addOption( 'lang-code', 'Check for subpages of this language code (default: root page against content language). ' . - 'Use value "*" to run for all mwfile language code subpages (including the base pages that override content language).', false, true ); + $this->addOption( 'lang-code', 'Check for subpages of this language code (default: root ' + . 'page against content language). Use value "*" to run for all mwfile language code ' + . 'subpages (including the base pages that override content language).', false, true ); } /** * @param string|bool $langCode See --lang-code option. + * @param array &$messageInfo */ protected function fetchMessageInfo( $langCode, array &$messageInfo ) { global $wgContLang; @@ -59,7 +62,8 @@ class DeleteEqualMessages extends Maintenance { // Normalise message names for NS_MEDIAWIKI page_title $messageNames = array_map( array( $wgContLang, 'ucfirst' ), $messageNames ); - $statuses = AllmessagesTablePager::getCustomisedStatuses( $messageNames, $langCode, $nonContLang ); + $statuses = AllMessagesTablePager::getCustomisedStatuses( + $messageNames, $langCode, $nonContLang ); // getCustomisedStatuses is stripping the sub page from the page titles, add it back $titleSuffix = $nonContLang ? "/$langCode" : ''; @@ -130,11 +134,14 @@ class DeleteEqualMessages extends Maintenance { if ( $messageInfo['equalPages'] === 0 ) { // No more equal messages left $this->output( "\ndone.\n" ); + return; } - $this->output( "\n{$messageInfo['relevantPages']} pages in the MediaWiki namespace override messages." ); - $this->output( "\n{$messageInfo['equalPages']} pages are equal to the default message (+ {$messageInfo['equalPagesTalks']} talk pages).\n" ); + $this->output( "\n{$messageInfo['relevantPages']} pages in the MediaWiki namespace " + . "override messages." ); + $this->output( "\n{$messageInfo['equalPages']} pages are equal to the default message " + . "(+ {$messageInfo['equalPagesTalks']} talk pages).\n" ); if ( !$doDelete ) { $list = ''; @@ -151,6 +158,7 @@ class DeleteEqualMessages extends Maintenance { $this->output( " (include --delete-talk to also delete the talk pages)" ); } $this->output( "\n" ); + return; } @@ -170,7 +178,6 @@ class DeleteEqualMessages extends Maintenance { foreach ( $messageInfo['results'] as $result ) { wfWaitForSlaves(); $dbw->ping(); - $dbw->begin( __METHOD__ ); $title = Title::makeTitle( NS_MEDIAWIKI, $result['title'] ); $this->output( "\n* [[$title]]" ); $page = WikiPage::factory( $title ); @@ -181,9 +188,9 @@ class DeleteEqualMessages extends Maintenance { $this->output( "\n* [[$title]]" ); $page = WikiPage::factory( $title ); $error = ''; // Passed by ref - $page->doDeleteArticle( 'Orphaned talk page of no longer required message', false, 0, false, $error, $user ); + $page->doDeleteArticle( 'Orphaned talk page of no longer required message', + false, 0, false, $error, $user ); } - $dbw->commit( __METHOD__ ); } $this->output( "\n\ndone!\n" ); } |