Rodolphe Quiedeville a écrit :
Le 02.02.2007 16:38, Franky Van Liedekerke a ecrit :
2 patches:
- remplacer le code
$product = new Product($db);
$product->fetch($_GET["id"]);
par
$product = new Product($db, $_GET["id"]);
Naturellement ce n'est pas nécessaire, mais c'est plus facile à
comprendre: si tu veut adapter un produit, tu donne l'id avec, et si tu
veux créer un nouveau produit, tu donne rien comme id.
Bonjour,
Je ne suis pas d'accord avec cette modification, je n'en vois pas
l'intérêt et de plus dans certaine page cela créérait un bug.
Donc on n'applique pas le patch.
A++
Je suis également partisan de ne pas appliquer le patch. Mes motifs
diffèrent et sont les suivant:
A force de développer en objet (je suis plus javaiste que php), j'avoue
préférer l'écriture actuelle:
$product = new Product($db);
$product->fetch($_GET["id"]);
plutot que
$product = new Product($db, $_GET["id"]);
Je n'ai jamais aimé qu'on passe des paramètre structurant et qui plus est
optionnel dans un constructeur. Mais la c'est plus une question de gout et de
couleurs
--
Laurent Destailleur.
---------------------------------------------------------------
EMail: [EMAIL PROTECTED]
Web: http://www.destailleur.fr
IM: IRC=Eldy, Jabber=Eldy
AWStats (Author) : http://awstats.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
AWBot (Author) : http://awbot.sourceforge.net
Dolibarr (Contributor) : http://www.dolibarr.org
_______________________________________________
Dolibarr-dev mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
