[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