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

Key: DOXIA-79
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Vincent Siveton
Reporter: Henning Schmiedehausen
Votes: 1
Watchers: 1
Operations

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

Doxia generates different XHTML for the same xdoc code

Created: 12/Dec/06 04:49 PM   Updated: 21/Jul/07 05:20 AM
Component/s: Core
Affects Version/s: 1.0-alpha-8, 1.0
Fix Version/s: 1.0-alpha-9

Time Tracking:
Not Specified

Issue Links:
Duplicate
 
Related
 


 Description  « Hide
Consider the following xdoc code:

<section name="1">
<subsection name="s1">
<p>p1</p>
<ul>
<li>l1</li>
</ul>
</subsection>
<subsection name="s2">
<p>p2</p>
<ul>
<li>l1</li>
</ul>
</subsection>
</section>

This renders to the following XHTML code:

<a name="1"></a><div class="section"><h2>1</h2>
<a name="s1"></a><div class="section"><h3>s1</h3>
<p>p1</p>
<ul><li>l1</li></ul>
</div>
<a name="s2"></a><div class="section"><h3>s2</h3>
p2
<ul><li>l1</li></ul>
</div>
</div>

Please not the missing <p> </p> around 'p2'.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Wendy Smoak - 12/Dec/06 04:55 PM
This looks like the same thing as http://jira.codehaus.org/browse/MSITE-153.

Vincent Siveton - 16/Jan/07 08:15 PM
Already fixed

Henning Schmiedehausen - 24/Jan/07 04:23 PM
Nah. I don't believe you.

I just fetched the Doxia HEAD, built it and the issue is not only still there but for the fun of it I've also opened DOXIA-93, DOXIA-94, DOXIA-95 and DOXIA-96 just to show you that this thing still has not just "a few bugs".


Stefan Broetz - 25/Jan/07 02:43 AM
Henning, I was able to workaround this issue by modifying the xdoc list items in the following way:

<ul>
<li>
<p>Bla bla bla</p>
</li>
</ul>

Note the additional paragraph tag around each list item content. This may or may not work with HEAD, but it seems to work for Maven 2.0.4 and its dependencies (at least for me...)


Vincent Siveton - 27/Jan/07 08:26 AM
Henning, why don't you trust me? (

We have several test cases in site-renderer to demonstrate that this issue was solved

https://svn.apache.org/repos/asf/maven/doxia/trunk/doxia-site-renderer/src/test/java/org/apache/maven/doxia/siterenderer/DefaultSiteRendererTest.java