Bug 16907: Koha::Patrons - Move DelMember to ->delete
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 11 Jul 2016 20:18:56 +0000 (21:18 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 21 Oct 2016 16:20:41 +0000 (16:20 +0000)
commit26c034d1f50762f0548ce3ee61767d64139cd311
treec48c1c6aac2c73fa58b20652a1d5c21850816234
parent20c44f005138d2f585eeb1336a93920629b52d3c
Bug 16907: Koha::Patrons - Move DelMember to ->delete

This patch moves the C4::Members::DelMember subroutine to the
Koha::Patron module.
The delete method must be overwritten to permit handling of patron's
holds.

Test plan:
(With the 2 patches applied)
1/ Create a patron with holds and owner of lists
2/ Delete patrons using the web interface:
 - More > Delete on a patron page
 - Batch patron deletion tools
3/ and the cronjob script
 - perl misc/cronjobs/delete_patrons.pl -c [more options]

The patron should have been moved to the deletedborrowers table, his/her
holds and lists should have been deleted.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/Members.pm
Koha/Patron.pm
members/deletemem.pl
misc/cronjobs/delete_expired_opac_registrations.pl
misc/cronjobs/delete_patrons.pl
misc/cronjobs/delete_unverified_opac_registrations.pl
t/db_dependent/Auth_with_ldap.t
t/db_dependent/Koha/Patrons.t
t/db_dependent/Members.t
tools/cleanborrowers.pl