Redo addArgs() as addArg() so we can actually do useful things with arguments like...

[lhc/web/wiklou.git] / maintenance / moveBatch.php

diff --git a/maintenance/moveBatch.php b/maintenance/moveBatch.php
index 13dd40a..9f23677 100644 (file)
--- a/maintenance/moveBatch.php
+++ b/maintenance/moveBatch.php
@@ -2,6 +2,21 @@
 /**
  * Maintenance script to move a batch of pages
  *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
+ *
  * @ingroup Maintenance
  * @author Tim Starling
  *
@@ -18,16 +33,16 @@
  * e.g. immobile_namespace for namespaces which can't be moved
  */
 
-require_once( "Maintenance.php" );
+require_once( dirname(__FILE__) . '/Maintenance.php' );
 
 class MoveBatch extends Maintenance {
        public function __construct() {
                parent::__construct();
                $this->mDescription = "Moves a batch of pages";
-               $this->addParam( 'u', "User to perform move", false, true );
-               $this->addParam( 'r', "Reason to move page", false, true );
-               $this->addParam( 'i', "Interval to sleep between moves" );
-               $this->addArgs( array( 'listfile' ) );
+               $this->addOption( 'u', "User to perform move", false, true );
+               $this->addOption( 'r', "Reason to move page", false, true );
+               $this->addOption( 'i', "Interval to sleep between moves" );
+               $this->addArg( 'listfile', 'List of pages to move, newline delimited', false );
        }
        
        public function execute() {
@@ -49,7 +64,7 @@ class MoveBatch extends Maintenance {
 
                # Setup
                if( !$file ) {
-                       $this->error( "Unable to read file, exiting\n", true );
+                       $this->error( "Unable to read file, exiting", true );
                }
                $wgUser = User::newFromName( $user );
                
@@ -62,13 +77,13 @@ class MoveBatch extends Maintenance {
                        }
                        $parts = array_map( 'trim', explode( '|', $line ) );
                        if ( count( $parts ) != 2 ) {
-                               $this->error( "Error on line $linenum, no pipe character\n" );
+                               $this->error( "Error on line $linenum, no pipe character" );
                                continue;
                        }
                        $source = Title::newFromText( $parts[0] );
                        $dest = Title::newFromText( $parts[1] );
                        if ( is_null( $source ) || is_null( $dest ) ) {
-                               $this->error( "Invalid title on line $linenum\n" );
+                               $this->error( "Invalid title on line $linenum" );
                                continue;
                        }