#10594 closed defect (fixed)
[patch] [cla] dijit.form.Select::attr('value', 'something', FALSE) fires onChange even when the priorityChange==FALSE
Reported by: | s van gemmert | Owned by: | ben hockey |
---|---|---|---|
Priority: | high | Milestone: | 1.6 |
Component: | Dijit - Form | Version: | 1.4.0 |
Keywords: | Select onChange | Cc: | |
Blocked By: | Blocking: |
Description
dojo: version 1.4.0
when calling
dijit.form.Select::attr('value', 'something', FALSE)
with the "priorityChange" param explicitly set to FALSE, the onChange event should not be triggered This example shows that the Select widget does trigger the onChange in such a case. This seems to be caused by the _handleOnChange call in dijit.form.Select::_updateSelection. By setting the priorityChange to False there solves the problem. But it seems like the whole call to _handleOnChange can be removed there...
Attachments (2)
Change History (10)
Changed 11 years ago by
Attachment: | SelectBug.html added |
---|
comment:1 Changed 11 years ago by
Owner: | set to Nathan Toone |
---|
comment:3 Changed 11 years ago by
Summary: | dijit.form.Select::attr('value', 'something', FALSE) fires onChange even when the priorityChange==FALSE → [patch] [cla] dijit.form.Select::attr('value', 'something', FALSE) fires onChange even when the priorityChange==FALSE |
---|
i've just noticed this same problem in 1.5 beta and also had figured out the same fix - remove the call to _handleOnChange
in _updateSelection
. i've attached a patch for it and all the unit tests still pass with the change but i'm not familiar enough with this widget to be sure it won't break anything.
if someone else will take a look at it, they can either apply the patch or let me know to do it. hopefully if it doesn't make 1.5.0 it could make 1.5.1
Changed 11 years ago by
Attachment: | 10594.diff added |
---|
comment:4 Changed 11 years ago by
Owner: | set to ben hockey |
---|
comment:5 Changed 11 years ago by
Milestone: | tbd → 1.5.1 |
---|
comment:6 Changed 11 years ago by
Milestone: | 1.5.1 → 1.6 |
---|
I talked to neonstalwart about this on IRC. The patch looks good, although I don't think it's critical enough to go into a point release. I'll mark it for 1.6 for now although I don't mind it going into 1.5 if it's put in before the RC which will theoretically happen on Monday.
comment:7 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:8 Changed 10 years ago by
Component: | Dijit → Dijit - Form |
---|
code to reproduce