wpautop error when div tags lack space before or after other text

[erik]

Texturize appears to have an issue figuring out where to put the paragraph tags if there are no lines between a div tag and the previous or next piece of text.

(Note: This is all with the visual rich editor turned off.)

I type this as the post content:

<div>
<a href="http://example.com">example.com</a>

Some pretty words here.
<div>

The source of the post when it is viewed is this:

					<div>
<a href="http://example.com">example.com</a></p>

<p>Test
</div>

The paragraph tags don't come out quite right.

---

This text, with a line between opening the div and the link:

<div>

<a href="http://example.com">example.com</a>

Some pretty words here.
</div>

Gets this source:

					<div>
<p><a href="http://example.com">example.com</a></p>
<p>Some text here.
</div>

Now the first paragraph tags open and close as expected.

---

Finally, if I put lines between both the open and close div tags and the nearest text:

<div>

<a href="http://example.com">example.com</a>

Some pretty words here.

</div>

I get this source:

					<div>
<p><a href="http://example.com">example.com</a></p>
<p>Some text here.</p>
</div>

Now both come out as expected.

[erik]

Further examples, with only one thing between the divs. These:

<div>
<a href="http://example.com">example.com</a>

</div>

<div>

<a href="http://example.com">example.com</a>
</div>

<div>
Sample text.

</div>

<div>

Sample text.
</div>

Create these:

<div>
<a href="http://example.com">example.com</a></p>
</div>

<div>
<p><a href="http://example.com">example.com</a>
</div>

<div>
Sample text.</p>
</div>

<div>
<p>Sample text.
</div>

[erik]

Oops. Not texturize, but wpautop, I think.

[Denis-de-Bernardy]

the output changed somewhat, but the bug is still current

[Denis-de-Bernardy]

See also: #3362 (same bug with blockquotes)

[hakre]

Please close as wontfix. Fixing things will break other things and vice-versa. I do not know a single serious developer that wants to touch this.

Best would be to get a binding description first what must (not)/should (not)/can (not) be done by wpautop. Then testcases must be written (executeable ones) and then the function can be developed accordingly.

related: #2833, #1706

[wonderboymusic]

Currently, trunk (using the original examples) produces:

<div>
<p><a href="http://example.com">example.com</a>Some pretty words here.</p>
</div>
<div>
<p><a href="http://example.com">example.com</a></p>
<p>Some pretty words here.</p>
</div>
<div>
<p><a href="http://example.com">example.com</a></p>
<p>Some pretty words here.</p>
</div>

I'm inclined to think this is working as expected. If you are using newlines in the editor to produce line-breaks that result in <p>s, you can preview til your heart's content.