https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23695

--- Comment #2 from Martin Renvoize <[email protected]> ---
Comment on attachment 93231
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=93231
Bug 23695: Set holding branch to transferring branch when manually initiating a
transfer

Review of attachment 93231:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=23695&attachment=93231)
-----------------------------------------------------------------

::: circ/branchtransfers.pl
@@ +127,5 @@
>  if ($barcode) {
>  
>      ( $transferred, $messages ) =
> +        transferbook({
> +            from_branch => C4::Context->userenv->{'branch'},

Hmm, not sure this is right.. you might be transferring from the branch you're
logged in at.. but you may also be transferring from elsewhere to you logged in
branch... I'd say we probably need to add a field to the form and default to
the items current holdingbranch if the user doesn't explicitly state the items
in their branch?

-- 
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/

Reply via email to