https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18450
Marc Véron <ve...@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #62339|0 |1 is obsolete| | --- Comment #2 from Marc Véron <ve...@veron.ch> --- Created attachment 62447 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62447&action=edit Bug 18450 - Implemented a check to determine if the 'Override and renew' button is displayed based on the setting of the AllowRenewalLimitOverride and AllowRenewalIfOtherItemsAvaliable preferences. This will display the 'Override and renew' button only if both preferences are set to 'Allow'. Test plan: 1. In More->Administration->Global system preferences ensure that both the AllowRenewalLimitOverride and AllowRenewalIfOtherItemsAvaliable preferences are both set to 'Dont allow'. 2. Place a hold on an item from a patron 3. Checkout the item out to a different patron 4. Renew the item and notice the 'Cannot renew: The item is on hold for another patron' 5. Click the 'Override and renew' button and notice that although you can bypass the hold 6. Apply patch 7. Renew the item again and notice that the message does not have a 'Override and renew' button it just has the 'Ignore and continue' button 8. Go back to More->Administration->Global system preferences and set AllowRenewalLimitOverride to 'Allow' 9. Try renewing the same item again and notice that the 'Override and renew' button is still not being displayed. 10. In systems preferences now set the AllowRenewalIfOtherItemsAvailable to 'Allow' 11. Try renewing the item again and notice that now both of these preferences are allowed the 'Override and renew' button is displayed because you will not be violating these preferences. Note: Checking if the renewal exceeds the renewal level, if auto renew is set up, or if renewal is to soon is checked earlier in this conditional if statement earlier and so the check in this patch does not need to check the renewal level, auto renewal, if renewal is too soon. Folloed test plan, works as expected. Signed-off-by: Marc Véron <ve...@veron.ch> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/