[open-ils-commits] r19013 - trunk/Open-ILS/web/js/ui/default/conify/global/acq (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2010-12-17 14:26:05 -0500 (Fri, 17 Dec 2010)
New Revision: 19013

Modified:
   trunk/Open-ILS/web/js/ui/default/conify/global/acq/provider.js
Log:
Acq: Tracked down a small but subtle bug with the Holding Subfield tab of
the Provider editor interface.

We apparently have a global variable "name" defined somewhere which, when
assigned in place of an overrideWidget for an AutoGrid thanks to some vestigial
code, ultimately causes an error from within AutoFieldWidget, if you're using
the staff client.  That's because name is the string "browser_browser" in the
staff client, but if you access this interface from Firefox, name is apparently
set to some value that evaluates to false in a boolean context, because
AutoFieldWidget doesn't choke on it (the comparison at line 182 of that file
will return false).


Modified: trunk/Open-ILS/web/js/ui/default/conify/global/acq/provider.js
===================================================================
--- trunk/Open-ILS/web/js/ui/default/conify/global/acq/provider.js	2010-12-17 18:21:11 UTC (rev 19012)
+++ trunk/Open-ILS/web/js/ui/default/conify/global/acq/provider.js	2010-12-17 19:26:05 UTC (rev 19013)
@@ -81,7 +81,6 @@
         case 'tab-hold': 
             phsListGrid.overrideEditWidgets.provider = new
                 dijit.form.TextBox({disabled: 'true', value: providerId});
-            phsListGrid.overrideEditWidgets.name = name;
             phsListGrid.resetStore();
             phsListGrid.loadAll({order_by:{acqphsm : 'name'}}, {provider : providerId});
             break;