| Summary: | Automatic renewals performed when no reservable items are available but items that can be issued are available. | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Andreas Jonsson <andreas.jonsson> |
| Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
| Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
| Severity: | major | ||
| Priority: | P5 - low | CC: | gmcharlt, jonathan.druart, kyle |
| Version: | 20.05 | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 |
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| Attachments: | Bug 27065: Fix incorrect handling of return value of CanItemBeRenewed in CanBookBeRenewed | ||
Created attachment 113879 [details] [review] Bug 27065: Fix incorrect handling of return value of CanItemBeRenewed in CanBookBeRenewed |
The return value of CanItemBeReserved is incorrectly used in a condition in CanBookBeRenewed: next unless CanItemBeReserved($borrowernumber,$itemnumber); CanItemBeReserved returns a hash reference wich always evaluates to true. The condition should instead be: next unless CanItemBeReserved($borrowernumber,$itemnumber)->{status} eq 'OK';