WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 7 months ago

#47413 reviewing defect (bug)

nested paragraphs in comment post errors

Reported by: tnbnicer Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 4.5
Component: Comments Keywords: has-patch
Focuses: Cc:

Description

Line 30 in ‘wordpress/wp-comments-post.php’:

<?php
'<p>' . $comment->get_error_message() . '</p>'

An example of the Html code generated:

<p><p><strong>ERROR</strong>: please fill the required fields (name, email).</p></p>

The reason is that in function _default_wp_die_handler, line 3072 in ‘wordpress/wp-includes/functions.php’ (WordPress 5.2.1), there already is a paragraph tag.

<?php
$message = "<p>$message</p>"

I would suggest removing the paragraph in ‘wp-comments-post.php’. Simply:

<?php
$comment->get_error_message()

ought to do.

Attachments (1)

47413.patch (460 bytes) - added by dkarfa 3 years ago.
Make sense

Download all attachments as: .zip

Change History (7)

@dkarfa
3 years ago

Make sense

#1 @dkarfa
3 years ago

  • Keywords has-patch added; needs-patch removed

#2 @tnbnicer
3 years ago

Thanks. The fix works for me. To be honest, I removed the tags in my source in release 5.1.1 some weeks ago. I noticed them again while updating to 5.2.1 yesterday. Good.

Last edited 3 years ago by tnbnicer (previous) (diff)

#3 @jeremyfelt
2 years ago

  • Milestone changed from Awaiting Review to 5.3
  • Owner set to jeremyfelt
  • Status changed from new to reviewing
  • Version changed from 5.2.1 to 4.5

Thanks @tnbnicer and @dkarfa, this looks good. I also opened #47580 to address the overall problem with invalid HTML in wp_die() output as it looks like this is something that happens in many areas of core. This particular case was introduced in [36424].

#4 @davidbaumwald
2 years ago

  • Milestone changed from 5.3 to Future Release

The latest patch still applies cleanly, but this ticket hasn't seen any movement this cycle. With 5.3 RC1 approaching, this is being moved to Future Release. @jeremyfelt if you feel strongly about this one being in 5.3, it can be moved back up.

#5 @tnbnicer
2 years ago

A "complex" error message is valid:

<h1>You need a higher level of permission.</h1><p>Sorry, you are not allowed to manage terms in this taxonomy.</p>

while:

<p><h1>You need a higher level of permission.</h1><p>Sorry, you are not allowed to manage terms in this taxonomy.</p></p>

is not.

@davidbaumwald. I only occasionally look at WP code, so speaking as a lay person, I wonder whether leaving it for Future Release is sensible. There is nested html code. Still in 5.2.3. Removing the extra <p></p> tags from ‘wp-comments-post.php’ corrects the nested html. The changes proposed in [45909] are neither ruled out in future nor required now to fix the problem, but incorrect tags ought to be removed.

Last edited 2 years ago by tnbnicer (previous) (diff)

#6 @jeremyfelt
7 months ago

  • Owner jeremyfelt deleted
Note: See TracTickets for help on using tickets.