[wp-trac] [WordPress Trac] #41057: Update PHP codebase per WordPress PHP Coding Standards

WordPress Trac noreply at wordpress.org
Fri Jun 16 15:12:56 UTC 2017


#41057: Update PHP codebase per WordPress PHP Coding Standards
----------------------------+-----------------------
 Reporter:  netweb          |       Owner:  pento
     Type:  task (blessed)  |      Status:  accepted
 Priority:  normal          |   Milestone:  4.9
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:
----------------------------+-----------------------

Comment (by pento):

 [attachment:phpcs.xml.dist] is what I have so far. The
 `WordPress.Arrays.ArrayIndentation` rule is to work around
 [https://github.com/WordPress-Coding-Standards/WordPress-Coding-
 Standards/issues/968 this issue], but it seems there are a few more edge
 cases to be tackled.

 If you run `phpcbf` with WPCS develop and PHPCS 2.9.1, you'll see a
 handful of files that aren't fixed, it seems due to clashing indentation
 rules.

 There've been some instances of a line in an `else {}` clause being moved
 up to the same line as the `else`, you can see them with this search: `ack
 'else {.+$' --php`.

 I've also noticed some cases of massive indents being added, which isn't
 ideal.

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


More information about the wp-trac mailing list