[regression] FilteringSelect validation issue

[Fausto Lemos]

Hello guys!

i have a filtering select marked as required. When user types something wrong or leave the filtering blanked the validation works.

But try to leave the filtering empty, click at the arrow to show the result list, and then click outside the filtering. The validation marks the filtering as ok!

this is a know error?

tks!

[bill]

It works for me. I tried ​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/_autoComplete.html?testWidget=dijit.form.FilteringSelect, "US State Test 4". I clicked the down arrow icon, and then clicked away, and the field turned yellow.

Can you give steps to reproduce and if it doesn't reproduce on the above link, then attach a test case?

[Fausto Lemos]

Replying to bill:

It works for me. I tried ​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/_autoComplete.html?testWidget=dijit.form.FilteringSelect, "US State Test 4". I clicked the down arrow icon, and then clicked away, and the field turned yellow.

Can you give steps to reproduce and if it doesn't reproduce on the above link, then attach a test case?

Hello Bill!

looking at FilterinSelect? source i discover what is causing this behavior:

_openResultList:function(_4,_5){
                    if(_5.query[this.searchAttr]!=this._lastQuery){
                       return;
                    }
                    this._isvalid=_4.length!=0;
                    this.validate(true);
                    dijit.form.ComboBoxMixin.prototype._openResultList.apply(this,arguments);
                }

the line "this._isvalid=_4.length!=0;" at "_openResultList()" is setting the filtering "_isvalid" is set to "true" just because the filtering has items. This is ok?

tks!

[bill]

As I said above, you need to give steps to reproduce the problem using either the above link, or a test case that you attach.

[Thomas Pfeiffer]

Replying to bill:

As I said above, you need to give steps to reproduce the problem using either the above link, or a test case that you attach.

Steps to reproduce:

open ​http://docs.dojocampus.org/dijit/form/FilteringSelect

Version 1.3 should be selected. Append some invalid text to the FilteringSelect? => FilteringSelect? switches state to invalid.

Click somewhere else so it looses focuses. Open the list of the FilteringSelect? => State switches to valid

Do not select anything (select still contains the invalid text) but immediatly click somewhere else so that it looses the focus => State switches back to invalid

Reproduce using Trunk.

Last step fails and State stays valid (same in 1.4 stable).

[bill]

Thanks, I see that 1.3 is working correctly but trunk is failing.

[Douglas Hays]

Already fixed in trunk [21276] which means that dojocampus is using a really old trunk build. I suspect dojocampus trunk is really 1.4 since it also doesn't know about the claro theme.

[Marcus Reimann]

Replying to doughays:

Just for your info: As we have learned in the last two weeks, docs.dojocampus.org wasn't updated with 1.4, so the trunk available at docs.dojocampus.org should be a V1.3 trunk (at it is still the same old version, right now).