[#DOXIA-294] Apt verbatim box not correct - jira.codehaus.org

Maven Doxia

Apt verbatim box not correct

Created: 03/Mar/09 07:34 AM Updated: 10/Mar/09 04:03 AM Resolved: 08/Mar/09 03:43 PM

Return to search

Component/s:

Module - Apt, Site Renderer

Affects Version/s:

1.1

Fix Version/s:

1.1.1

Time Tracking:

Not Specified

File Attachments:	1. dump.txt (108 kB) 2. proof.zip (0.1 kB)
Environment:	Max OS X (Leopard), Sun Java 1.5.0_16. Maven 2.0.10

Testcase included:

yes

Description

« Hide

It seems to me that there are a mess on what Doxia packages versions that are chosen when rendering a site with mvn site:site. See the attached dump.txt. It seems to me that version 1.0 of doxia is not chosen. And that older versions are used.

I have added this to my pom.xml

plugin>
<groupId>org.apache.maven.doxia</groupId>
<artifactId>doxia-maven-plugin</artifactId>
<version>1.0</version>
</plugin>

I discovered this when trying to find out why verbatime text doesn't work like it did before from an apt file.

-------------------------
Verbatime
-------------------------

gives thes same markup as

----------------------
Verbatime 2
----------------------

Both are converted into <div class="source"><pre>Verbatime</pre></div>. This is a new bug that I haven't seen before.

I have attached a small maven project that demonstrates this. demo.html generated with site:site demonstrates the problem.

Sort Order:

[ Permalink | « Hide ]

Jakob Vad Nielsen added a comment - 03/Mar/09 07:36 AM

The second verbatime lines came out the wrong way. Should have been a single line with plus in front and rear.

+ - - - - - - - + (without the spaces)

[ Permalink | « Hide ]

Lukas Theussl added a comment - 05/Mar/09 03:19 AM

I don't see anything unusual in dump.txt, what exactly are you referring to? See also ~~MNG-3402~~, (maven always uses the embedded doxia version).

The attached zip is empty, can you update it?

[ Permalink | « Hide ]

Jakob Vad Nielsen added a comment - 05/Mar/09 03:40 AM

It might be me that do not understand the dependencies correctly. But why are older versions of Doxia used?

e.g.:

[DEBUG] Retrieving parent-POM: org.apache.maven:maven-parent:pom:6 for project: org.apache.maven.doxia:doxia:pom:1.0-alpha-10 from the repository.
[DEBUG] Adding managed dependencies for unknown:doxia-core
[DEBUG] org.apache.maven.doxia:doxia-sink-api:jar:1.0-alpha-10
[DEBUG] org.apache.maven.doxia:doxia-core:jar:1.0-alpha-10
[DEBUG] org.apache.maven.doxia:doxia-core:jar:tests:1.0-alpha-10

Or am I reading it wrong?

But the important issue for me here is that I do not get different markup for ----------- ad + ----------- + anymore. Up until last week I was able to separate these verbatims from eachother via my special maven theme. Now it gives me the exact same output.

Lukas Theussl made changes - 06/Mar/09 06:18 AM

Field	Original Value	New Value
Affects Version/s	1.0 [ 14834 ]
Affects Version/s	1.0-alpha-11 [ 14265 ]
Affects Version/s		1.1 [ 13617 ]
Fix Version/s		1.1.1 [ 15073 ]
Summary	Problems with Doxia	Apt verbatim box not correct

[ Permalink | « Hide ]

Lukas Theussl added a comment - 06/Mar/09 06:20 AM

The different versions are the versions used by various transitive dependencies, they are not all used actually in the final classpath.

I confirm the verbatim box issue, we'll fix it after the 1.1 release.

[ Permalink | « Hide ]

Lukas Theussl added a comment - 08/Mar/09 03:43 PM

Fixed in r751515

Lukas Theussl made changes - 08/Mar/09 03:43 PM

Assignee		Lukas Theussl [ lukas ]
Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Closed [ 6 ]

[ Permalink | « Hide ]

Lukas Theussl added a comment - 10/Mar/09 04:03 AM

I just realized that we have the same problem in 1.0 too. It's a different issue though, for 1.1 the bug was in AptParser (this one I fixed), for 1.0 it's in XhtmlSink. The effect is the same, verbatim blocks are always boxed. I'm not sure if we'll do a 1.0.1 release but I'll open another issue for it.