[wp-trac] [WordPress Trac] #23012: Refresh the code for the default widgets

WordPress Trac noreply at wordpress.org
Fri Jul 3 13:13:40 UTC 2015


#23012: Refresh the code for the default widgets
-------------------------------------------------+-------------------------
 Reporter:  Viper007Bond                         |       Owner:  chriscct7
     Type:  enhancement                          |      Status:  reviewing
 Priority:  normal                               |   Milestone:  Future
Component:  Widgets                              |  Release
 Severity:  normal                               |     Version:  3.5
 Keywords:  good-first-bug has-patch dev-        |  Resolution:
  feedback                                       |     Focuses:
-------------------------------------------------+-------------------------

Comment (by jdgrimes):

 Replying to [comment:19 chriscct7]:
 > @welcher The changes like ```esc_html_e( 'Page IDs, separated by
 commas.' );``` where the string is hardcoded don't need to be escaped
 (because they are hardcoded). Running esc_html on those just is a little
 performance hit for no real gain there
 >

 [https://github.com/WordPress-Coding-Standards/WordPress-Coding-
 Standards/issues/159#issuecomment-46705167 Some would disagree] (myself
 included).

--
Ticket URL: <https://core.trac.wordpress.org/ticket/23012#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list