Opened 9 years ago

Closed 9 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 9 years ago by Kenneth G. Franqueiro

Component: GeneralDijit - Form
Owner: set to Douglas Hays

comment:2 Changed 9 years ago by Douglas Hays

Milestone: tbd1.7
Status: newassigned

comment:3 Changed 9 years ago by Douglas Hays

(In [24254]) Fixes #12622. Add call to this.inherited to postCreate method.

comment:4 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

comment:5 Changed 9 years ago by bill

Resolution: fixed
Status: closedreopened

[24254] breaks the Form.html with an error in the reset test (about the value of the MultiSelect).

comment:6 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: reopenedclosed

(In [24286]) Fixes #12622. Changed MultiSelect? to follow the form widget rules like setting this.value via _set. Added tests to check this.value.

Note: See TracTickets for help on using tickets.