From bb891d8646d4a825cad995dcc97feff9715c2ee4 Mon Sep 17 00:00:00 2001
From: Frank Morgner <morgnerf@ub.uni-leipzig.de>
Date: Wed, 23 Jan 2019 15:10:31 +0100
Subject: [PATCH] refs #14448 [patch] - fixes null type error * for
 $holding['locationid'] at finc/templates/RecordTab/holdingils.phtml

---
 themes/finc/templates/RecordTab/holdingsils.phtml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/themes/finc/templates/RecordTab/holdingsils.phtml b/themes/finc/templates/RecordTab/holdingsils.phtml
index 42faaa8d261..9b230296b34 100644
--- a/themes/finc/templates/RecordTab/holdingsils.phtml
+++ b/themes/finc/templates/RecordTab/holdingsils.phtml
@@ -98,7 +98,9 @@ if (!empty($holdingTitleHold)): ?>
 
   <? /* finc: this next section produces an empty table in some cases - the table borders collapse, producing a thick line - should be solved more elegantly - Fixme - CK*/ ?>
   <? /* nxt line = finc-specific - branch info - CK */ ?>
-  <?=$this->branchInfo($this->driver)->getBranchInfo($holding['locationid']);?>
+  <? if (isset($holding['locationid'])): ?>
+    <?=$this->branchInfo($this->driver)->getBranchInfo($holding['locationid']);?>
+  <? endif; ?>
   <table class="table table-striped">
     <? /* Table summary not supported in html 5, finc-specific solution, CK */ ?>
     <caption class="sr-only"><?=$this->transEsc('holdings_details_from', ['%%location%%' => $this->transEsc($holding['location'])]) ?></caption>
-- 
GitLab