Opened 12 years ago

Closed 12 years ago

Last modified 8 years ago

#4075 closed defect (duplicate)

multiple ComboBox's will not render options (when this.query has more than one property)

Reported by: guest Owned by:
Priority: high Milestone: 0.9
Component: Dijit - Form Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

(i'm jamesholder on IRC if you have questions)

I've got a page with 4 different ComboBox?'s (well FilterSelect? actually).

Combo#1 can point to 1 of 4 different data stores.

Boxes 2,3, and 4 all point to set datastores.

The behavior I'm seeing is that (on a fresh page), i can click on any of the boxes and successfully render the list. However, upon trying to use another combobox, the others become unusable. Async requests are still amde properly, and the data is correctly returned, but the list is never brought up. Trying to go back and use the one that initially worked, that box also no longer works.

I did a little digging and it turns out that on the first render attempt the this.query object shows only one propery. (in my case, display="*")

Upon using the second box, this.query now shows two properties (display="*" and category="*"). and so on and so forth, so that by the end, this.query has four properties.

On line 426 of ComboBox?.js (from the svn, i'm not sure of the revision #) I see

if(this.query==dijit.form.ComboBoxMixin?.prototype.query){query={};}

I replaced that line with:

this.query={};

And the boxes now behave like I expect them too. The properties of this.query are now being overwritten (i think thats what's happening) each time I try and get the results back for the specified combobox, so there is no collision.

There is a comment on line 425 that reads:

instantiate query so comboboxes with different data stores and default query work together

Am I instantiating, or calling something incorrectly here? Or is fetch() not parsing out the correct attribute to search on?

setting this.query to a fresh object each time works, but i'm not sure what consequences that may have.

Thanks!

--james

Change History (2)

comment:1 Changed 12 years ago by bill

Resolution: duplicate
Status: newclosed

Dup of #3962 and already fixed, apparently.

comment:2 Changed 8 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.