[Koha-bugs] [Bug 32398] New: Patron import may partially delete restrictions

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

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 at lists.koha-community.org
          Reporter: andrewfh at dubcolib.org
        QA Contact: testopia at 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.