[qmtest] Nesting qm.fields.SetField

[Mark Mitchell]

On Thu, 2003-10-09 at 06:36, Anders Hasselqvist wrote:
> > -----Original Message-----
> > From: Mark Mitchell [mailto:mark at codesourcery.com] 
> > Sent: den 8 oktober 2003 17:09
> > To: Anders Hasselqvist
> > Subject: RE: [qmtest] Nesting qm.fields.SetField
> > 
> > 
> > > 
> > > > 
> > > > The other possibility is that SetField.ParseFormValue is not 
> > > > working correctly.  You can look through that code; redisplay 
> > > > should get set to 1 when you add a value in the second set field.
> > > > 
> > > 
> > > First of all I'm running QMTest 2.0.3 so I don't know if 
> > this is fixed in
> > > cvs.
> > 
> > Ah!
> > 
> > Yes, I know that QMTest 2.0.3 did not handle recursive uses 
> > of SetField
> > correctly.  I can't remember if the problems were only in the
> > ParseFormValue function, or whether they were elsewhere as well.
> > 
> 
> I tested nesting SetField using the cvs version last night. The same problem
> exist there too. I guess that SetField.ParseFormValue should do like
> TupleField.ParseFormValue, that is: 1. Initialize redisplay = 0 at the start
> of
> 2. remove the else clause in the if action sequence.

Yes!

I've applied the attached patch.

-- 
Mark Mitchell <mark at codesourcery.com>
CodeSourcery, LLC
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diffs
URL: <http://sourcerytools.com/pipermail/qmtest/attachments/20031015/46d159b0/attachment.ksh>