wpautop() should not break up script and object tags

[Viper007Bond]

Make a post with this:

Test<object><param name="text" value="test" /></object>test

Results in this:

<p>Test<br />
<object>
<param name="text" value="test" /></object>
<p>test</p>

The <br /> probably shouldn't be there, there shouldn't be a line break after the <object>, and most importantly, there's a missing </p>.

[Viper007Bond]

Another example:

Test

<span>Test</span><script type="text/javascript">/* Test */</script>

Test

p>Test</p>
<p><span>Test</span><br />
<script type="text/javascript">/* Test */</script>
<p>Test</p>

[Viper007Bond]

Er...

<p>Test</p>
<p><span>Test</span><br />
<script type="text/javascript">/* Test */</script>
<p>Test</p>

[Viper007Bond]

See also: #3476

(Forgot I already made a ticket on this quite a while back.)

[foolswisdom]

Another duplicate is #3679

[mm6]

patch to make wpautop() not put line breaks before and after object and script tags

[mm6]

New output with wpautop_dont_break_script_object.patch:

In:

Test<object><param name="text" value="test" /></object>test

Out:

<p>Test<object>
<param name="text" value="test" /></object>test</p>

[dllmain]

I had trouble with broken object tags in 2.1.2, too.
I fixed this in another way that at least worked with
raw posts from phpMyAdmin.
I changed line 75 of formatting.php from

$pee = preg_replace('/<(script|style).*?<\/\\1>/se', 'str_replace("\n", "<WPPreserveNewline />", "\\0")', $pee);

to

$pee = preg_replace('/<(object|script|style).*?<\/\\1>/se', 'str_replace("\n", "<WPPreserveNewline />", "\\0")', $pee);

I have no idea what that code actually is doing - just though I should share this.
Would be great if you could save me patching this in future releases.

[Otto42]

It would be nice if we could get dllmain's patch into 2.2 for another reason. That bit of code prevents it from converting newlines into BR's inside script and style blocks. By adding object blocks to that, it would eliminate a lot of the issues people have with pasting in the object codes given out from sites like youtube and such.

[foolswisdom]

Otto42, do you currently run with this patch installed? I will install it now on my live blog as well.

[foolswisdom]

How does this relate to the issue and fix in #3854?

[Otto42]

With this patch? No.

With the adjustment given in dllmain's comment above, yes.

I have not analyzed the diff attached to this ticket, but I have made the same core hack that dllmain describes in his comment above, for probably the same reasons he did.

[rob1n]

This was fixed in [5262]. I can't reproduce anymore in r5269.