From: gmc <gmc@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Wed, 12 Jan 2011 20:51:27 +0000 (+0000)
Subject: AutoFieldWidget no longer uses CurrencyTextBox dijit
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=9eceec4e830c51784bafcc7337f9e18f674349a9;p=evergreen%2Fpines.git

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 a3ba60b058..98b702eae7 100644
--- 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':