wpautop breaks multi-line textarea content

[takayukister]

Applying wpautop() breaks multi-line content of textarea element because it replaces each newline with <br />. A textarea can't have HTML in its content.

You can reproduce the problem with the following code:

$text = '<textarea name="test">
Lorem ipsum dolor sit amet,
consectetur adipisicing elit,
sed do eiusmod tempor
incididunt ut labore
et dolore magna aliqua.</textarea>';

echo wpautop( $text );

[takayukister]

patch for wpautop

[chriscct7]

I strongly suspect this is related to #33377.

[chriscct7]

Patch needs refresh to apply from correct directory, not a new patch. Working on it now.

[chriscct7]

Applies from proper directory

[chriscct7]

New patch from proper directory. Ready for merge post @azaozz or @miqrogroove review. Could be a 4.3.1 candidate.

[miqrogroove]

I think we need to take a very thoughtful look at why we are treating PRE differently from SCRIPT and STYLE. Until we have that sorted out, I'm not confident we have the best patch here. Anyone have thoughts on that already?

[miqrogroove]

4.3 and 4.2 both failed the attached tests. I am correcting the ticket, including the description to reflect the fact that a newline is required after any textarea element. Omitting the newline causes browsers to strip leading whitespace.

[azaozz]

The four tags that need "special treatment" are pre, scritp, style and textarea. We want to preserve line breaks/white space in them and prevent any extra tags like <br>. This needs to happen in both wpautop -- JS and PHP.

In addition:

• For <pre> we need to keep consecutive white spaces.
• For <textarea> we may have to make sure </textarea> is escaped to </textarea> (needs testing).

The above patches are a good start but thinking we can handle these four tags in the same way as suggested in comment 5. This needs to be in both JS and PHP.

[miqrogroove]

Based on the earlier test results, I'd say PRE is being handled correctly, whereas none of the others seem to work, including the patches above.

[wonderboymusic]

33466.diff​ combines patches - unit tests fail

[miqrogroove]

As long as it has my name on it, let's get real about the Milestone :) This is a maybe 4.5 ticket for now.