velocity-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jeff Schnitzer" <>
Subject Re: [VelocityViewServlet] [PATCH] (actually, missing patch)
Date Thu, 09 May 2002 21:15:20 GMT
Thanks guys!

The second issue (persisting the velocity context across multiple
include()s) is not really an issue for me anymore.  After thinking about
it, I have come to the conclusion that it's probably not all that great
of an idea - it's probably fine for the velocity context to be the
equivalent of JSP page scope.

Instead, I just defined

#macro(title $title)
	$request.setAttribute("title", $title)

And now I have what I want :-)

Thanks though!

Jeff Schnitzer

--- Original Message ---
Jeff Schnitzer wrote:

> I notice the current CVS copy of VelocityViewServlet is missing the
> patch for fixing include() behavior from this mail:
> ml

> Any chance of getting this in soon?

Yes, just added this. Sorry, that this got lost.

Regarding the second issue you mentioned in your mail, I would happily
accept your offer to submit a patch for the proposed change.


View raw message