[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