From 5957d7c5848514dc1f07f63997f5b65ef96d5eab Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Wed, 26 Apr 2017 09:59:07 -0400 Subject: [PATCH] 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 --- Open-ILS/web/js/ui/default/staff/circ/services/holds.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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) }); } -- 2.11.0