>
> This kind of expression is there primarily to offer
> compatibility with the previous versions of the taglib while
> introducing a single attribute to set object, context and
> property: using LE will break this compatibility...
A step may be to leave the name attribute as a struts-style
reference (deprec., for compatibility), but then add a "value"
attribute that would be evaluated as el (as it would be in a
jstl tag). I would not allow the user to limit the search of the
scope to a particular context; it would be implemented as
pageContext.findAttribute(..).
The implementation could be something as naive as
public void setName(String name){
setValue("${" + name + "}");
}
> This makes easier in the future to switch to a more common
> expression language or to provide different versions of the
> taglib (just one point of change). It's fairly easy to
> implement, but if we want to move to the jstl direction now
> we should state that the new version of the taglib will not
> be compatible with 0.8.x releases.
>
> BTW, the expressions used are common beanutils expressions,
> whit a scope prefix borrowed from jstl
>
> fabrizio
>
> -----Original Message-----
> From: Andy Pruitt [mailto:[EMAIL PROTECTED]
> Sent: Wednesday, September 24, 2003 9:58 PM
> To: [EMAIL PROTECTED]
> Subject: RE: [displaytag-devel] PageContext / el not supported
>
> Absolutely- I've not heard of another taglib that uses this syntax.
>
> AFAIK, JSTL requires a JSP 1.2 container, but I think that
> JSP EL - as
> implemented in commons - has no such dependency.
>
> I'd be willing to supply a patch if there is interest.
>
> -----Original Message-----
> From: John York [mailto:[EMAIL PROTECTED]
> Sent: Wednesday, September 24, 2003 2:02 PM
> To: [EMAIL PROTECTED]
> Subject: RE: [displaytag-devel] PageContext / el not supported
>
>
> This is one of the features that I don't think we should
> release. It would
> be better to pull it out until we can get commons-el in place than to
> force people to use this custom syntax. Or at least, make it
> clear in the
> documentation that this feature will most certainly be
> changed in the next
> release.
>
> Just my $.02
>
> John
>
>
>
> On Wed, 24 Sep 2003, Fabrizio Giustina wrote:
>
> > Object placed in the pageContext ARE found by the the table tag, but
> > you have to explicitely set the scope:
> >
> > <% pageContext.setAttribute( "test", new TestList( 10 ) ); %>
> > <display:table name="pageScope.test">
> >
> > The main reason for not using jstl expression till now is that jstl
> > requires j2ee 1.3, and I want to preserve compatibility
> with 1.2. I'm
> > planning to look at the recent commons-el package from Jakarta to
> > implement an el version of the display taglib.
> >
> > fabrizio
> >
> >
> > -----Original Message-----
> > From: Andy Pruitt [mailto:[EMAIL PROTECTED]
> > Sent: Wednesday, September 24, 2003 6:38 PM
> > To: [EMAIL PROTECTED]
> > Subject: [displaytag-devel] PageContext / el not supported
> >
> >
> > Lists placed in the pageContext are not being found by the
> new display
>
> > tag.
> >
> > Would it be possible to simply use the JSTL el evaluator? It would
> > mean replacing TemplateTag#evaluateExpression with something like:
> >
> > return ExpressionEvaluatorManager.evaluate("name", valueEl,
> > String.class, this, pageContext);
> >
> >
> >
> >
> >
> > -------------------------------------------------------
> > This sf.net email is sponsored by:ThinkGeek
> > Welcome to geek heaven.
> > http://thinkgeek.com/sf
> > _______________________________________________
> > displaytag-devel mailing list [EMAIL PROTECTED]
> > https://lists.sourceforge.net/lists/listinfo/displaytag-devel
> >
> >
> > -------------------------------------------------------
> > This sf.net email is sponsored by:ThinkGeek
> > Welcome to geek heaven.
> > http://thinkgeek.com/sf
> > _______________________________________________
> > displaytag-devel mailing list [EMAIL PROTECTED]
> > https://lists.sourceforge.net/lists/listinfo/displaytag-devel
> >
>
> --
> John York
> Software Engineer
> CareerSite Corporation
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> displaytag-devel mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/displaytag-devel
>
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> displaytag-devel mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/displaytag-devel
>
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> displaytag-devel mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/displaytag-devel
>
>
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
