<div dir="ltr">Not suppressing errors is inherent in the guideline regarding not throwing any PHP notices/errors. If Themes could avoid throwing errors via error suppression, that guideline becomes moot.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, May 22, 2013 at 4:03 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">It looked like "error suppression" to me as well, but we do not have any current guidelines referencing that technique ... or do we?<span class="HOEnZb"><font color="#888888"><br></font></span></div>
<div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div>Edward Caissie<br>

aka Cais.</div></font></span><div><div class="h5">
<br><br><div class="gmail_quote">On Wed, May 22, 2013 at 4:02 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div dir="ltr">The issue of passing anonymous functions to hooks is certainly valid. Hooks must reference a valid callback, so that they can be overridden.<div><br></div><div>Another major concern is that the Theme *requires* a PHP version higher than that required by WordPress core.</div>



<div><br></div><div>I also agree on the overall structure of the Theme; it completely circumvents the template hierarchy in a way that would prove extremely difficult for an end user to overcome.</div><div>
<br></div><div>I also see what I think is error suppression?</div></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 22, 2013 at 3:46 PM, esmi at quirm dot net <span dir="ltr"><<a href="mailto:esmi@quirm.net" target="_blank">esmi@quirm.net</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><<a href="https://themes.trac.wordpress.org/ticket/12206" target="_blank">https://themes.trac.<u></u>wordpress.org/ticket/12206</a>><br>




<br>
Is using <?php namespace theme; ?> a reasonable excuse for then using 'theme' as a textdomain? My gut instinct says "no".<br>
<br>
That said, this theme is totally unlike any standard WPORG theme. The whole thing is pure actions hooks & filters (which is cool) but...<br>
<br>
1. functions are not being named. - they're being added (wholesale) as the second argument to add_action and add_filter.<br>
2. locate_template() is being used instead of get_template().<br>
<br>
In general, trying to apply the guidelines to this theme is like trying to plait fog. It looks more like it's intended as a framework than a theme. Anyone care to weigh in?<br>
<br>
Mel<span><font color="#888888"><br>
-- <br>
<a href="http://quirm.net" target="_blank">http://quirm.net</a><br>
<a href="http://blackwidows.co.uk" target="_blank">http://blackwidows.co.uk</a><br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</font></span></blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div></div></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>