CurrencyTextBox: currency symbol does not change upon change of the currency property

[Jean-Rubin Leonard]

Hi, As per Bill (wildbill) Keese's answer to my post on dojo-interest : "Sounds like a bug. Looking at the code I see that this.currency is reference from postCreate() rather than a _setCurrencyAttr() method, which is surely the problem (why changing currency after creation isn't working).

So, please file a ticket." Bill's answer says it all. When setting the currency at creation time, the currency symbol is what it's supposed to be but any later change to the currency attribute of the cTB leaves the currency symbol whereas it should change to match the currency. Thanks for correcting. Test case attached.

[Cormac Flynn]

This issue is currently a serious blocking issue for our application - do you know if theres a workaround that could be used until 1.5 arrives? Thanks in advance!

[Douglas Hays]

marked currency as const and added attr('constraints', object) API, and added automated tests

[Douglas Hays]

bill, please review the attached patch

[Douglas Hays]

after initial feedback, renamed postCreate in RangeBoundTextbox? and postMixInProperties in CurrencyTextBox? and NumberTextbox? to _setConstraintsAttr

[Douglas Hays]

(In [21359]) Fixes #10477 !strict. Marked CurrencyTextBox::currency as const and added attr('constraints',object) API to ValidationTextBox? and subclasses, and added automated tests. Moved constraints processing from postMixinProperties and postCreate methods into _setConstraintsAttr methods.

[bill]

Something about [21359] is breaking DateTextBox, in test_DateTextBox.html in the firebug console if you type

dijit.byId("german").attr("value")

it returns undefined, rather than the date it should return.

[Douglas Hays]

(In [21385]) Fixes #10477 !strict. Must call _setConstraintsAttr from postMixInProperties since _set*Attr methods can be dependent on this.constraints

[Jean-Rubin Leonard]

Hi, I updated my trunks and that issue does not seem to be resolved. I am still not getting the currency symbol updated.

[Douglas Hays]

If you run dijit/tests/form/robot/test_validate.html with the latest trunk, there is a test at the very end that changes the currency symbol using

textWidget.attr('constraints', {currency:'USD', locale:'en-us'});

This test correctly runs and changes the test's currency symbol from Euro to "$". If you cannot get this to work, then please reopen after attaching a testcase using attr('constraints',blah).

[Jean-Rubin Leonard]

My bad. I wasn't using attr('constraints',blah) was going with straight attr.('blah')