Make WordPress Core

Opened 6 years ago

Last modified 6 years ago

#50498 new defect (bug)

Change "html" to "HTML" in atomlib.php

Reported by: man4toman Owned by:
Priority: normal Milestone: Awaiting Review
Component: Feeds Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

Follow #50473, there is a string in wp-includes\atomlib.php

https://core.trac.wordpress.org/browser/tags/5.4/src/wp-includes/atomlib.php#L217

Invalid content in element found. Content must not be of type text or html if it contains markup.

Attachments (2)

50498.patch (627 bytes ) - added by burhandodhy 6 years ago.
Replace html with HTML
#50498.patch (584 bytes ) - added by rehanali 5 years ago.
Improves the documentation.Replaced the "html" with "HTML".

Download all attachments as: .zip

Change History (8)

@burhandodhy
6 years ago

Replace html with HTML

#1 @SergeyBiryukov
6 years ago

Thanks for the ticket!

Similarly to class-pclzip.php in #49163, atomlib.php used to be treated as an external library and was previously excluded from any documentation or coding standard fixes. We only modify third-party libraries when it's needed to fix a bug, like in [37461], [38883], or [39591].

That said, looking at the commit history, there were some string changes in the past.

I think it's safe to treat atomlib.php as "adopted" library rather than external. While I don't feel strongly about making this change, I don't see any harm in it either.

#2 @SergeyBiryukov
6 years ago

  • Keywords has-patch added; needs-patch removed

#3 @SergeyBiryukov
6 years ago

  • Focuses docs added

#4 @SergeyBiryukov
6 years ago

  • Component GeneralFeeds

@rehanali
5 years ago

Improves the documentation.Replaced the "html" with "HTML".

Note: See TracTickets for help on using tickets.