Bug 21081 - There is no check that patron is guarantor when batch deleting
Summary: There is no check that patron is guarantor when batch deleting
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-17 08:17 UTC by Sophie MEYNIEUX
Modified: 2019-10-14 19:57 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sophie MEYNIEUX 2018-07-17 08:17:43 UTC
When deleting one patron (members/deletemem.pl), Koha checks that patron is not a guarantor, else it does not allow deletion.

But in tools > batch patron deletion (tools/borrowers.pl or misc/cronjobs/delete_patrons.pl), this check is not performed, so there is a lot of borrowers with guarantorid that does not exist any more.
Check if patron is a guarantor should be performed in all cases of deletion.
Comment 1 Sophie MEYNIEUX 2018-07-17 14:41:29 UTC
It seems that check is done in GetBorrowersToExpunge function

So I don't understand why there is so many borrowers with unknown guarantorid in our databases