https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
--- Comment #7 from Jonathan Druart
---
+DelAuthority({ authid => $data->{'authid'}, skip_merge => 1 }) unless
$test;
Checking $test must be done on the commit statements
--
You
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Jonathan Druart changed:
What|Removed |Added
Status|Passed QA
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
--- Comment #5 from Marcel de Rooy ---
Combining -test and -commit is kind of confusing but could be considered
outside the scope? Would welcome a confirm flag instead of having to add -test.
--
You are
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Marcel de Rooy changed:
What|Removed |Added
Status|Signed Off
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Marcel de Rooy changed:
What|Removed |Added
Attachment #72775|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
--- Comment #3 from Marcel de Rooy ---
Note: My first impression about your use of $commit was that we should not use
it that way (I would interpret it as a confirm flag like in other cron jobs),
but
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Marcel de Rooy changed:
What|Removed |Added
CC|
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Brendan Gallagher changed:
What|Removed |Added
Attachment #72727|0
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Brendan Gallagher changed:
What|Removed |Added
Status|Needs Signoff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
Mason James changed:
What|Removed |Added
Status|NEW |Needs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20377
--- Comment #1 from Mason James ---
Created attachment 72727
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=72727=edit
Bug 20377: Various speed tweaks to 'remove_unused_authorities.pl' script
11 matches
Mail list logo