Summary: | remove_unused_authorities.pl doesn't accept --test | ||
---|---|---|---|
Product: | Koha | Reporter: | Gaetan Boisson <gaetan.boisson> |
Component: | Command-line Utilities | Assignee: | Gaetan Boisson <gaetan.boisson> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 11417 - remove_unused_authorities.pl doesn't accept --test
Bug 11417: make sure remove_unused_authorities.pl accepts --test Bug 11417: make sure remove_unused_authorities.pl accepts --test |
Description
Gaetan Boisson
2013-12-18 14:02:11 UTC
Created attachment 23632 [details] [review] Bug 11417 - remove_unused_authorities.pl doesn't accept --test See commit message for the test plan. Created attachment 23661 [details] [review] Bug 11417: make sure remove_unused_authorities.pl accepts --test This patches adds support for the --test option, as well as a short message telling the user the script is running in test mode. Test plan : - Launch the script with -h to see the help - Launch the script with --test and --aut with an authtypecode that is used in your instance - Make sure it does the same thing as launching it with -t - Launch the script for real and make sure it still works as expected, deleting unused authorities. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Created attachment 23667 [details] [review] Bug 11417: make sure remove_unused_authorities.pl accepts --test This patches adds support for the --test option, as well as a short message telling the user the script is running in test mode. Test plan : - Launch the script with -h to see the help - Launch the script with --test and --aut with an authtypecode that is used in your instance - Make sure it does the same thing as launching it with -t - Launch the script for real and make sure it still works as expected, deleting unused authorities. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Pushed to master. Thanks, Gaetan! Patch pushed to 3.14.x, will be in 3.14.3. This patch has been pushed to 3.12.x, will be in 3.12.11. Thanks Gaetan! This is an important fix! |