From: Jason Etheridge <jason@esilibrary.com>
Date: Fri, 20 Apr 2012 02:37:56 +0000 (-0400)
Subject: fix double-click in Holdings Maintenance
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=e31c4337304ae6ebe819011ad8e1bd396acd0761;p=contrib%2FConifer.git

fix double-click in Holdings Maintenance

so that it won't attempt to spawn the Item Editor for non-items

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---

diff --git a/Open-ILS/xul/staff_client/server/cat/copy_browser.js b/Open-ILS/xul/staff_client/server/cat/copy_browser.js
index 19fc6d7cfc..89313fbf40 100644
--- a/Open-ILS/xul/staff_client/server/cat/copy_browser.js
+++ b/Open-ILS/xul/staff_client/server/cat/copy_browser.js
@@ -428,7 +428,9 @@ cat.copy_browser.prototype = {
                                         }
                                     );
 
-                                    xulG.volume_item_creator( {'existing_copies':list, 'onrefresh' : function() { obj.refresh_list(); } } );
+                                    if (list.length > 0) {
+                                        xulG.volume_item_creator( {'existing_copies':list, 'onrefresh' : function() { obj.refresh_list(); } } );
+                                    }
 
                                 } catch(E) {
                                     obj.error.standard_unexpected_error_alert(document.getElementById('catStrings').getString('staff.cat.copy_browser.edit_items.error'),E);