Bug 17669: [QA Follow-up] Rename --temp-uploads-override parameter
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Thu, 30 Mar 2017 06:51:17 +0000 (08:51 +0200)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 21 Apr 2017 17:55:25 +0000 (13:55 -0400)
As requested by QA on comment25.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
misc/cronjobs/cleanup_database.pl

index f67a934..4e2c243 100755 (executable)
@@ -44,7 +44,7 @@ use Koha::UploadedFiles;
 
 sub usage {
     print STDERR <<USAGE;
-Usage: $0 [-h|--help] [--sessions] [--sessdays DAYS] [-v|--verbose] [--zebraqueue DAYS] [-m|--mail] [--merged] [--import DAYS] [--logs DAYS] [--searchhistory DAYS] [--restrictions DAYS] [--all-restrictions] [--fees DAYS] [--temp-uploads] [--temp-uploads-override DAYS]
+Usage: $0 [-h|--help] [--sessions] [--sessdays DAYS] [-v|--verbose] [--zebraqueue DAYS] [-m|--mail] [--merged] [--import DAYS] [--logs DAYS] [--searchhistory DAYS] [--restrictions DAYS] [--all-restrictions] [--fees DAYS] [--temp-uploads] [--temp-uploads-days DAYS]
 
    -h --help          prints this help message, and exits, ignoring all
                       other options
@@ -82,7 +82,7 @@ Usage: $0 [-h|--help] [--sessions] [--sessdays DAYS] [-v|--verbose] [--zebraqueu
    --del-unv-selfreg  DAYS  Delete unverified self registrations older than DAYS
    --unique-holidays DAYS  Delete all unique holidays older than DAYS
    --temp-uploads     Delete temporary uploads.
-   --temp-uploads-override DAYS Override the corresponding preference value.
+   --temp-uploads-days DAYS Override the corresponding preference value.
 USAGE
     exit $_[0];
 }
@@ -106,7 +106,7 @@ my $pUnvSelfReg;
 my $fees_days;
 my $special_holidays_days;
 my $temp_uploads;
-my $override_temp_uploads;
+my $temp_uploads_days;
 
 GetOptions(
     'h|help'            => \$help,
@@ -128,7 +128,7 @@ GetOptions(
     'del-unv-selfreg'   => \$pUnvSelfReg,
     'unique-holidays:i' => \$special_holidays_days,
     'temp-uploads'      => \$temp_uploads,
-    'temp-uploads-override:i' => \$override_temp_uploads,
+    'temp-uploads-days:i' => \$temp_uploads_days,
 ) || usage(1);
 
 # Use default values
@@ -315,7 +315,7 @@ if( $temp_uploads ) {
     # If the pref is empty, nothing happens (unless you override).
     print "Purging temporary uploads.\n" if $verbose;
     Koha::UploadedFiles->delete_temporary({
-        override_pref => $override_temp_uploads,
+        override_pref => $temp_uploads_days,
     });
     print "Done purging temporary uploads.\n" if $verbose;
 }