<div dir="ltr">I would have commented in-ticket, but Justin covered it. :)<div><br></div><div>I would have recommended handling it in-ticket rather than the mail-list, but it seemed like a beneficial question and discussion.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 28, 2014 at 5:23 PM, Zulfikar Nore <span dir="ltr"><<a href="mailto:zulfikarnore@live.com" target="_blank">zulfikarnore@live.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div dir="ltr">I think it would be more constructive if you guys would first take a look at the ticket together with the issues raised and then comment in ticket either way. Commenting on the mailing list leaves issue in the air where neither the reviewer nor the author benefits and learn from your experience and knowledge :)<br>
<br><div><hr>Date: Tue, 27 May 2014 21:47:48 -0500<br>From: <a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a><br>To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
Subject: Re: [theme-reviewers] Prefixing Non-Global Variables<div><div class="h5"><br><br><p dir="ltr">No, because that's stupid.</p>
<p dir="ltr">Feel free to correct the reviewers when you think they are wrong. Theme review is a dialog, not a word handed down from up on high. Everybody makes mistakes. Don't judge, just try to achieve balance. </p>
<p dir="ltr">The goal is to produce the best code possible and to not give broken code users. Work towards that end, all around.<br></p>
<p dir="ltr"> </p>
<div>On May 27, 2014 7:16 PM, "Evan Henley" <<a href="mailto:henleyedition@gmail.com" target="_blank">henleyedition@gmail.com</a>> wrote:<br><blockquote style="border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div>Hi all,</div><div><br></div>This is regarding the following ticket: <a href="https://themes.trac.wordpress.org/ticket/18381" target="_blank">https://themes.trac.wordpress.org/ticket/18381</a><div>
<br></div><div>There is some conflict over the interpretation of the rules for prefixing variables:</div><div><br></div><div><blockquote>Themes are required to use theme-slug ( or a reasonably unique slug) as a prefix for anything in the public namespace, including all custom function names, classes, hooks, public/global variables, database entries (Theme options, post custom metadata, etc.) </blockquote>
</div><div><br></div><div>I am being told I need to prefix every variable used in my theme. Is it correct that even non-global variables need to be prefixed?</div><div><br></div><div>For instance, I’m told that <font face="Menlo">$title</font> needs to be changed to <font face="Menlo">$counterpoint_title</font>:</div>
<div><br></div><blockquote style="border:none;padding:0px"><div><div><span style="font-family:Menlo">function counterpoint_no_title($title) {</span></div></div><div><div><font face="Menlo"> if ( $title == '' ) $title = __('Untitled', 'counterpoint');</font></div>
</div><div><div><font face="Menlo"> return $title;</font></div></div><div><div><font face="Menlo">}</font></div></div><div><div><font face="Menlo">add_filter( 'the_title', 'counterpoint_no_title' );</font></div>
</div></blockquote><div><div><br></div><div>Any insight would be much appreciated.</div><div><br></div><div>Thanks,</div><div>Evan</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>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></div></div></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>