Hi all after a more detailed look I came to the following solution:
LdRenderer (in o.a.s.commons.ldpathtempate) has several responsibilities: (1) loading Freemarker templates from Bundles (basically freemarker TemplateLoader) (2) rendering RdfViewable (3) rendering Viewable Other than that the whole module is just a branch of the ldpath-template module that is compatible to the LDpath version currently used by Stanbol. The suggestion is to: * refactor the (1) to BundleTemplageLoader, an OSGI service that implements TemplateLoader. This service can be provided by the o.a.s.commons.web.viewable module * move (3) to the ViewableWriter: * create a new module o.a.s.commons.web.viewable.ldpath that provides the LdViewableWriter containing (2). Loading of Templates is provided by an injected TemplateLoader instance (the BundleTemplageLoader implementation provided by the commons.web.viewable module) The o.a.s.commons.ldpathtempate can be completely removed as soon as we upgrade to an LDpath version where the ldpath-template module is available in maven central. In addition I would suggest to move the Viewable and RdfViewable classes to the correct packages but keep the old classes (marked as @deprecated) for backward compatibility) I have already implemented ~80% of the suggestion and to me it feels much better as the current soltution WDYT Rupert On Mon, Feb 4, 2013 at 3:28 PM, Rupert Westenthaler <[email protected]> wrote: > Hi all > > While working on the Stanbol Bundlelists I discovered that the > o.a.s.commons.web.viewable module depends on LDpath. A dependency that > is IMO unwanted. > > The reason for that is that all Stanbol JAX-RS and Web UI modules do > depend on Viewable to render the Web UI. Because of that all such > modules depend also on LDpath without actually using it. > > I see two alternatives: > > 1) moving the LDpath specific Writer to ldpathtemplate module (would > add JAX-RS dependency there) > 2) creating an additional module providing LDViewable > > In addition the ViewableWriter will need to be changed so that it no > longer used the LDPath renderer > > WDYT > Rupert > > > -- > | Rupert Westenthaler [email protected] > | Bodenlehenstraße 11 ++43-699-11108907 > | A-5500 Bischofshofen -- | Rupert Westenthaler [email protected] | Bodenlehenstraße 11 ++43-699-11108907 | A-5500 Bischofshofen
