I agree.<div><br></div><div>As far as the Theme Review Team is concerned, the use of quotes versus double quotes needs to be consistent - and not even necessarily consistent throughout the file, but internally consistent within each string, so that no errors are generated.</div>
<div><br></div><div>Thus far, the Theme Review Team has shied away from "best practice" considerations strictly related to coding conventions. Maybe if we get to the point where the vast majority (I'd say, 90% or better) of submitted Themes pass the current Review Guidelines on first pass, then and only then would I be in favor of looking at things like the WP coding standard for PHP and CSS.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Thu, Mar 3, 2011 at 8:21 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;">
This really is not a Theme review related question as it stands ... <br><br>... either method (as is) would be about the same as the other; if you were working with internationalization it may affect the best-practice case, but again, not really a specific Theme review related item.<br>
<br>I'm sure the WordPress support forums would have a lot more input on this ...<br><br><div class="gmail_quote"><div class="im">On Thu, Mar 3, 2011 at 9:13 AM, Satish Gandham <span dir="ltr"><<a href="mailto:satish.iitg@gmail.com" target="_blank">satish.iitg@gmail.com</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><div><div></div><div class="h5">Which is better of the following two<br><br><?php <br>echo "This is a string with variable $test_variable"<br>
?><br><br><?php<br>echo 'This is another variant of the above statement '.$test_variable <br>
?><br><br><br>Regards<br><font color="#888888">Satish Gandham<br>
</font><br></div></div><div class="im">_______________________________________________<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>
<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>