J.D. Grimes

Results 172 comments of J.D. Grimes

I've just run into a plugin conflict caused by `calendar.js`. The [function listening to `$(document).click()`](https://github.com/Automattic/Edit-Flow/blob/master/modules/calendar/lib/calendar.js#L119) was getting called on a plugin's admin screen, since it is loaded on every page...

You could just exclude all `index.php` files in your [custom XML config file](https://github.com/squizlabs/PHP_CodeSniffer/wiki/Annotated-ruleset.xml): ``` xml */index.php ```

I agree with @GaryJones that it usually doesn't make sense to include index.php files in every directory in an open source project. They are only needed in directories which need...

I'm rethinking this. According to [the handbook](https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#6-file-headers): > Whenever possible, _all_ WordPress files should contain a header DocBlock, regardless of the file’s contents – this includes files containing classes. [emphasis...

> Question: Would you want the Handbook to advise plugin and theme developers to use index.php files in their directories? No directories within the plugin should contain user content, because...

> Just checking - was the actual issue caused by the use of add_action() or was it caused by the hooked in function not returning the expected value ... ?...

Duplicate of https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/469?

I thought that at one point this might have been supported, but I just checked against 0.10.0 and 0.11.0 and both also give the error. So this is apparently not...

Note that the `implode()` part should already be handled correctly, see #515.

No, that isn't supported. The feature could be added though.