https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32398 Bug ID: 32398 Summary: Patron import may partially delete restrictions Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Tools Assignee: koha-bugs@lists.koha-community.org Reporter: andrewfh@dubcolib.org QA Contact: testopia@bugs.koha-community.org To recreate: 1 - have a patron, put a manual restriction on their account 2 - create a CSV for patron import that matches your patron 3 - add the debarred and debarredcomment columns to your CSV, but don't put any values into them for your test patron 4 - import your file, matching your patron, and overwriting the existing patron with the new data 5 - reload your patron's record 6 - confirm your patron is allowed to check out but that the Restrictions tab still lists the restriction created in step 1 The patron import nulls out borrowers.debarred and borrowers.debarredcomment but does not update the borrower_debarments table. This leaves the patron with a zombie debarment that doesn't get enforced. I'm not entirely sure what the right answer here is. Either we should not allow patron imports to delete debarments at all or we should make sure it does the deletion properly and completely. For comparison, if one uses batch patron modification to clear the debarred and debarredcomment fields, Koha deletes ALL debarments on that account. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.