Regression in [10478]: test_FilteringSelect generates error

[haysmark]

Load this in Firefox:

​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/test_FilteringSelect.html

I get this in Firebug:

dojo.data.ItemFileReadStore is not a constructor
[Break on this error] myStore2 = new dojo.data.ItemFileReadStore({url: 'comboBoxDataToo.json'});

Which is strange because the line right before it also declares an ItemFileReadStore? and it works fine. This starts happening in [10478]. This did not happen in [10477].

[James Burke]

This is caused because this test page has debugAtAllCosts set to true in the page, however it tries to do these two lines outside of a dojo.addOnLoad() block:

myStore = new dojo.data.ItemFileReadStore({url:"comboBoxData.json"});
myStore2 = new dojo.data.ItemFileReadStore({url: 'comboBoxDataToo.json'});

I can see where they cannot be done in an addOnLoad callback, since that callback will not be called after widget parsing, and there are some widgets declared in markup that use those stores. So my fix is going to be to remove debugAtAllCosts for now. If you want to use debugAtAllCosts in the page, turn off parseOnLoad, put the myStore lines in a dojo.addOnLoad() call, then after that, parse the page for widgets.

[bill]

Thanks. Actually those data stores should be declared in markup like

	<div dojoType="dojo.data.ItemFileReadStore" jsId="myStore"
		url="comboBoxData"></div>
	<div dojoType="dojo.data.ItemFileReadStore" jsId="myStoreToo"
		url="comboBoxData"></div>

... just because that's the standard way we are doing stuff in dijit nowadays. I'll update that.

[bill]

(In [10528]) Declare data stores in markup. Refs #4463.