Re: svn commit: r1769055 - in /velocity/tools/trunk: velocity-tools-generic/ velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ velocity-tools-generic/src/test/java/org/apache/vel

2016-11-10 Thread Sergiu Dumitriu
Yep, thanks for clarifying. I was just puzzled because almost all of the others had @ValidScope(Scope.Application). I guess this one can be used both in Request and Application scopes, so it was easier to just exclude the third one than to include these two. On 11/10/2016 08:58 AM, Claude Brisson

Re: svn commit: r1769055 - in /velocity/tools/trunk: velocity-tools-generic/ velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ velocity-tools-generic/src/test/java/org/apache/vel

2016-11-10 Thread Nathan Bubna
On Thu, Nov 10, 2016 at 4:19 AM, Sergiu Dumitriu wrote: > On 11/10/2016 03:01 AM, cbris...@apache.org wrote: > > Author: cbrisson > > Date: Thu Nov 10 08:01:41 2016 > > New Revision: 1769055 > > > > URL: http://svn.apache.org/viewvc?rev=1769055=rev > > Log: > > [tools]

Re: svn commit: r1769055 - in /velocity/tools/trunk: velocity-tools-generic/ velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ velocity-tools-generic/src/test/java/org/apache/vel

2016-11-10 Thread Claude Brisson
When you restart a J2EE container, it will try to serialize the sessions on disk to restore them when reloading. This can only works if everything in the session is serializable, including the session toolbox itself. Since the RenderTool keeps references on a VelocityEngine, it cannot be

Re: svn commit: r1769055 - in /velocity/tools/trunk: velocity-tools-generic/ velocity-tools-generic/src/main/java/org/apache/velocity/tools/generic/ velocity-tools-generic/src/test/java/org/apache/vel

2016-11-10 Thread Sergiu Dumitriu
On 11/10/2016 03:01 AM, cbris...@apache.org wrote: > Author: cbrisson > Date: Thu Nov 10 08:01:41 2016 > New Revision: 1769055 > > URL: http://svn.apache.org/viewvc?rev=1769055=rev > Log: > [tools] a tool should either be Serializable or forbid Session scope > > Modified: >