From f77cb1e73dc6b58bf6b30c4e5833b51e259d9a75 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
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 <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
---
 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