Backport r12586 from trunk to prevent 'Replace Barcode' from trying to set a barcode...
authordbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Thu, 19 Mar 2009 01:49:26 +0000 (01:49 +0000)
committerdbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Thu, 19 Mar 2009 01:49:26 +0000 (01:49 +0000)
git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_4@12590 dcc99617-32d9-48b4-a31d-7c20da2025e4

Open-ILS/xul/staff_client/chrome/content/main/menu.js
Open-ILS/xul/staff_client/server/cat/util.js

index 20d7283..0bdfe33 100644 (file)
@@ -130,7 +130,8 @@ main.menu.prototype = {
        
                                                var new_bc = window.prompt(offlineStrings.getString('menu.cmd_replace_barcode.replacement.prompt'),'',offlineStrings.getString('menu.cmd_replace_barcode.replacement.label'));
                                                new_bc = String( new_bc ).replace(/\s/g,'');
-                                               if (!new_bc) {
+                                               /* Casting a possibly null input value to a String turns it into "null" */
+                                               if (!new_bc || new_bc == 'null') {
                                                        alert(offlineStrings.getString('menu.cmd_replace_barcode.blank.error'));
                                                        return;
                                                }
index f7bd376..d730e32 100644 (file)
@@ -50,7 +50,8 @@ cat.util.replace_barcode = function(old_bc) {
                        '',
                        $("catStrings").getString('staff.cat.util.replace_barcode.new_bc_window_prompt.title'));
                new_bc = String( new_bc ).replace(/\s/g,'');
-               if (!new_bc) {
+               /* Casting a possibly null input value to a String turns it into "null" */
+               if (!new_bc || new_bc == 'null') {
                        alert($("catStrings").getString('staff.cat.util.replace_barcode.new_bc.failed'));
                        return old_bc;
                }