Summary: | Hold requests require cardnumbers to be set | ||
---|---|---|---|
Product: | Koha | Reporter: | Robin Sheat <robin> |
Component: | Hold requests | Assignee: | Robin Sheat <robin> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | major | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, gmcharlt, wizzyrea |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 6687 | ||
Attachments: |
Bug 6526 - make the reserves code use the borrowernumber
[SIGNED OFF] Bug 6526 - make the reserves code use the borrowernumber |
Description
Robin Sheat
2011-06-23 09:10:50 UTC
Created attachment 4520 [details] [review] Bug 6526 - make the reserves code use the borrowernumber Previously it used the cardnumber, which caused numerous issues if your users don't have card numbers. robin, should this be tested with both borrowers with card numbers and those without, for completeness? LR Yes, definitely. In theory there should be no difference in the way they work now. In practice, there may be. Testing with different configurations is always good :) Created attachment 4522 [details] [review] [SIGNED OFF] Bug 6526 - make the reserves code use the borrowernumber Previously it used the cardnumber, which caused numerous issues if your users don't have card numbers. Tested with both a patron with a card # and one without, in both OPAC and Intranet. Pushed please test I suspect this patch has caused Bug 6687 (cannot move people in the holds queue). I think this has been sufficiently tested now :) |