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';
Created attachment 113879 [details] [review] Bug 27065: Fix incorrect handling of return value of CanItemBeRenewed in CanBookBeRenewed
Please have a look at bug 26659. *** This bug has been marked as a duplicate of bug 26659 ***