Bug 16966: [QA Follow-up] Check count, not size in template
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Mon, 27 Feb 2017 12:21:18 +0000 (13:21 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 3 Mar 2017 17:20:05 +0000 (17:20 +0000)
commitabdf2319cd0fca27046699ae029107e92bb7f02f
tree39b4aeda053809ae082015c9c99b23065524680a
parentb4cd5faeb09b4872b5ea413e0aace3ebbaae26c5
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@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/tools/cleanborrowers.tt