[Koha-bugs] [Bug 16966] Koha:: Patrons - Move GetBorrowersWithIssuesHistoryOlderThan to search_patrons_to_anonymise

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Feb 27 13:46:15 CET 2017


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

--- Comment #57 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
Created attachment 60709
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60709&action=edit
Bug 16966: [QA Follow-up] Check count, not size in template

The method size is not covered by tests at
koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt line 163.
The method size is not covered by tests at
koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt line 183.

Script cleanborrowers passes patrons_to_delete and patrons_to_anonymize to
the template. One is an array ref where we should use 'size', the other is a
Koha::Patrons object where we should use 'count'.
If we don't, we will not anonymize a lot of records ;)

Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list