forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Crossley <>
Subject Re: Suggested changes to wiki plugin
Date Thu, 17 Jan 2013 00:08:42 GMT
Sjur Moshagen wrote:
> Hi all,
> I needed to fix some bugs (or missing features) in the wiki plugin, to better cope with
the jspwiki format. So far I have only stored the changes locally in my project folder (thus
the previous commit to fix the locationmap for finding resource files). Now I would like to
move the changes to the plugin, but there are one change that I would like feedback on before
I do this.
> The changes I have made are these:
> * I added support for formatting and links within table cells, so that tables now should
behave as the jspwiki spec [1] says
> * I added support for definition lists (terms + definition), but *NOT* in accordance
with the jspwiki spec
> The reason for not following the spec was that I could not get the Chaperon grammar (which
is used to parse the jspwiki document and convert it to xml) to accept the syntax used by
jspwiki. I don't know why, but to me it looks like a bug in the Chaperon lexer.
> The jspwiki syntax is:
> ; Term : definition

I wonder if it is the "space" characters. The reference [1] shows no spaces.

> My syntax is:
> #; Term
> ##: definition
> Not as elegant, but it works, and my implementation allows for links, formatting and
breaks within the definition (which should be in accordance with the jspwiki syntax).
> Now the question(s):
> * is it ok with you if I commit these changes, even though one is with a different syntax
than expected? I will of course update the documentation accordingly

I reckon that it is fine to add whatever solution enables it to work for you.
We can always issue a new version of the plugin later that fixes it differently.

> * it is said in the wiki plugin docs that we should keep the sources in sync with the
Cocoon 2.1 block - but does that even apply for the syntax-breaking changes?

I reckon so. There is not enough activity to warrant separate versions.


> Best regards,
> Sjur
> [1]

View raw message