Cleanup r62798: If you add a new param then document it, simplify check for --force

diff --git a/maintenance/deleteArchivedFiles.php b/maintenance/deleteArchivedFiles.php
index 3d8253f..f5d1f27 100644 (file)
--- a/maintenance/deleteArchivedFiles.php
+++ b/maintenance/deleteArchivedFiles.php
@@ -30,6 +30,7 @@ class DeleteArchivedFiles extends Maintenance {
                parent::__construct();
                $this->mDescription = "Deletes all archived images.";
                $this->addOption( 'delete', 'Perform the deletion' );
+               $this->addOption( 'force', 'Force deletion of rows from filearchive' );
        }
 
        public function execute() {
@@ -37,10 +38,7 @@ class DeleteArchivedFiles extends Maintenance {
                        $this->output( "Use --delete to actually confirm this script\n" );
                        return;
                }
-               $force = false;
-               if( $this->hasOption('force') ) {
-                       $force = true;
-               }
+               $force = $this->hasOption( 'force ');
                # Data should come off the master, wrapped in a transaction
                $dbw = wfGetDB( DB_MASTER );
                $dbw->begin();