[theme-reviewers] pluggable functions

[Chip Bennett]

Absent pre-existing guidelines, I would list your findings as *recommended*
only. It is always good to promote and to educate regarding best practices,
but we should only ever *not-approve* (even if "required fix in next
revision") those criteria that are stated in the guidelines.

(That said: feel free to propose guidelines revisions wrt pluggable vs.
filterable functions!)

Thanks,

Chip

On Sat, Oct 27, 2012 at 2:17 PM, Kirk Wight <kwight at kwight.ca> wrote:

> Howdy,
>
> I'm reviewing a theme that has made all functions in functions.php
> pluggable, including those on hooks. From what I understand, this won't
> break anything, but doesn't feel very "best practice"-y (anything on a hook
> can just be removed from the hook, making the pluggable code un-necessary).
>
> The theme was already approved, but I prefer to encourage the best
> practice; perhaps a "fix in next release" note is appropriate, rather than
> blocking approval?..
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20121027/096b637e/attachment.htm>