Opened 2 years ago
Last modified 2 years ago
#56562 assigned defect (bug)
<br> Tag should be self closing on wp-mail.php file
Reported by: | rajanpanchal2028 | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Keywords: | has-patch close | |
Focuses: | coding-standards | Cc: |
Description
Previously <br> tag is not self-closing so I have make <br> tag self-closing.
Attachments (1)
Change History (7)
#3
@
2 years ago
- Keywords close added
Hi there, welcome back to WordPress Trac! Thanks for the patch.
I think the current code is correct, see the note for the strip_tags() function in the PHP Manual:
Self-closing XHTML tags are ignored and only non-self-closing tags should be used in
allowed_tags
. For example, to allow both<br>
and<br/>
, you should use:
<?php strip_tags($input, '<br>'); ?>
#4
@
2 years ago
@SergeyBiryukov is correct: https://3v4l.org/nvbKe
I agree with close
.
Note: See
TracTickets for help on using
tickets.
Patch added.