Opened 10 years ago
Closed 10 years ago
#12112 closed defect (fixed)
[patch][cla][regression] dojox.form.manager._Mixin clobbers dijit._Widget "watch" function
Reported by: | ben hockey | Owned by: | Eugene Lazutkin |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | DojoX Form | Version: | 1.5 |
Keywords: | Cc: | Kris Zyp | |
Blocked By: | Blocking: |
Description
in 1.6, dijit._Widget will introduce a watch
function. the naming of this function causes an unfortunate clash with dojox.form.manager._Mixin which has a watch
property.
to avoid this clash, i'm suggesting that dojox.form.manager._Mixin rename it's property. this change would not be backwards compatible but i would think that the name is not likely to be changed in dijit. any other options?
i think this change should be made before the release of 1.6 since the 2 classes are incompatible without this change. unless dijit changes the name of the property then it's inevitable that existing code using dojox.form.manager._Mixin will be broken in one way or another with the release of 1.6
Attachments (1)
Change History (5)
comment:1 Changed 10 years ago by
Owner: | changed from dante to Eugene Lazutkin |
---|
comment:2 Changed 10 years ago by
Cc: | Kris Zyp added |
---|---|
Summary: | dojox.form.manager._Mixin clobbers dijit._Widget "watch" function → [regression] dojox.form.manager._Mixin clobbers dijit._Widget "watch" function |
Changed 10 years ago by
Attachment: | 12112-watching.diff added |
---|
comment:3 Changed 10 years ago by
Summary: | [regression] dojox.form.manager._Mixin clobbers dijit._Widget "watch" function → [patch][cla][regression] dojox.form.manager._Mixin clobbers dijit._Widget "watch" function |
---|
i'm proposing that dojox.form.manager.* changes to use watching
.
i've attached a patch which would change watch
to watching
in dojox.form.manager.*
comment:4 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
proposed patch to change to
watching