Oui, c'est vrai.
Dans ce cas, cela peut justifier une présence. Il est alors possible de
documente cela dans la doc du développeur de module (sur le wiki).
A discuter au cas par cas pour ces cas.
Le 28/05/2012 23:10, Régis Houssin a écrit :
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 coeur et pourront servir
le coeur 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)"
<[email protected] <mailto:[email protected]>> 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
[email protected]
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
[email protected]
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
Eldy (Laurent Destailleur).
---------------------------------------------------------------
EMail:[email protected]
Web:http://www.destailleur.fr
Dolibarr (Project leader):http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal:[email protected]
AWStats (Author) :http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal:[email protected]
AWBot (Author) :http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) :http://cvschangelogb.sourceforge.net
_______________________________________________
Dolibarr-dev mailing list
[email protected] <mailto:[email protected]>
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
_______________________________________________
Dolibarr-dev mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
Eldy (Laurent Destailleur).
---------------------------------------------------------------
EMail: [email protected]
Web: http://www.destailleur.fr
Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: [email protected]
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: [email protected]
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
_______________________________________________
Dolibarr-dev mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
