| Summary: | cleanborrowers.pl does not check for anonymisation errors | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Peter Kelly <peterAtKohaBugzilla> |
| Component: | Tools | Assignee: | Galen Charlton <gmcharlt> |
| Status: | NEW --- | QA Contact: | |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | jonathan.druart, peterAtKohaBugzilla |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16966 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 |
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| Bug Depends on: | 6506, 10165 | ||
| Bug Blocks: | |||
|
Description
Peter Kelly
2013-03-28 07:32:11 UTC
Still valid:
141 # Anonymising all members
142 if ($do_anonym) {
143 #FIXME: anonymisation errors are not handled
144 ($totalAno,my $anonymisation_error) = AnonymiseIssueHistory($last_issue_date);
145 $template->param(
146 do_anonym => '1',
147 );
148 }
Note that bug 16966 will improve it a bit: the number of records anonymised will be displayed. Code is different now, but the FIXME remained:
# Anonymising all members
146 if ($do_anonym) {
147 #FIXME: anonymisation errors are not handled
148 my $rows = Koha::Old::Checkouts
149 ->filter_by_anonymizable
150 ->filter_by_last_update({
151 to => $last_issue_date, timestamp_column_name => 'returndate' })
152 ->anonymize;
153
154 $template->param(
155 do_anonym => $rows,
156 );
157 }
158
|