Opened 10 years ago
Closed 10 years ago
#12622 closed defect (fixed)
dijit.form.MultiSelect: postCreate breaks the inheritance chain
| Reported by: | Kenneth G. Franqueiro | Owned by: | Douglas Hays |
|---|---|---|---|
| Priority: | high | Milestone: | 1.7 |
| Component: | Dijit - Form | Version: | 1.6.0 |
| Keywords: | MultiSelect | Cc: | |
| Blocked By: | Blocking: |
Description
dijit.form.MultiSelect contains a postCreate method which does not call this.inherited(arguments). This causes the inheritance chain to be broken, and thus the postCreate methods of its ancestors (of which there are eventually many) do not get called.
I happened to notice this because events that should be hooked up in dijit._CssStateMixin were not being hooked up. Perhaps other people may experience other symptoms.
This bug seems to have existed for a while - possibly even further back than 1.3 - but has never been addressed.
Change History (6)
comment:1 Changed 10 years ago by
| Component: | General → Dijit - Form |
|---|---|
| Owner: | set to Douglas Hays |
comment:2 Changed 10 years ago by
| Milestone: | tbd → 1.7 |
|---|---|
| Status: | new → assigned |
comment:3 Changed 10 years ago by
comment:4 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:5 Changed 10 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
[24254] breaks the Form.html with an error in the reset test (about the value of the MultiSelect).
comment:6 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
(In [24286]) Fixes #12622. Changed MultiSelect? to follow the form widget rules like setting this.value via _set. Added tests to check this.value.
(In [24254]) Fixes #12622. Add call to this.inherited to postCreate method.