http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9252

--- Comment #17 from Chris Cormack <[email protected]> ---
Comment on attachment 19084
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19084
Bug 9252 - Add option to send patron's home branch in AF field

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

::: C4/SIP/Sip/MsgType.pm
@@ +453,5 @@
>       }
>  
> +    $resp .= maybe_add( FID_SCREEN_MSG, $patron->screen_msg );
> +    $resp .= maybe_add( FID_SCREEN_MSG, $patron->{branchcode} )
> +      if ( $server->{account}->{send_patron_home_library_in_af} );

I see at line 492
$resp = build_patron_status($patron, $lang, $fields, $server ); 

but
line 742 has 
 $resp = build_patron_status($patron, $patron->language, $fields);

In the handle_block_patron subroutine.

This means that $server will be undefined and throw a warn.

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

Reply via email to