Hi,

Le 10/12/2009 13:56, Paul Poulain a écrit :
> Hello koha-dev,
>
> with koha 3.2 almost knocking on the door, and chris C. ideas for 3.4, I
> think it's time to harden&  strengthen our coding guidelines, that are
> really too light. The guidelines are here :
> http://wiki.koha.org/doku.php?id=en:development:codingguidelines
>
> I suggest we create a page "newcodingguidelines", with everyone/anyone
> ideas, that could be discussed through a chat session.
>
> is it a good suggestion ?
>    
I'm looking the koha code every day, the most difficult to read the code 
is : Identation
My propositions are:
   * don't use tabs, indentation should be 4 white spaces
   * Identation must be valid to accept a patch

Then, as you know, urls are hard coded in templates, they should be a 
baseurl everywhere like:
CGIBASE + "/members/memberentry.pl"

I propose tu use everytime $myhash->{mykey} instead of $$myhash{mykey}.

Deny patches with SQL in .pl scripts, accept "SQL" only in C4, and 
encourage the use of SQLHelper.

New ideas will come later :)

bests,

-- 
Nahuel ANGELINETTI

_______________________________________________
Koha-devel mailing list
Koha-devel@lists.koha.org
http://lists.koha.org/mailman/listinfo/koha-devel

Reply via email to