search

Kevincosme / flexlib

Automatically exported from code.google.com/p/flexlib
0 stars 0 forks source link

AdvancedForm does not honor initial valid (or not) state of all validators #25

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. create an AdvancedForm with two validators, each against it's own field
2. set the validators up such that the first is valid on the inital form
load, and the 2nd isn't - for instance the first field must have content,
but not the 2nd, and populate both fields with a default
3. change the field for the initialy invalid validator
4. AdvancedForm.isValid remains false

What is the expected output? What do you see instead?
AdvancedForm.isValid should become true.

What version of the product are you using? On what operating system?
v1.7 on Flex 2.0.1 w/hotfix 1

Please provide any additional information below.
Patch against v1.7 attached. The patch calls each validator as it is found
to discover it's initial state rather than assuming false. It does not
propogate this inital check up to the GUI, so the red 'error' arrows do not
appear initial, this matches the current behaviour and is I think desirable.

Original issue reported on code.google.com by tom.chiv...@gmail.com on 15 May 2007 at 10:18

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by dmcc...@gmail.com on 30 Jun 2007 at 11:14

GoogleCodeExporter commented 8 years ago 
This is still not fixed in 2.1, but the fix is the same:
replace 
  obj.isValid = false;
in setValidatorListenerEvents()
with
   //get initial state of vaidator
   var res:ValidationResultEvent=obj.reference.validate(null,true);
   obj.isValid =  ( res.type == ValidationResultEvent.VALID );

Original comment by tom.chiv...@gmail.com on 6 Nov 2007 at 11:15

GoogleCodeExporter commented 8 years ago

Original comment by dmcc...@gmail.com on 26 Dec 2007 at 11:47

GoogleCodeExporter commented 8 years ago 
Not fixed in 2.3.2 either.
Is there something up with the patch that I can help with ?

Original comment by tom.chiv...@gmail.com on 11 Feb 2008 at 4:29

GoogleCodeExporter commented 8 years ago 
Issue 142 has been merged into this issue.

Original comment by dmcc...@gmail.com on 8 Jan 2009 at 5:29