<div dir="ltr">$title in this case is a function parameter, and does not match any global variables ... it also follows common practice for the function at hand. I would accept it as is.</div><div class="gmail_extra"><br clear="all">

<div>Edward Caissie<br>aka Cais.</div>
<br><br><div class="gmail_quote">On Tue, May 27, 2014 at 8:50 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</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"><div class="gmail_default" style="font-family:verdana,sans-serif">(<i>I haven't seen the ticket in question</i>)</div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default">


<span style="font-family:verdana,sans-serif">Not sure what difference would </span><font face="courier new, monospace">$counterpoint_title</font><font face="verdana, sans-serif"> make. </font></div><div class="gmail_default">


<font face="verdana, sans-serif">This is not a <i>deal breaker</i> and it could be accepted as-is.</font></div><div class="gmail_default" style="font-family:verdana,sans-serif"><br></div><div class="gmail_default" style="font-family:verdana,sans-serif">


Em</div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Tue, May 27, 2014 at 7:15 PM, Evan Henley <span dir="ltr"><<a href="mailto:henleyedition@gmail.com" target="_blank">henleyedition@gmail.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><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 type="cite">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="margin:0px 0px 0px 40px;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></div></div>_______________________________________________<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>
<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>