Craig R. McClanahan wrote:
Given a little time to work on Struts (finally), I'd like to tackle the
top two bugs (based on severity) in our remaining list:
* 14669 -- reset() in DynaActionForm acts the same as in ActionForm
* 14800 -- Fix initialization bug and size parameter to form-property
What I propose to do for 14669 is based on the patch proposed by Peter
Pilgrim, but with the "loaded" property renamed to "cleared" -- for
consistency with the method name clear() -- and the "cleared" property
stuff being protected instead of public. For general users of action
forms, I don't see why the cleared state would be relevant, although it
might to advanced DynaActionForm subclasses. The clear() methods will, as
proposed, be public so that application logic can call them as needed.
Well, Craig, I made it public because may be you want to "clear" the
attributes back to the default initial values in a action controller.
I think I am repeated what you just wrote.
When I added the "loaded" / "cleared" attribute I was not sure whether
to make it `public' or `protected'. I called the attributed "loaded"
because the DynaActionForm loads the states XML config files. I almost
called it "configured". Also I assume logically and semantically
that ` DAF.isLoaded() != DAF.isCleared() '
--////--
How many bugs are out standing now?
--
Peter Pilgrim
ServerSide Java Specialist
My on-line resume and for interview videos about myself, J2EE
Open Source, Struts and Expresso.
||
\\===> `` http://www.xenonsoft.demon.co.uk/no-it-striker.html ''
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
