[Koha-bugs] [Bug 10612] Add ability to delete patrons with batch patron deletion tool

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10612

Jacek Ablewicz <abl at biblos.pk.edu.pl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |abl at biblos.pk.edu.pl

--- Comment #27 from Jacek Ablewicz <abl at biblos.pk.edu.pl> ---
(In reply to Owen Leonard from comment #26)
> Are patron automatically excluded from deletion for some reason? I tried to
> delete based on a patron list that contains 82 patrons and the tool offers
> to delete 2 patrons.

Only patrons with non-zero account balance (with unpaid fines/charges) are
explicitly excluded when deleting from a list.

Batch deleting (from a list) patrons with active check-outs really needs some
thought, IMO (apart from the somehow missleading count mentioned in comment
#25). While such patron records would not get actually deleted (due to the
foreign key constraint in issues table):

- their reserves and lists/virtual shelves (if any) would be deleted,
- if the choosen action is 'Move these patrons to the trash', I think that the
surplus record would be created in the deletedborrowers table, with original
patron record still remaining in borrowers table.

Also, when deleting from a list, it's possible to (accidentally) delete a staff
patron, and a patron which is a guarantor for some other patron[s]. 

On an unrelated subject: 'patron category' filter in batch patron
deletion/anonymization tool seems to be badly broken in the current master
(different issue, encountered while testing this patch - I will file separate
bug report for it).

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