Make WordPress Core

Opened 10 years ago

Closed 7 years ago

Last modified 7 years ago

#7009 closed defect (bug) (invalid)

Invalid XHTML Produced by Comments

Reported by: singpolyma Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.7
Component: Formatting Keywords:
Focuses: Cc:


If you look at http://singpolyma.net/plugins/actionstream/#comment-67645 you will see that a <p> tag is opening before a <code> tag - and then closing *inside* the <code> tag - clearly in violation of XHTML (or even HTML4) specs. There is a similar problem near the </code>. These <p> blocks are being generated by WordPress (the <code> tags were entered by the user), so it seems to be a bug.

Change History (11)

#1 @DD32
10 years ago

  • Milestone set to 2.6
  • Version set to 2.5

#2 @mrmist
9 years ago

  • Keywords needs-patch added
  • Milestone changed from 2.9 to 2.8
  • Priority changed from high to normal
  • Severity changed from major to normal
  • Version changed from 2.5 to 2.7

Still reproducable.

#3 @ryan
9 years ago

  • Component changed from General to Comments
  • Owner anonymous deleted

#4 @Denis-de-Bernardy
9 years ago

  • Milestone changed from 2.8 to Future Release

#5 @Denis-de-Bernardy
9 years ago

there is a balanceTags call on pre_comment_content. shouldn't this deal with it, or is it really a balanceTags bug?

#6 @Denis-de-Bernardy
9 years ago

  • Component changed from Comments to Formatting
  • Milestone changed from Future Release to 2.9

#7 @dwright
9 years ago

  • Cc david_v_wright@… added

I can not reproduce this issue.

how did you reproduce this issue?

the site in question appears fine to me.

<p><code><br> Warning: simplexml_load_string() [function.simplexml-load-string]: Entity: line 1: parser error : Start tag expected, '&lt;' not found in /var/www/vhosts/dev.boldlygoing.com/httpdocs/wp-content/plugins/wp-diso-actionstream/actionstream.php on line 323</code></p>

#8 @ryan
9 years ago

  • Milestone changed from 2.9 to Future Release

#9 @solarissmoke
7 years ago

  • Keywords close added; needs-patch removed

Can't reproduce - and the HTML in the link provided looks fine as dwright says.

#10 @solarissmoke
7 years ago

  • Keywords close removed
  • Resolution set to invalid
  • Status changed from new to closed

No feedback on how to reproduce.

#11 @ocean90
7 years ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.