From f77cb1e73dc6b58bf6b30c4e5833b51e259d9a75 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 21 Dec 2017 10:49:41 -0300 Subject: [PATCH] [SIGNED-OFF] Bug 19315: [16.11.x] Prevent crash if no items and not library set for the subscription Signed-off-by: Jonathan Druart Signed-off-by: Katrin Fischer --- serials/routing-preview.pl | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/serials/routing-preview.pl b/serials/routing-preview.pl index 2ad7cdb..fec8ff5 100755 --- a/serials/routing-preview.pl +++ b/serials/routing-preview.pl @@ -60,13 +60,14 @@ my $subs = GetSubscription($subscriptionid); my ($tmp ,@serials) = GetSerials($subscriptionid); my ($template, $loggedinuser, $cookie); +my $library; if($ok){ # get biblio information.... my $biblionumber = $subs->{'bibnum'}; my ($count2,@bibitems) = GetBiblioItemByBiblioNumber($biblionumber); my @itemresults = GetItemsInfo( $biblionumber ); my $branch = @itemresults ? $itemresults[0]->{'holdingbranch'} : $subs->{branchcode}; - my $branchname = Koha::Libraries->find($branch)->branchname; + $library = Koha::Libraries->find($branch); if (C4::Context->preference('RoutingListAddReserves')){ # get existing reserves ..... @@ -106,7 +107,6 @@ if($ok){ flagsrequired => {serials => '*'}, debug => 1, }); - $template->param("libraryname"=>$branchname); } else { ($template, $loggedinuser, $cookie) = get_template_and_user({template_name => "serials/routing-preview.tt", @@ -118,6 +118,8 @@ if($ok){ }); } +$template->param( libraryname => $library->branchname ) if $library; + my $memberloop = []; for my $routing (@routinglist) { my $member = GetMember( borrowernumber => $routing->{borrowernumber} ); -- 2.1.4