AutoFieldWidget no longer uses CurrencyTextBox dijit

NumberTextBox used instead to work around problem with
CurrencyTextBox's support for negative monetary amounts
in Dojo 1.3.  See https://bugs.launchpad.net/evergreen/+bug/702117

Signed-off-by: Galen Charlton <gmc@esilibrary.com>

git-svn-id: svn://svn.open-ils.org/ILS/trunk@19167 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/web/js/dojo/openils/widget/AutoFieldWidget.js b/Open-ILS/web/js/dojo/openils/widget/AutoFieldWidget.js
index a3ba60b..98b702e 100644 (file)
--- a/Open-ILS/web/js/dojo/openils/widget/AutoFieldWidget.js
+++ b/Open-ILS/web/js/dojo/openils/widget/AutoFieldWidget.js
@@ -224,18 +224,13 @@ if(!dojo._hasResource['openils.widget.AutoFieldWidget']) {
                         break;
 
                     case 'money':
-                        dojo.require('dijit.form.CurrencyTextBox');
-                        // plug in fix for Dojo bug http://bugs.dojotoolkit.org/ticket/9438
-                        // to allow entry of negative values; won't be needed after we upgrade
-                        // to Dojo 1.5
-                        dojo.extend(dijit.form.CurrencyTextBox, {
-                            regExpGen: function(constraints){
-                                return this._focused ?
-                                    dojo.number.regexp(dojo.mixin(dojo.mixin(this.editOptions, constraints), {type: 'decimal'})) :
-                                    dojo.currency.regexp(constraints);
-                            }
-                        });
-                        this.widget = new dijit.form.CurrencyTextBox(this.dijitArgs, this.parentNode);
+                        // dojo.require('dijit.form.CurrencyTextBox');
+                        // the CurrencyTextBox dijit is broken in Dojo 1.3; upon upgrading
+                        // to Dojo 1.5 or later, should re-evaluate work-around use of dijit.form.NumberTextBox.
+                        // See https://bugs.launchpad.net/evergreen/+bug/702117
+                        dojo.require('dijit.form.NumberTextBox');
+                        this.dijitArgs = dojo.mixin({constraints:{places:'0,2'}}, this.dijitArgs || {});
+                        this.widget = new dijit.form.NumberTextBox(this.dijitArgs, this.parentNode);
                         break;
 
                     case 'int':