Sorry for french language: Je suis d'accord pour commenter, Par contre si des modules externes utilisent des fonctionnalités communes avec un fort potentiel d'amélioration de dolibarr, il me semble plus judicieux de les intégrer dans le cœur et pourront servir le cœur par la suite en transposant ce qui a été fait sur les modules externes.
----------------------------------------- Régis Houssin Tél. +33633020797 http://www.dolibarr.fr http://www.dolibox.fr Le 28 mai 2012 à 22:58, "Laurent Destailleur (eldy)" <e...@destailleur.fr> a écrit : > The rule should be the opossite. > > Every code that is not required must not exists. If some code is here in > advance for a future feature, it's the job of the developer adding such > feature to add comment to justify why useless code is added to prevent it to > be removed and to explain to quality guys (guys that tracks dead code) why > such a code and why it must be kept. > When a code is useless it can be commented if we are not sure it is used by > dolibarr but if we are sure it is not (no reference to a function for > example), there is no reason to be kept. > > And no, the code of other is not the code of other: The code of other is the > code of nobody or everybody (We are an opensource project). > > > > Le 25/05/2012 22:00, Cyrille de Lambert a écrit : >> >> Bof !!! >> >> >> Le 25/05/2012 21:56, Régis Houssin a écrit : >>> >>> Règle N°1 : le code des autres c'est le code des autres ! ;-) >>> >>> j'aimerais bien qu'on demande et/ou qu'on mette en commentaire le code >>> au lieu de le supprimer. Car si un code est présent c'est qu'il devait >>> servir. >>> >>> Je me retrouve avec un module bancale et je n'arrive pas à retrouver le >>> pourquoi du comment... >>> >>> Bon week end ;-) >>> >>> >>> Cordialement, >> >> -- >> <mime-attachment.jpg> >> Cyrille de Lambert >> cyrille.delamb...@auguria.net >> 26 bis rue des olivettes >> 44300 NANTES Tél : +33 (0) 2 51 13 50 12 >> Mobile :+33 (0) 6 29 41 81 22 >> Fax : +33 (0) 2 51 13 52 88 >> http://www.auguria.net >> >> _______________________________________________ >> Dolibarr-dev mailing list >> Dolibarr-dev@nongnu.org >> https://lists.nongnu.org/mailman/listinfo/dolibarr-dev > > -- > Eldy (Laurent Destailleur). > --------------------------------------------------------------- > EMail: e...@destailleur.fr > Web: http://www.destailleur.fr > > Dolibarr (Project leader): http://www.dolibarr.org > To make a donation for Dolibarr project via Paypal: cont...@destailleur.fr > AWStats (Author) : http://awstats.sourceforge.net > To make a donation for AWStats project via Paypal: cont...@destailleur.fr > AWBot (Author) : http://awbot.sourceforge.net > CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net > _______________________________________________ > Dolibarr-dev mailing list > Dolibarr-dev@nongnu.org > https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
_______________________________________________ Dolibarr-dev mailing list Dolibarr-dev@nongnu.org https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
