[open-ils-commits] r19590 - trunk/Open-ILS/web/js/dojo/openils/widget (dbs)

[svn at svn.open-ils.org]

Author: dbs
Date: 2011-03-07 10:07:39 -0500 (Mon, 07 Mar 2011)
New Revision: 19590

Modified:
   trunk/Open-ILS/web/js/dojo/openils/widget/AutoGrid.js
Log:
Fix cloning of AutoField widgets: LP #728055

Cloning Action/Trigger event definitions was not working properly, as
all values were being copied from the original object using their display
strings rather than the underlying values.

Bill Erickson suggested a fix in IRC that works, so full credit to him
for this change!


Modified: trunk/Open-ILS/web/js/dojo/openils/widget/AutoGrid.js
===================================================================
--- trunk/Open-ILS/web/js/dojo/openils/widget/AutoGrid.js	2011-03-05 03:18:56 UTC (rev 19589)
+++ trunk/Open-ILS/web/js/dojo/openils/widget/AutoGrid.js	2011-03-07 15:07:39 UTC (rev 19590)
@@ -469,8 +469,8 @@
                         if(field.widget.widget.attr('disabled')) return;
                         var w = clonePane.fieldList.filter(
                             function(i) { return (i.name == field.name) })[0];
-                        w.widget.baseWidgetValue(field.widget.widgetValue); // sync widgets
-                        w.widget.onload = function(){w.widget.baseWidgetValue(field.widget.widgetValue)}; // async widgets
+                        w.widget.baseWidgetValue(field.widget.widget.attr('value')); // sync widgets
+                        w.widget.onload = function(){w.widget.baseWidgetValue(field.widget.widget.attr('value'))}; // async widgets
                     }
                 );
                 origPane.destroy();