[wp-trac] [WordPress Trac] #21698: Remove HTML from __() in load.php and wp_set_wpdb_vars()

WordPress Trac wp-trac at lists.automattic.com
Mon Aug 27 15:24:12 UTC 2012


#21698: Remove HTML from __() in load.php and wp_set_wpdb_vars()
-------------------------+------------------------------
 Reporter:  mordauk      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  I18N         |     Version:
 Severity:  minor        |  Resolution:
 Keywords:  close        |
-------------------------+------------------------------

Comment (by scribu):

 We don't even need sprintf():

 {{{
 '<p><a href="' . __( 'http://codex.wordpress.org/Appearance_Header_Screen'
 ) . '" target="_blank">' . __( 'Documentation on Custom Header' ) .
 </a></p>'
 }}}

 but it does seem cleaner:

 {{{
 sprintf( '<p><a href="%s" target="blank">%s</a></p>',
   __( 'http://codex.wordpress.org/Appearance_Header_Screen' ),
   __( 'Documentation on Custom Header' )
 );
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21698#comment:8>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list