[wp-trac] [WordPress Trac] #11491: Install.php confusing line

WordPress Trac wp-trac at lists.automattic.com
Mon Dec 21 17:35:28 UTC 2009


#11491: Install.php confusing line
-----------------------------+----------------------------------------------
 Reporter:  joshtime         |       Owner:           
     Type:  defect (bug)     |      Status:  new      
 Priority:  lowest           |   Milestone:  3.0      
Component:  Upgrade/Install  |     Version:  2.9      
 Severity:  minor            |    Keywords:  has-patch
-----------------------------+----------------------------------------------

Comment(by nacin):

 Replying to [comment:4 hakre]:
 > Patch adds too much spaces. Functions do not need space on both sides of
 the bracket, that's only need for some language constructs like if.

 That's the WP coding standard.

 I think it looks cleaner to avoid padding spaces inside {{{__()}}} and
 possibly {{{_e()}}} function calls. Indeed, the core adheres to this more
 than it doesn't -- 4,324 instances of __('blah') or _e('blah'), to 577
 {{{__( 'blah' )}}} and {{{_e( 'blah' )}}}. Otherwise, I'm all for the
 extra whitespace and so are the core devs.

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


More information about the wp-trac mailing list