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