<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 style>Another major concern is that the Theme *requires* a PHP version higher than that required by WordPress core.</div>
<div style><br></div><div style>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 style>
<br></div><div style>I also see what I think is error suppression?</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 class="HOEnZb"><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>