Costin,
> I'm a bit worried here - most of the stuff is pretty basic and
> common use of tags. Watchdog and the test suite was supposed
> to detect that withou any problems.
>
Use of scripting varibles in nested tag never work before, so obviously
no body uses it much. I think the whole scripting variable in JSP1.2 is
poorly designed, and not well understood.
I am also surprised that how much watchdog *does not* test!
> I'll vote for 'alpha' status for the next build,
> and I hope more people will check their apps against it -
> I like the changes that are going on, but we need a bit
> more testing.
>
> How do we deal with nested tags of different type ? I don't have
> a test case, but I assume someone may have a NESTED tag
> 'foo' in 2 different tags, with different declared types ( String and
> Integer ). The current generator seems to not create { },
> and I'm not sure how this will work.
>
> ( no test case - but I'm sure you can find such tags )
>
That would not work, and would never be made to work, not only for
nested case, but also for non-nested tags with AT_BEGIN scopes.
I tend to think this falls into the user's reponsibility if she uses
the same variable name for two different types, somewhat analogous to
declaring two variables of the same name two different types in Java.
Hopefully, there won't be a need for scripting variables anymore when
expression language becomes available in JSP2.0.
-Kin-man
> Costin
>
>
> On Fri, 21 Jun 2002, Jan Luehe wrote:
>
> > Costin,
> >
> >
> > > > If those variable declaration problems are fixed, I'll release a
new
> > > > 4.1.6 milestone as soon as I can fix the JNDI problems.
> > >
> > > Not yet...
> > >
> > > I attached the failed jsp.
> > >
> > > Costin
> > >
> > > [...]
> > >
> > > <%@ page language="java" %>
> > > <%@ taglib uri="/WEB-INF/struts-logic.tld" prefix="logic" %>
> > >
> > > <logic:iterate id="a" name="foo" property="bar">
> > > </logic:iterate>
> > >
> > > <logic:present name="b">
> > > <logic:iterate id="a" name="foo" property="bar">
> > > </logic:iterate>
> > > </logic:present>
> >
> >
> > last and final attempt! :)
> > The attached patch should fix the last issue you brought up.
> >
> > Since I am still waiting for my commit privileges, Kin-Man
> > has again volunteered to apply the patch. Thanks Kin-Man!
> >
> >
> > Jan
> >
> >
>
>
> --
> 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]>
