Summary: | Patron import fails if matching on cardnumber and userid also matches | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Patrons | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Bug 21951: Patron import fails if matching on cardnumber and userid also matches
Bug 21951: Patron import fails if matching on cardnumber and userid also matches |
Description
Kyle M Hall (khall)
2018-12-05 12:36:44 UTC
Created attachment 82860 [details] [review] Bug 21951: Patron import fails if matching on cardnumber and userid also matches If a patron CSV file has both cardnumber and userid, and cardnumber is used as a matchpoint, the patron import will fail with an error like -- Userid kyle.hall is already used by another patron. -- even though that userid is used by the very same patron! Matching on userid does not generate this error. Test Plan: 1) Generate a patron CSV file where the userid and cardnumber columns match an existing patron 2) Attempt to overlay existing patron using this CSV file via the patron import tool 3) Note the "already used" error 4) Apply this patch 5) No error should be generated 6) Modify the cardnumber in the CSV file 7) Attempt to import and overlay again 8) Note you get the userid "already used" error Created attachment 82869 [details] [review] Bug 21951: Patron import fails if matching on cardnumber and userid also matches If a patron CSV file has both cardnumber and userid, and cardnumber is used as a matchpoint, the patron import will fail with an error like -- Userid kyle.hall is already used by another patron. -- even though that userid is used by the very same patron! Matching on userid does not generate this error. Test Plan: 1) Generate a patron CSV file where the userid and cardnumber columns match an existing patron 2) Attempt to overlay existing patron using this CSV file via the patron import tool 3) Note the "already used" error 4) Apply this patch 5) No error should be generated 6) Modify the cardnumber in the CSV file 7) Attempt to import and overlay again 8) Note you get the userid "already used" error Signed-off-by: Owen Leonard <oleonard@myacpl.org> |