Hi,E l vie, 18-03-2011 a las 10:11 +0100, Systho escribió: > I've began the netbeans config according to conventions (because I think > it is a more important tool than the CodeSniffer Standard) but I have a > few questions : > > White spaces before parentheses : My point of view is to use that > whitespace to make a difference between control structure and methods. > > Therefore I would add a whitespace before each opening parenthesis of > control structure (if, elseif, for, foreach, while, catch, switch, > array) and not before the opening parenthesis of method (neither > definition nor call)
That's the exact contrary from the PEAR2 conventions, which are already tried and tested by a whole lot of people (it's version 2 :-)). I would recommend going the same way. > I do not agree with the wrapping policy I would personnaly wrap in > almost all cases when the line is longer than 120 chars. The only > exception is for strings which sometimes cannot be wrapped (but I > personnaly wrap them as much as keeping the meaning of the string is > possible). If wrapping, it should be from 80 chars (this should not be forced), mostly because this is the standard width of a terminal, and it makes the code easier to read when working on a server (I don't think UNIX servers allow you to resize your terminal). If wrapping from 120 chars, you'd better not wrap at all (given it doesn't follow any standard). Regards, Yannick > Here is a config transforming beforeFormatting.php into afterFormatting.php > > PS : this is my nbproject/project.properties file , if you just want to > add the formatter rules, just copy the lines related to formatting ;) > > Systho > > Le 18/03/2011 9:10, Stefaan Vanbillemont a écrit : > > Systho, > > > > I will try the CLI script today .. I keep you informed about the outcome. > > > > Kind Regards, > > > > Stefaan. > > > > > > Op 17/03/11 17:20, Philippe Van Eerdenbrugghe schreef: > >> Dear all, > >> > >> The new CLI installer is there. > >> > >> if you want to use it just go into your console and type : > >> > >> php script/chamilo.php install -c MyConfigFile > >> > >> or > >> > >> php script/chamilo.php install --configuration=MyConfigFile > >> > >> The format of the config file is the one you cand find in > >> install/php/lib/command_line_configuration.dist.php but you should > >> only override the values you want to change. > >> > >> > >> Systho > >> > >> _______________________________________________ > >> Dev mailing list > >> [email protected] > >> http://lists.chamilo.org/listinfo/dev > > > > _______________________________________________ > > Dev mailing list > > [email protected] > > http://lists.chamilo.org/listinfo/dev > _______________________________________________ > Dev mailing list > [email protected] > http://lists.chamilo.org/listinfo/dev _______________________________________________ Dev mailing list [email protected] http://lists.chamilo.org/listinfo/dev
