[wp-trac] [WordPress Trac] #52627: Coding Standards fixes for WP 5.8

WordPress Trac noreply at wordpress.org
Thu Apr 1 18:30:51 UTC 2021


#52627: Coding Standards fixes for WP 5.8
----------------------------+-------------------------------
 Reporter:  SergeyBiryukov  |       Owner:  (none)
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  5.8
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:  coding-standards
----------------------------+-------------------------------

Comment (by davidbaumwald):

 Replying to [comment:9 mukesh27]:
 > @davidbaumwald Do we need to remove parentheses from here
 https://github.com/WordPress/WordPress/blob/master/wp-includes/class-
 requests.php#L146?
 >
 > Replying to [comment:7 davidbaumwald]:
 > > In [50631]:
 > >
 > > Coding Standards: Removing unnecessary parentheses from `require_once`
 in `wp-admin/options-privacy.php`.
 > >
 > > In [47198], parentheses were removed from `include` and `require`
 statements, as they are language constructs, not function calls.  However,
 [50161] introduced a new `require_once` instance in `wp-admin/options-
 privacy.php`.  This change removes the unnecessary parentheses.
 > >
 > > See #53627.

 @mukesh27 Good catch, but that's one of the third-party library
 exclusions.  See
 https://core.trac.wordpress.org/browser/tags/5.7/phpcs.xml.dist#L116

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


More information about the wp-trac mailing list