Markdown in Python: `</li>\n</ul>` is wrapped in `p` tags, producing invalid HTML

Anonymous users must enter captcha below.

Don't put anything here

Ticket Information

Ticket Title

This is a _very_ strange bug.

Markdown:

Paragraph.
	<ul>
	<li>
	Paragraph.
	</li>
	</ul>

Desired HTML output:

Paragraph.
	<ul>
	<li>
	Paragraph.
	</li>
	</ul>

Actual HTML output:

Paragraph.
	<ul>
	<li>
	Paragraph.
	
	</li>
	</ul>

I've simplified the input text as much as possible – correct behaviour results if either paragraph is removed, or if the first paragraph is replaced by its raw Markdown equivalent (i.e. "Paragraph.\n" rather than "&lt;p&gt;Paragraph.&lt;/p&gt;").

**The input text is valid HTML and should be left as is.**

###Comments

__By Waylan on 7/14/2010__

First, let's run the given sample through the parser

>>> md = markdown.Markdown()
 >>> md.convert("foo\n<ul>\n<li>\nbar\n</li>\n</ul>")
 u'foo\n<ul>\n<li>\nbar\n\n</li>\n</ul>'
 >>> md.htmlStash.rawHtmlBlocks
 [(u'foo\n<ul>\n<li>\nbar', False), (u'</li>', False), (u'</ul>', False)]

Notice that the raw html is seen as three separate blocks and the second two are closing tags only. Thats why they are getting wrapped in a `` tag.

Now let's remove that first paragraph from the input text:

>>> md.reset()
 >>> md.convert("<ul>\n<li>\nbar\n</li>\n</ul>")
 u'<ul>\n<li>\nbar\n</li>\n</ul>'
 >>> md.htmlStash.rawHtmlBlocks
 [(u'<ul>\n<li>\nbar\n</li>\n</ul>', False)]

This time everything is seen as one block so it works correctly.  So the trick is to figure out why the rawhtml parser is thrown off by the first paragraph so that is doesn't include the closing tags in the last two lines in the same block -- and then to fix it.

Now one final test:

>>> md.reset()
 >>> md.convert("foo\n\n<ul>\n<li>\nbar\n</li>\n</ul>")
 u'foo\n\n<ul>\n<li>\nbar\n</li>\n</ul>'
 >>> md.htmlStash.rawHtmlBlocks
 [(u'foo', False), (u'<ul>\n<li>\nbar\n</li>\n</ul>', False)]

Here I added a blank line between the first paragraph and the list and everything works as it should. In the input that breaks I suspect that the parser is matching the first `` with the last `` and completely ignores the `<ul><li>` in the middle. So to fix it we need the parser to stop on the first ``, then restart on the list. Doesn't really matter if everything is all in one block or two as long as each block contains a valid block of html.

Reported by

Assigned to

Status

Resolution

Don't put anything here

Advanced Fields

Priority

Resolution Explanation

Component

About This Edit

Minor Edit

Edit Summary

Don't put anything here