From 132ef2968c4a26bf573449d4d576e1bdb5910bf2 Mon Sep 17 00:00:00 2001 From: Chad Horohoe Date: Sun, 5 Dec 2010 05:19:31 +0000 Subject: [PATCH] No need for globals, works fine without. Also add missing (and required) title arg --- maintenance/protect.php | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/maintenance/protect.php b/maintenance/protect.php index 15780fbb5f..e97fc108b6 100644 --- a/maintenance/protect.php +++ b/maintenance/protect.php @@ -28,10 +28,11 @@ class Protect extends Maintenance { $this->addOption( 'semiprotect', 'Adds semi-protection' ); $this->addOption( 'u', 'Username to protect with', false, true ); $this->addOption( 'r', 'Reason for un/protection', false, true ); + $this->addArg( 'title', 'Title to protect', true ); } public function execute() { - global $wgUser, $wgTitle, $wgArticle; + global $wgUser; $userName = $this->getOption( 'u', 'Maintenance script' ); $reason = $this->getOption( 'r', '' ); @@ -46,16 +47,16 @@ class Protect extends Maintenance { $wgUser = User::newFromName( $userName ); $restrictions = array( 'edit' => $protection, 'move' => $protection ); - $wgTitle = Title::newFromText( $this->getArg() ); - if ( !$wgTitle ) { + $t = Title::newFromText( $this->getArg() ); + if ( !$t ) { $this->error( "Invalid title", true ); } - $wgArticle = new Article( $wgTitle ); + $article = new Article( $t ); # un/protect the article $this->output( "Updating protection status... " ); - $success = $wgArticle->updateRestrictions( $restrictions, $reason ); + $success = $article->updateRestrictions( $restrictions, $reason ); if ( $success ) { $this->output( "done\n" ); } else { -- 2.20.1