[wp-trac] [WordPress Trac] #57042: URL escaping added in 'about.php' file
WordPress Trac
noreply at wordpress.org
Thu Nov 10 00:02:02 UTC 2022
#57042: URL escaping added in 'about.php' file
---------------------------+-----------------------------------------------
Reporter: monzuralam | Owner: (none)
Type: defect (bug) | Status: closed
Priority: normal | Milestone:
Component: Help/About | Version: 6.1
Severity: normal | Resolution: wontfix
Keywords: has-patch | Focuses: administration, coding-standards
close |
---------------------------+-----------------------------------------------
Changes (by peterwilsoncc):
* status: new => closed
* resolution: => wontfix
* milestone: 6.1.1 =>
Comment:
I agree this can be closed without a fix.
While the intent is good, the escaping functions are only needed for user
input. As the URL is hard coded in this instance, escaping isn't required.
It's a nuanced point and I realise that the inconsistently in WordPress
contributes to the confusion, so I really appreciate your taking the time
to open the ticket @monzuralam
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57042#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list