History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: DOXIA-89
Type: Bug Bug
Status: Closed Closed
Resolution: Won't Fix
Priority: Major Major
Assignee: Lukas Theussl
Reporter: Henning Schmiedehausen
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Maven Doxia

Doxia Macro syntax for xdoc sucks

Created: 04/Jan/07 02:13 AM   Updated: 12/Jul/07 05:51 AM
Component/s: Core
Affects Version/s: 1.0-alpha-8
Fix Version/s: None

Time Tracking:
Not Specified

File Attachments: 1. Text File doxia-core.patch (1 kb)

Issue Links:
Supercedes
 
dependent
 


 Description  « Hide
In the xdoc parser, Doxia uses <macro id="foo" /> to reference the 'foo' macro. This is an unfortunate choice because it allows each macro on each xdoc page only once (id must be unique).

I'd suggest that this gets changed to e.g. macroName.

Also, some macros (most prominently the snippet macro) might require an 'id' parameter, which not only clashes with the notation above but also leaves the problem that this id might clash with another id and can not be used twice on a page.

I'd suggest that in a possible Doxia manual, the usage of an 'id' parameter is strongly discouraged.

For the snippet macro, I'd suggest changing 'id' to 'snippetId'.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Henning Schmiedehausen - 04/Jan/07 02:16 AM
This patch changes the id in the xdoc parser to macroId and the id in the snippet macro to snippetId.

Jason van Zyl - 28/May/07 12:51 PM
If this incorporate using the current notation that would be fine.