From: Galen Charlton <gmc@equinoxinitiative.org>
Date: Wed, 26 Apr 2017 13:59:07 +0000 (-0400)
Subject: LP#1621178: fix fleshing of copy status for available items
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=5957d7c5848514dc1f07f63997f5b65ef96d5eab;p=working%2FEvergreen.git

LP#1621178: fix fleshing of copy status for available items

Since copy status 0 means 'Available', test for null
explicitly.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
---

diff --git a/Open-ILS/web/js/ui/default/staff/circ/services/holds.js b/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
index 204d2a5319..ab0b07dc09 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/services/holds.js
@@ -466,7 +466,8 @@ function($uibModal , $q , egCore , egConfirmDialog , egAlertDialog) {
             hold.current_copy(hold_data.copy);
             
             // likewise, current_copy's status isn't fleshed in the API
-            if(hold.current_copy().status() && typeof hold.current_copy().status() != 'object')
+            if(hold.current_copy().status() !== null &&
+               typeof hold.current_copy().status() != 'object')
                 egCore.pcrud.retrieve('ccs',hold.current_copy().status()
                     ).then(function(c) { hold.current_copy().status(c) });
         }