https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21336

--- Comment #22 from M. Tompsett <[email protected]> ---
Comment on attachment 80214
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=80214
Bug 21336: Adjust cleanup_database.pl

Review of attachment 80214:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=21336&attachment=80214)
-----------------------------------------------------------------

::: misc/cronjobs/cleanup_database.pl
@@ +308,5 @@
> +Koha::Patrons->search_unsubscribed->lock({ expire => 1, remove => 1, verbose 
> => $verbose });
> +# Anonymize patron data, depending on PatronAnonymizeDelay
> +Koha::Patrons->search_anonymize_candidates({ locked => 1 })->anonymize({ 
> verbose => $verbose });
> +# Remove patron data, depending on PatronRemovalDelay (will raise an 
> exception if problem encountered
> +eval { Koha::Patrons->search_anonymized->delete({ verbose => $verbose }) };

Eval may mask output, which the user should see.
Though, I understand the use of eval, since the code should keep going.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://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/

Reply via email to