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 Kenneth G. Franqueiro

Component: GeneralDijit - Form
Owner: set to Douglas Hays

comment:2 Changed 10 years ago by Douglas Hays

Milestone: tbd1.7
Status: newassigned

comment:3 Changed 10 years ago by Douglas Hays

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

comment:4 Changed 10 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

comment:5 Changed 10 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 10 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.