Yarg, I botched this when fixing a Bugzilla bug against the old
"assignable from" test that didn't really allow subclassing.

I just checked in this fix.  Martin, you'll really want to update the
beta1 tag to include it -- otherwise DynaActionForms (including the login
page of the struts example) won't work.

This is also the cause of the humorous self-contradictory error message
that Matt pointed out.

Craig

On Wed, 13 Mar 2002, Porter, Roy wrote:

> Date: Wed, 13 Mar 2002 15:12:51 -0000
> From: "Porter, Roy" <[EMAIL PROTECTED]>
> Reply-To: Struts Developers List <[EMAIL PROTECTED]>
> To: 'Struts Developers List' <[EMAIL PROTECTED]>
> Subject: [Patch] DynaActionFormClass.
>
> >From introspect method in DynaActionFormClass
>
>          if (DynaActionForm.class.isAssignableFrom(beanClass)) {
>
> this should be
>
>          if (!DynaActionForm.class.isAssignableFrom(beanClass)) {
>
> I raised this a couple of weeks ago (the whole line was the other way round
> then). I was ignored ....
>
> It was raised again by somebody else last week, but there's still a problem
> with the latest version on CVS, i.e. the ! is missing.
>
> Can somebody please put this right ?
>
>
> --
> To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to