Thanks for the patch Kelvin, I've committed it to CVS. I made a minor patch so that the URI itself could denote the script to run. So you could just run http://localhost:8080/foo/index.jelly for example, rather than requiring a template=index.jelly query parameter..
Some thoughts for further improvement could be... * implement a JellyServletContext so that the getResource() method will use the ServletContext.getResource() method and allow access of relative URIs when performing <j:include>'s. * have a parent JellyServletContext to allow access to the initParams of the ServletContext via variable expressions * implement the JSTL mappings of request parameters, session parameters, cookies etc in the expression language. * it'd be nice to have a cache of Jelly Scripts to avoid parsing them each time! :-) James ------- http://radio.weblogs.com/0112098/ ----- Original Message ----- From: "Kelvin Tan" <[EMAIL PROTECTED]> To: "Jakarta Commons Developers List" <[EMAIL PROTECTED]> Sent: Monday, December 09, 2002 3:22 AM Subject: Re: [Jelly] JellyServlet Sorry for the quick repost. I refactored the getTemplate method to return a URL instead. On Mon, 9 Dec 2002 11:02:38 +0800, Kelvin Tan said: >Apologies. Trouble was with error-reporting code. > >I've ripped the error-printing stuff from VelocityServlet, and added > javadocs. > >See attached. > >> >>Could this be included in the Jelly tree after some testing ? >> > >That's why I submitted it. :) > >Kelvin ---------------------------------------------------------------------------- ---- > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> __________________________________________________ Do You Yahoo!? Everything you'll ever need on one web page from News and Sport to Email and Music Charts http://uk.my.yahoo.com -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
