[theme-reviewers] Theme Check: enqueuing 'comment-reply' as a dependency fails the theme

Edward Caissie edward.caissie at gmail.com
Wed Jul 13 21:57:39 UTC 2011


Nice adaptation to look for comment-reply only.


Cais.


On Wed, Jul 13, 2011 at 3:38 PM, Simon Prosser <pross at pross.org.uk> wrote:

> On 13 July 2011 20:05, Otto <otto at ottodestruct.com> wrote:
> > On Wed, Jul 13, 2011 at 1:57 PM, Chip Bennett <chip at chipbennett.net>
> wrote:
> >> Perhaps make it an "INFO" if "comment-reply" is found, but not in the
> >> expected location? That way, the reviewer is flagged to verify the
> >> implementation?
> >
> > Yep, had the same thought here. I just wrote a comment reply specific
> > check. It looks for the full method, then if it doesn't find that, it
> > looks for just "comment-reply" anywhere. If neither is found, that's a
> > fail. If only the "comment-reply" is found, then it generates an INFO
> > with:
> >
> > "INFO: Could not find the comment-reply script enqueued, however a
> > reference to 'comment-reply' was found. Make sure that the
> > comment-reply script is being enqueued properly on singular pages."
> >
> > -Otto
>
> Thanks Otto, btw love the plugin diff emails!
>
>
>
> --
> My Blog: http://www.pross.org.uk/
> Plugins : http://www.pross.org.uk/plugins/
> Themes: http://wordpress.org/extend/themes/profile/pross
> _______________________________________________
> 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/20110713/8eb84aeb/attachment.htm>


More information about the theme-reviewers mailing list