Bug 37181

Summary: Add --confirm option switch to pseudonymize_statistics.pl
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: Command-line UtilitiesAssignee: Nick Clemens (kidclamp) <nick>
Status: Needs documenting --- QA Contact: Tomás Cohen Arazi (tcohen) <tomascohen>
Severity: enhancement    
Priority: P5 - low CC: andrew, lucas, robin, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
This adds a `--confirm` option to the `pseudoymize_statistics.pl` command line script. Without this option there will be no changes made when the script is run, avoiding any accidental changes.
Version(s) released in:
24.11.00
Circulation function:
Bug Depends on: 34611    
Bug Blocks:    
Attachments: Bug 37181: Add --confirm option to pseudonymize_statistics.pl

Description Tomás Cohen Arazi (tcohen) 2024-06-25 14:10:38 UTC

    
Comment 1 Tomás Cohen Arazi (tcohen) 2024-06-25 14:14:15 UTC
Created attachment 168070 [details] [review]
Bug 37181: Add --confirm option to pseudonymize_statistics.pl

To test:
1 - Perform some transactions
2 - Enable Pseudonymization
3 - perl misc/maintenance/pseudonymize_statistics.pl -v
4 - Confirm test run an nothing changed
5 - perl misc/maintenance/pseudonymize_statistics.pl -v -c
6 - Confirm statistics are pseudonymized

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Amend: tidied the changes [tcohen]
Comment 2 Tomás Cohen Arazi (tcohen) 2024-06-25 14:21:49 UTC
The RM asked me to QA this patch on the original bug report. I moved it here for clarity.

Straight QA.
Comment 3 Katrin Fischer 2024-06-27 12:06:38 UTC
Pushed for 24.11!

Well done everyone, thank you!
Comment 4 Lucas Gass (lukeg) 2024-07-19 15:27:50 UTC
Enhancement will not be backported to 24.05.x