slachiewicz opened a new pull request, #1069:
URL: https://github.com/apache/maven-doxia/pull/1069
MacroBlock.traverse() located the end of the macro with indexOf('}'),
which is the *first* closing brace. A parameter value containing braces
therefore truncated the macro, silently dropping the rest of the value.
This shows up when the raw source of a *.apt.vm file is parsed, where
Velocity references are still unresolved:
%{snippet|file=${project.build.directory}/test-classes/resolve.txt}
parsed as file=${project.build.directory}, dropping the path. In a normal
site build Velocity runs first so no brace remains, but doxia-converter
parses the unprocessed source and lost the parameter.
The macro block always ends with the closing brace, so use lastIndexOf.
Part of the wider migration tracked in
https://github.com/apache/maven-doxia-converter/issues/139
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
