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

WordPress Trac noreply at wordpress.org
Wed Oct 11 06:11:24 UTC 2017


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

Comment (by pento):

 I had thought that [attachment:41057.2.diff] was just a one off. As it
 turns out, it is not - I found 44 instances using this:

 `ack --php '<[^?](?:[^>]+|<\?php[^>]+\?>)+(<\?php\s+if)' src`

 I don't really expect WPCS to handle that, because that's really bad code,
 and we should feel bad about it. These instances will need to be manually
 fixed before `phpcbf` can be run across the codebase.

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


More information about the wp-trac mailing list