From bd1d2d4f2085f864006e3482cfff1274e8a5109e Mon Sep 17 00:00:00 2001
From: Joe Montibello <josephmontibello@gmail.com>
Date: Mon, 2 Nov 2015 16:44:49 -0500
Subject: [PATCH] Update Sierra.php

---
 module/VuFind/src/VuFind/ILS/Driver/Sierra.php | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/module/VuFind/src/VuFind/ILS/Driver/Sierra.php b/module/VuFind/src/VuFind/ILS/Driver/Sierra.php
index 1fc8b27302c..a44f3b6d5fe 100644
--- a/module/VuFind/src/VuFind/ILS/Driver/Sierra.php
+++ b/module/VuFind/src/VuFind/ILS/Driver/Sierra.php
@@ -171,11 +171,15 @@ class Sierra extends AbstractBase implements TranslatorAwareInterface
             $results = pg_query_params(
                 $this->db, $query, [$this->idStrip($id)]
             );
-            $callnumberarray = pg_fetch_array($results, 0, PGSQL_NUM);
-            $callnumber = $callnumberarray[0];
+            if (pg_num_rows($results) > 0) {
+                $callnumberarray = pg_fetch_array($results, 0, PGSQL_NUM);
+                $callnumber = $callnumberarray[0];
+                // stripping subfield codes from call numbers
+                $callnumber = preg_replace('/\|(a|b)/', ' ', $callnumber);
+            } else {
+                $callnumber = '';
+            }
         }
-        // stripping subfield codes from call numbers
-        $callnumber = preg_replace('/\|(a|b)/', ' ', $callnumber);
         return $callnumber;
     }
 
@@ -493,8 +497,9 @@ class Sierra extends AbstractBase implements TranslatorAwareInterface
             pg_prepare($this->db, "prep_query", $query1);
             foreach ($itemIds as $item) {
                 $callnumber = null;
+                $barcode = null;
                 $results1 = pg_execute($this->db, "prep_query", [$item]);
-                $number = "";
+                $number = null;
                 while ($row1 = pg_fetch_row($results1)) {
                     if ($row1[4] == "b") {
                         $barcode = $row1[3];
-- 
GitLab