https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25193
Marcel de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|[email protected] |[email protected] |y.org | Status|Signed Off |Failed QA --- Comment #7 from Marcel de Rooy <[email protected]> --- Aleisha, Have some concern about the interaction of both sysprefs and if the interface now helps us to understand it well.. AllowRenewalIfOtherItemsAvailable / AllowRenewalIfOtherItemsAvailableLocation Dont allow / Allow a patron to renew an item with unfilled holds if other available items at any branch / at the hold's pickup branch can fill that hold. If the pref is set to Allow, we may examine the other pref. But if we say Do not allow we now suggest that it is conditional depending on the second pref. But that is not the case: if ( $resfound && C4::Context->preference('AllowRenewalIfOtherItemsAvailable') ) { I wonder if we confuse people? How could we improve that? Another question: Would holding branch be a better wording? Changing status for feedback. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
