Brian Ewins <[EMAIL PROTECTED]> wrote on 19/12/2002 04:01:04 AM:
> dIon, I took a tour of the code thats handling putting the reports in,
> it seems a bit fragile to me. Any change to which reports are included
> requires changes to the maven-reports.xml, site.jsl, site.dvsl, ...
> Also, if you generate a report for yourself that isnt in the standard
> list, you need to add the navigation for it by hand. It adds a barrier
> to entry to anyone writing a plugin that produces reports.
Yep.
> I was going to suggest (or preferably implement) a more modular
> approach: each reporting tool will 'register' its report and both the
> maven-reports.xml, reports navigation are generated from these report
> descriptors. This would mean that the xdoc plugin would no longer have
> any knowledge of any specific reporting plugins, and the documentation
> of what the reports are for are in the reporting plugins instead of in
> the xdoc plugin.
Sounds great
> To get this to work, I was hoping to be able to define a jelly tag that
> would do the work of writing the descriptors. I reckoned I'd implement
> this by merging descriptors directly into maven-reports.xml, like so:
>
> <jsl:template match="table">
> <jsl:applyTemplates select="tr"/>
> <tr><td><a href="${blah.location}">${blah.title}</a></td>
> <td><!-- blah.description ... not sure what to put here to
> produce
valid xml from a property? --></td>
> </tr>
So this would be a report:display-all type of tag, yeah?
> </jsl:template>
> <!-- however the identity transform looks in jsl... -->
> <jsl:template match="*">
> </jsl:template>
>
> and then build the jelly tag so that calls to this would look something
> like:
> <report:register location="blah" title="blah" description="blah"/>
Cool.
> However obviously I don't know enough jelly - how would I wrap this up
> in a jelly 'define:tag' tag? And how do I emit properly escaped xml for
> the description? Any ideas?
The register tag would have to store the entries in the jelly context, and
the display-all tag would have to simply write them out.
Does this help, or do you need more info on define:tag itself?
--
dIon Gillard, Multitask Consulting
Blog: http://www.freeroller.net/page/dion/Weblog
Work: http://www.multitask.com.au
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
