Opened 8 years ago
Closed 8 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 8 years ago by
| Component: | General → Dijit - Form |
|---|---|
| Owner: | set to Douglas Hays |
comment:2 Changed 8 years ago by
| Milestone: | tbd → 1.7 |
|---|---|
| Status: | new → assigned |
comment:3 Changed 8 years ago by
comment:4 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:5 Changed 8 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 8 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.