[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