Re: com web/php: Removal of left menu class prefixes for method names since they're already indented within that class name: include/header.inc
include/shared-manual.inc
| From: | Hannes Magnusson | Date: | Wed, 09 Jan 2013 03:38:04 +0000 |
| Subject: | Re: com web/php: Removal of left menu class prefixes for method names since they're already indented within that class name: include/header.inc include/shared-manual.inc |
||
| References: | 1 2 3 | Groups: | php.webmaster |
| Request: | Send a blank email to php-webmaster+get-15258@lists.php.net to get a copy of this message | ||
On Wed, Jan 9, 2013 at 1:34 AM, Levi Morrison <morrison.levi@gmail.com> wrote:
> On Tue, Jan 8, 2013 at 10:29 AM, Hannes Magnusson
> <hannes.magnusson@gmail.com> wrote:
>> I beg you to stop introducing more and more coding styles.
>>
>> It is completely irrelevant how much readable you think it is.
>> Consistency throughout the project is of much greater value then
>> random coding styles.
>
> In principal I agree with what you said, but in this project I'm not
> sure what to consider the coding style to be. Is there some official
> style guide for the website project I don't know about?
We generally use the PEAR coding style, with already enough exceptions
with homebrewed styles.
At the very least keeping things consistent within the same file is a
requirement.
Naming conventions, escaping, php tags, content location.. these are
very important things to keep consistent.
-Hannes