The XSL template for <wadl:doc> generates wrong HTML in some cases
Bug #426858 reported by
Michael Bienia
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Benji York | ||
Bug Description
While working on a Table of Contents for the API documentation, I've also fixed some HTML errors in the XSL stylesheet (see http://
But one error I couldn't fix with my (limited) knowledge of XSL/XPath: the template for <wadl:doc> generates a <p>...</p> around its content. This is OK unless the content already contains markup itself (some documentation strings have one (or more) <p>...</p>). In this case wrong HTML is generated (e.g. <p><p>...</p></p>) which the W3C validator marks as an error.
Related branches
lp:~benji/launchpadlib/fix-nested-p-tags
- Curtis Hovey (community): Approve (code)
- j.c.sackett (community): Approve (code*)
-
Diff: 53 lines (+20/-4)3 files modifiedsrc/launchpadlib/NEWS.txt (+5/-0)
src/launchpadlib/__init__.py (+1/-1)
src/launchpadlib/wadl-to-refhtml.xsl (+14/-3)
Revision history for this message
| Gary Poster (gary) wrote | #1 |
| Changed in launchpadlib: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| assignee: | nobody → Benji York (benji) |
| affects: | launchpadlib → launchpad-foundations |
| Changed in launchpad-foundations: | |
| status: | Triaged → In Progress |
| Changed in launchpad-foundations: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| Changed in launchpad: | |
| milestone: | none → 11.01 |
To post a comment you must log in.
I believe we want to move the XSLT to the LP tree, so moving this bug to the Foundations team.