2 Mar
2011
2 Mar
'11
6:52 p.m.
On Wed, Mar 2, 2011 at 18:44, Vincent Massol <vincent(a)massol.net> wrote:
I forgot to menton the alternative which is to create
a WebAppVelocityFactory component impl (hint = "webapp"), in the same manner
that I have already created WebAppVelocityConfiguration and WebAppVelocityEngine.
I think my preference foes to the API modification now.
WebAppVelocityFactory sounds nicer but I don't know very well this
subject so I think I will trust you on that.
Here is a +0.5 ;)
Thanks
-Vincent
On Mar 2, 2011, at 4:49 PM, Vincent Massol wrote:
--
Thomas Mortagne
ok I have no added the following 2 new component
impls:
* WebAppVelocityConfiguration
* WebAppVelocityEngine
However it's not enough since DefaultVelocityEngine.createVelocityEngine() does a
component lookup on VelocityEngine and we need a way for it to look up with a passed
hint.
I'm proposing to modify the current VelocityFactory API from:
VelocityEngine createVelocityEngine(String key, Properties properties) throws
XWikiVelocityException;
to:
VelocityEngine createVelocityEngine(String key, Properties properties, String
engineType) throws XWikiVelocityException;
where engineType is the hint corresponding to the engine to look up.
Then DefaultVelocityManager can call VelocityFactory, passing the "webapp" hint
to createVelocityEngine.
WDYT?
Thanks
-Vincent
On Mar 2, 2011, at 1:56 PM, Vincent Massol wrote:
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
Just a heads up of what I'm working on right
now: cleaning up xwiki-velocity dependencies.
I'm going to split it into several submodules:
* xwiki-velocity-management (or xwiki-velocity-jmx)
* xwiki-velocity-default
* xwiki-velocity-webapp (implementation of VelocityEngine and VelocityConfiguration with
hint = "webapp", this is to use use velocity with the WebappResourceLoader
loading templates from the webapp's root dir)
-Vincent