[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