Make WordPress Core

Opened 15 years ago

Closed 12 years ago

#2239 closed defect (bug) (fixed)

Nested pre/blockquote in li produces invalid XHTML

Reported by: Juergen Owned by: pishmishy
Milestone: 2.6.1 Priority: normal
Severity: normal Version: 2.0
Component: General Keywords: has-patch formatting wpautop xhtml
Focuses: Cc:


Entering the following code in the plain editor produces invalid XHTML in WP 2.0 (worked fine with 1.5):


The result has an unclosed p:


If one inserts an empty line before the closing li after "after", the p gets closed.

Attachments (1)

2239.patch (1.4 KB) - added by pishmishy 12 years ago.
freshened patch (woops)

Download all attachments as: .zip

Change History (10)

#1 @jhodgdon
13 years ago

  • Keywords needs-patch formatting added
  • Milestone set to 2.2

This is still happening in SVN version [4984], exactly as described. Seems like it should be fixed...

#2 @foolswisdom
13 years ago

  • Milestone changed from 2.2 to 2.3

#3 @ryan
13 years ago

  • Milestone changed from 2.3 to 2.4

#4 @pishmishy
13 years ago

Still present in trunk. Bug is somewhere within wptexturize or wpautop.

#5 @pishmishy
13 years ago

  • Keywords has-patch wpautop added; needs-patch removed

This is caused by the ordering of replacements in wpautop(). "\n\n" is purposely inserted by one rule and later replaced with a <p> by another. The patch reorders the rules to avoid this.

#6 @pishmishy
13 years ago

  • Owner changed from anonymous to pishmishy
  • Status changed from new to assigned

#7 @pishmishy
12 years ago

Still present in trunk.

12 years ago

freshened patch (woops)

#8 @pishmishy
12 years ago

  • Keywords xhtml added

Attached a patch that cleanly applies to the current trunk code. Tested and appears to works fine.

#9 @azaozz
12 years ago

  • Milestone changed from 2.9 to 2.6.1
  • Resolution set to fixed
  • Status changed from assigned to closed

The current code in wpautop() seems to be exactly the same as the patch (apart from some insignificant white space). Closing as fixed, feel free to reopen if needed.

Note: See TracTickets for help on using tickets.