Opened 7 years ago

Closed 4 years ago

#16731 closed defect (patchwelcome)

Options and value handling in CheckedMultiSelect

Reported by: Stefan Meyer Owned by: dante
Priority: undecided Milestone: 1.13
Component: DojoX Form Version: 1.8.3
Keywords: Cc:
Blocked By: Blocking:

Description

I noticed that the options and the value array passed to dojox.form.CheckedMultiSelect? are modified. The value array is changed into an array of selected options. The options array is turned into the stateful representation of the current selection by adding the selected property. This is a heavy side effect, that is not apparent from the api. In my case I created a series of multiSelects all with the same options instance. This resulted in all having the same value. I am using dojox.mvc to bind the value to a StatefulArray? and I have to clone the value constantly because it is being modified.

I thing that cloning of value and options inside of the CheckedMultiSelect? would be very good.

Change History (1)

comment:1 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: newclosed

Given that no one has shown interest in creating a patch in the past 2+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.