https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25183
--- Comment #18 from Julian Maurice <[email protected]> --- Comment on attachment 119593 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119593 Bug 25183: Add option to cleanup-database.pl to move expired self registrations to deleted borrowers table Review of attachment 119593: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=25183&attachment=119593) ----------------------------------------------------------------- ::: misc/cronjobs/cleanup_database.pl @@ +27,4 @@ > use constant DEFAULT_SEARCHHISTORY_PURGEDAYS => 30; > use constant DEFAULT_SHARE_INVITATION_EXPIRY_DAYS => 14; > use constant DEFAULT_DEBARMENTS_PURGEDAYS => 30; > +use constant DEFAULT_EXP_SELFREG_TYPE => 'hard'; This is not used anywhere @@ +167,4 @@ > 'restrictions:i' => \$pDebarments, > 'all-restrictions' => \$allDebarments, > 'del-exp-selfreg' => \$pExpSelfReg, > + 'del-exp-selfreg-type:s' => \$pExpSelfRegType, ':s' means the value is optional. In this case the value should be mandatory. Use '=s' instead @@ +229,4 @@ > || $pPseudoTransactionsTo > || $pMessages > || defined $lock_days && $lock_days ne q{} > + || $pExpSelfRegType !~ m/^hard|soft$/ I guess this line is supposed to avoid giving wrong value for the parameter, but it doesn't work. I was able to give --del-exp-selfreg-type foo without the script complaining -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
