I wasn't aware that the 0.8 version used these expressions? If it did, then I suppose there's no harm in releasing it, although I'd still vote to refactor with commons-el in the near future.
On Wed, 24 Sep 2003, Fabrizio Giustina wrote: > 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... > > 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
