Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1497 closed defect (fixed)

ComboBox - corresponding value not selected

Reported by: acirov@… Owned by: Douglas Hays
Priority: high Milestone:
Component: Widgets Version: 0.3
Keywords: ComboBox Cc: acirov@…
Blocked By: Blocking:

Description

When an element from the list is selected the corresponding value is always one step behind. It corresponds to the pair that was previously selected.

Attachments (1)

1497.patch (1.0 KB) - added by Douglas Hays 13 years ago.
changed order of set values in Select.js as well

Download all attachments as: .zip

Change History (9)

comment:1 Changed 13 years ago by Douglas Hays

Cc: acirov@… added
Resolution: duplicate
Status: newclosed

Please verify with the patch contained in ticket 1523.

comment:2 Changed 13 years ago by acirov@…

Resolution: duplicate
Status: closedreopened

Issue still persists. Patch 1523 solves the onValueSelect issue but he combo box still presents the comboBoxSelectionValue of the item that was previously selected. It's always one step behind the value shown in the box itself ( comboBoxValue ).

comment:3 Changed 13 years ago by acirov@…

Steps to recreate this defect:

Have the ComboBox? trigger onValueChanged function. In the function put the following:

alert(dojo.widget.byId('some_id').comboBoxValue.value + "_" + dojo.widget.byId('some_id').comboBoxSelectionValue.value);

You'll see that the comboBoxSelectionValue corresponds to the previously selected item, initially blank. Then just select another value from the ComboBox? and observe the corresponding value. It's exactly what was previously selected.

This happens only in the ComboBox?. Select works just fine.

comment:4 Changed 13 years ago by Douglas Hays

Owner: changed from bill to Douglas Hays
Status: reopenednew

Please verify with the attached patch. This patch supersedes #1523.

comment:5 Changed 13 years ago by Douglas Hays

Status: newassigned

Changed 13 years ago by Douglas Hays

Attachment: 1497.patch added

changed order of set values in Select.js as well

comment:6 Changed 13 years ago by acirov@…

Yes, it works.

comment:7 Changed 13 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [5943]) Fixes #1497 Reverses order of setLabel and setValue

comment:8 Changed 12 years ago by (none)

Milestone: 0.4

Milestone 0.4 deleted

Note: See TracTickets for help on using tickets.