[wp-trac] [WordPress Trac] #56674: Twenty Ten: Use 'esc_url' for 'get_author_posts_url' where appropriate
WordPress Trac
noreply at wordpress.org
Wed Sep 28 10:07:35 UTC 2022
#56674: Twenty Ten: Use 'esc_url' for 'get_author_posts_url' where appropriate
---------------------------+-------------------------------
Reporter: hztyfoon | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: Bundled Theme | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses: coding-standards
---------------------------+-------------------------------
Comment (by rudlinkon):
Replying to [comment:6 SergeyBiryukov]:
> Replying to [comment:5 audrasjb]:
> > Removing from 6.1 as the beta cycle already started and this is not
something introduced by 6.1.
>
> Right, but I think this is just another coding standards update along
the lines of [54341] / #56667. This would bring more consistency, as
`get_author_posts_url()` appears to be escaped in other bundled themes, as
well as in other files of the same theme (two links are escaped and two
are not, which is fixed by this patch).
>
> So if that change went in, it seems like this should go in too :)
Otherwise we'd have to bump the theme version just for [54341], and then
bump it again just for this patch.
Yes, you are right, I think this patch should publish in 6.1 because
already some of this type of issue is accepted in 6.1 and this is not a
big change.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/56674#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list