Fix use of argument perfLogFilename in convertLinks.php

Maintenance::getArg needs a number to get the argument, not a string
Reorg the hasOption calls to match the addArg from constructor

Change-Id: Ice7013affb328363daf6e831010de4640583f486

diff --git a/maintenance/convertLinks.php b/maintenance/convertLinks.php
index 02152f7..2dcabe3 100644 (file)
--- a/maintenance/convertLinks.php
+++ b/maintenance/convertLinks.php
@@ -100,10 +100,10 @@ class ConvertLinks extends Maintenance {
                # not used yet; highest row number from links table to process
                # $finalRowOffset = 0;
 
+               $this->logPerformance = $this->hasOption( 'logperformance' );
+               $perfLogFilename = $this->getArg( 1, "convLinksPerf.txt" );
                $overwriteLinksTable = !$this->hasOption( 'keep-links-table' );
                $noKeys = $this->hasOption( 'noKeys' );
-               $this->logPerformance = $this->hasOption( 'logperformance' );
-               $perfLogFilename = $this->getArg( 'perfLogFilename', "convLinksPerf.txt" );
 
                # --------------------------------------------------------------------