The only valid exception (or, more properly: corollary) to the rule that I know of would be function declarations inside of classes - in which case, the Class itself must be properly prefixed.<div><br></div><div>Chip<br><br>
<div class="gmail_quote">On Mon, Jul 25, 2011 at 8:13 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com">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;">
It is simply a better safe than sorry approach. I cannot think of any function that should not be properly prefixed with an appropriate namespace, especially those related to a specific theme.<br><br><br clear="all">Cais.<div>
<div></div><div class="h5"><br>
<br><br><div class="gmail_quote">On Mon, Jul 25, 2011 at 7:41 AM, 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">
Why would a library be exempted? Its functions wind up in the run-time just the same as any other function, and thus would have the same opportunity for naming conflicts.<div><br></div><div>(And in this case in particular, the non-prefixed function names are sufficiently generic as to have a high chance of such conflicts: remove_generator_link(), include_all(), get_profile_uri(), add_lightbox_rel(), etc.)</div>
<div><br></div><div><font color="#888888">Chip<br><br></font><div class="gmail_quote"><div>On Mon, Jul 25, 2011 at 4:31 AM, Vicky Arulsingam <span dir="ltr"><<a href="mailto:vicky.arulsingam@gmail.com" target="_blank">vicky.arulsingam@gmail.com</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div><div>If a theme is using a library of functions, are those functions exempt from the "functions must be prefixed with the theme slug or an appropriate variant" rule?</div>
<div><br></div><div>The theme I'm reviewing (<a href="http://themes.svn.wordpress.org/evolve/1.2.6/library/" target="_blank">http://themes.svn.wordpress.org/evolve/1.2.6/library/</a>) has a library section that contains some prefixed functions while others aren't.</div>
<br><br>-----<div>Vicky Arulsingam</div><font color="#888888"><br>
</font><br></div></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></div></blockquote></div><br></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><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>