autop creates unclosed paras inside list items

[Viper007Bond]

This is what I entered into my post:

<strong>Frequently Asked Questions:</strong>

<ul>
	<li><strong>Q:</strong> My points are too low! Both BFHQ ingame and BF2S.com show more points than the plugin is currently showing. Why is that?

<strong>A:</strong> This plugin only refreshes it's data every 6 hours as BF2S.com only allows 3 queries per 6 hours per IP address. This plugin is limited to just 1 query per 6 hours incase anyone else on your server also uses this plugin, plus 6 hours isn't all that long.</li>
	<li><strong>Q:</strong> The plugin output says I'm 100% to the next rank and I have more points than required to get the next rank. What's going on?

<strong>A:</strong> Go play on a different server and get a kill. Your rank should be updated then.</li>
</ul>

This is what was outputted:

<p><strong>Frequently Asked Questions:</strong></p>
<ul>
<li><strong>Q:</strong> My points are too low! Both BFHQ ingame and BF2S.com show more points than the plugin is currently showing. Why is that?
<p><strong>A:</strong> This plugin only refreshes it&#8217;s data every 6 hours as BF2S.com only allows 3 queries per 6 hours per IP address. This plugin is limited to just 1 query per 6 hours incase anyone else on your server also uses this plugin, plus 6 hours isn&#8217;t all that long.</li>
<li><strong>Q:</strong> The plugin output says I&#8217;m 100% to the next rank and I have more points than required to get the next rank. What&#8217;s going on?

<p><strong>A:</strong> Go play on a different server and get a kill. Your rank should be updated then.</li>
</ul>

Note the lack of </p>'s.

[skeltoac]

Please note exactly what behavior you expected.

[davidhouse]

I guess,

{{{<p><strong>Frequently Asked Questions:</strong></p>
<ul>
<li><p><strong>Q:</strong> My points are too low! Both BFHQ ingame and BF2S.com show more points than the plugin is currently showing. Why is that?</p>
<p><strong>A:</strong> This plugin only refreshes it&#8217;s data every 6 hours as BF2S.com only allows 3 queries per 6 hours per IP address. This plugin is limited to just 1 query per 6 hours incase anyone else on your server also uses this plugin, plus 6 hours isn&#8217;t all that long.</p></li>
<li><p><strong>Q:</strong> The plugin output says I&#8217;m 100% to the next rank and I have more points than required to get the next rank. What&#8217;s going on?</p>

<p><strong>A:</strong> Go play on a different server and get a kill. Your rank should be updated then.</p></li>
</ul>}}}

Would be the correct behaviour.

[davidhouse]

Grr.

<p><strong>Frequently Asked Questions:</strong></p> <ul> <li><p><strong>Q:</strong> My points are too low! Both BFHQ ingame and BF2S.com show more points than the plugin is currently showing. Why is that?</p> <p><strong>A:</strong> This plugin only refreshes it&#8217;s data every 6 hours as BF2S.com only allows 3 queries per 6 hours per IP address. This plugin is limited to just 1 query per 6 hours incase anyone else on your server also uses this plugin, plus 6 hours isn&#8217;t all that long.</p></li> <li><p><strong>Q:</strong> The plugin output says I&#8217;m 100% to the next rank and I have more points than required to get the next rank. What&#8217;s going on?</p>

<p><strong>A:</strong> Go play on a different server and get a kill. Your rank should be updated then.</p></li> </ul>

[davidhouse]

Let's try that one more time with feeling.

<p><strong>Frequently Asked Questions:</strong></p>

<ul>
<li><p><strong>Q:</strong> My points are too low! Both BFHQ ingame and BF2S.com show more points than the plugin is currently showing. Why is that?</p>
<p><strong>A:</strong> This plugin only refreshes it&#8217;s data every 6 hours as BF2S.com only allows 3 queries per 6 hours per IP address. This plugin is limited to just 1 query per 6 hours incase anyone else on your server also uses this plugin, plus 6 hours isn&#8217;t all that long.</p></li>
<li><p><strong>Q:</strong> The plugin output says I&#8217;m 100% to the next rank and I have more points than required to get the next rank. What&#8217;s going on?</p>

<p><strong>A:</strong> Go play on a different server and get a kill. Your rank should be updated then.</p></li>
</ul>

[davidhouse]

Okay, I give up. Basically, the <p>s should be closed before the </li>s and probably be opened <p> after the <li>s.

[skeltoac]

I don't expect autop to nest blocks or read minds. Write a patch, use a proper definition list or use <br /> in your list items.

[Viper007Bond]

I thought I was being clear, cut me some slack please.

Let me to to re-explain. I enter this mutli-paragraph unordered list:

<ul>
	<li>Item 1, Paragraph 1

Item 1, Paragraph 2</li>
	<li>Item 2, Paragraph 1

Item 2, Paragraph 2</li>
</ul>

No <p>'s, just a list.

I get back this:

<ul>
<li>Item 1, Paragraph 1
<p>Item 1, Paragraph 2</li>
<li>Item 2, Paragraph 1
<p>Item 2, Paragraph 2</li>
</ul>

It has added in 2 <p>'s. I'm not expecting it to read my mind, but it's not even adding the closing tags for the tags it inserted.

As for a patch, I'm sorry, but I'm not as experienced in how this function works as you may be and wouldn't know where to begin on how to fix it.

Sorry for trying to make WordPress better.

[skeltoac]

Viper, I understand your bug report and I wouldn't prevent you from submitting more like it. I should have made this a balance_tags issue rather than close it. It'll be taken care of. I'm changing the summary to reflect the actual problem.

[priv]

In 2.1 the autop are more insane.

Following source:

<ul>
<li>Item 1, Paragraph 1<br /> <br />Item 1, Paragraph 2</li>
<li>Item 2, Paragraph 1<br /> <br />Item 2, Paragraph 2</li>
</ul>

after apply filter will also become

<ul>
<li>Item 1, Paragraph 1
<p>Item 1, Paragraph 2</li>
<li>Item 2, Paragraph 1
<p>Item 2, Paragraph 2</li>
</ul>

You don't have chance to add a newline in the list without break html.

And also, seems now 2.1 eat up any amount of br tags and newlines, and results in p tags.

The only way to get extra space is <p>&nbsp;</p>, and don't think of use it in the list, since it will create a three-line space. Oh, wait, maybe we can turn off the space between paragraph by inline css.

I must say the autop guys are really hate br and newlines, aren't they?

More worse, in 2.1 they're trying to strip the <p> structures created by TinyMCE before saving(to tidy the result??), so the output will have no choice but rely on autop.

If the filter will stay as a set of regexp and won't become a full parser, please don't do too much of the 'smart things', ridiculous.

[ono]

This patch fixes closing of para for last line of multiline list items

[ono]

Hi,

I've hit the same problem, and my site no longer was validating with XHTML.

Attaching patch that hopefully fixes it. Please include it into new bugfix release of WP.

Best regards!

[azaozz]

Closing as invalid. This seems to be more a question of how to use the HTML editor. Although it allows entering of raw html, it's still an editor that works together with wpautop() to replace line breaks with <p> tags.

Entering:

<ul>
<li>

Item 1, Paragraph 1

Item 1, Paragraph 2

</li>
<li>

Item 2, Paragraph 1

Item 2, Paragraph 2

</li>
</ul>

would create all <p> properly and the source on the site would look like this:

<ul>
<li>
<p>Item 1, Paragraph 1</p>
<p>Item 1, Paragraph 2</p>
</li>
<li>
<p>Item 2, Paragraph 1</p>
<p>Item 2, Paragraph 2</p>
</li>
</ul>

[Viper007Bond]

Sorry, but I have to disagree. That's just a work-around and while it produces valid XHTML, it looks rediculous when it actually renders.

Expected output should be this IMO:

<ul>
<li>Item 1, Paragraph 1<br />
<br />
Item 1, Paragraph 2</li>
<li>Item 2, Paragraph 1<br />
<br />
Item 2, Paragraph 2</li>
</ul>

But if it's gonna be wontfix, so be it I guess.