| 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.m.hall |
| Version: | 20.05 | ||
| 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: | |||
| 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';