http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7376
--- Comment #18 from Olli-Antti Kivilahti <[email protected]> --- I think we should make a single function &CheckBranchTransferLimits($item, $biblio, $destinationBranch, $sourceBranch) # either $item or $biblio should be given, preferably both due to # C4::Context->preference("item-level_itypes") Otherwise we get crazy code duplication since the check for BranchTransferLimits is really messy in the calling method. Also this check is performed during circ/branchtransfers.pl and it definetely should be checked when placing reservations/holds as well. Also some Restful API's we have, place holds and they should be UseBranchTransferLimits-capable. Placing holds is beyond the scope of this bug, but now that a patch has been applied I'd prefer to have a central way of checking the UseBranchTransferLimit instead of replicating this code everywhere. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
