[theme-reviewers] usage of single quotes and double quotes in PHP

[Edward Caissie]

Done

On Thu, Mar 3, 2011 at 10:35 AM, Lance Mericle <lance at mericles.com> wrote:

> Can I get removed from this list?
>
> --
> *
> Lance W. Mericle
> Owner
>
> www.mericles.com
> t: @mericles
> 828.329.5359
> *
>
> From: Chip Bennett <chip at chipbennett.net>
> Reply-To: <theme-reviewers at lists.wordpress.org>
> Date: Thu, 3 Mar 2011 09:33:19 -0600
> To: <theme-reviewers at lists.wordpress.org>
> Subject: Re: [theme-reviewers] usage of single quotes and double quotes in
> PHP
>
> I agree.
>
> 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.
>
> 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.
>
> Chip
>
> On Thu, Mar 3, 2011 at 8:21 AM, Edward Caissie <edward.caissie at gmail.com>wrote:
>
>> This really is not a Theme review related question as it stands ...
>>
>> ... 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.
>>
>> I'm sure the WordPress support forums would have a lot more input on this
>> ...
>>
>> On Thu, Mar 3, 2011 at 9:13 AM, Satish Gandham <satish.iitg at gmail.com>wrote:
>>
>>> Which is better of the following two
>>>
>>> <?php
>>> echo "This is a string with variable $test_variable"
>>> ?>
>>>
>>> <?php
>>> echo 'This is another variant of the above statement '.$test_variable
>>> ?>
>>>
>>>
>>> Regards
>>> Satish Gandham
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
> _______________________________________________ theme-reviewers mailing
> list theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110303/b76b446c/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/png
Size: 4511 bytes
Desc: not available
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110303/b76b446c/attachment-0001.png>