http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6151
Paul Poulain <paul.poul...@biblibre.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |paul.poul...@biblibre.com
--- Comment #19 from Paul Poulain <paul.poul...@biblibre.com> ---
Can't this feature be achieved already with branch transfer limits ?
I thought that, with this feature activated, one can do much precise return
rules than with just a syspref.
Looking at the code, I realize I'm wrong:
C4/Circulation.pm, sub AddReturn says:
if ($hbr ne $branch && C4::Context->preference("IndependantBranches") &&
!(C4::Context->preference("canreservefromotherbranches"))){
$messages->{'Wrongbranch'} = {
Wrongbranch => $branch,
Rightbranch => $hbr,
};
$doreturn = 0;
# bailing out here - in this case, current desired behavior
# is to act as if no return ever happened at all.
# FIXME - even in an indy branches situation, there should
# still be an option for the library to accept the item
# and transfer it to its owning library.
return ( $doreturn, $messages, $issue, $borrower );
But shouldn't we use the branch_transfer_limit anyway (and not this new
syspref)
(just throwing asking the question, not rejecting the patch)
--
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/
