RE: [Dolibarr-dev] 2 patches - ou l'objectivité de l a validation des patches .... :p

[Vianney ASSOFI]

Hehe ... :)

Je ne suis pas développeur, mais bon ;) je crois comprendre que ça chambre
un chouilla :)

Allez allez, faut se détendre, cela dit, c'est vrai qu'il faut être cohérent
:), si Franky s'attaque au 'code' avec tant d'acharnement ca n'est que pour
le bien du projet... 
il n'a aucune "mauvaise" intention, même s'il peut arriver de faire des
erreurs en codant (mais que celui à qui ça n'est jamais arrivé jette la
première pierre ? :p )

L'ouverture d'un projet mène irrémédiablement à des façons de coder
différentes, mais n'y a-t-il pas plusieurs routes pour aller jusqu'à Rome ?

Je comprends que dans la situation actuelle, j'aurais été assez énervé et
limite j'aurais pensé que j'emmerde le monde si on refusait mes correctifs
sous prétexte que le code n'est pas joli et qu'après je m'aperçoive que le
reste du code est blindé des mêmes méthodes que l'on m'a reproché être 'pas
jolies ...'

Maintenant il faut appeler un chat un chat et un développeur hyperactif, un
développeur hyperactif. Et moi j'avoue apprécier les développeurs
hyperactifs ... parce qu'ils sont productifs... même s'il en faut d'autres
pour faire les gardes fous et "valider", mais il faut valider sans animosité
et surtout avec discernement et objectivité.

Ca n'était que mon petit sermon à 3 balles mais bon ;) ca détend.

Bon courage.

Vianney



-----Original Message-----
From: address@hidden
[mailto:address@hidden On Behalf Of
Franky Van Liedekerke
Sent: lundi 5 février 2007 10:14
To: Discussions sur le developpement de Dolibarr
Subject: Re: [Dolibarr-dev] 2 patches


Hi,

je peux bien accepter si vous n'aimer pas la manière

$product = new Product($db, $_GET["id"]);

mais alors il faut être consistant dans le code.
Si on fait un grep comme:

grep -ri "new product" *|grep "\,"

on voit beaucoup de lignes pareilles:

categories/categorie.class.php:                         $prod = new
Product ($this->db, $rec['fk_product']);
comm/propal.php:            $prod = new Product($db, $_POST['idprod']);
comm/propal.php:                        $prod = new Product($db,
$_POST['idprod']);
comm/propal.php:                                        $prod_arbo = new
Product($db, $value[1]);
commande/fiche.php:       $prod = new Product($db, $_POST['idprod']);
commande/fiche.php:                     $prod = new Product($db,
$_POST['idprod']);
commande/fiche.php:                                     $prod_arbo = new
Product($db, $value[1]);
commande/commande.class.php:                $prod = new
Product($this->db, $this->lignes[$i]->fk_product);
commande/commande.class.php:       $product = new
Product($this->db,$ligne->fk_product);
commande/commande.class.php:    $prod=new Product($this->db,$idproduct);
commande/commande.class.php:          $prod = new Product($this->db,
$idproduct);
commande/commande.class.php:            $product = new
Product($this->db,$ligne->fk_product);
commande/commande.class.php:            $product = new
Product($this->db,$LigneOld->produit_id);
commande/commande.class.php:       $product = new
Product($this->db,$ligne->fk_product);
commande/commande.class.php:    $product = new
Product($this->db,$this->fk_product);
compta/facture.php:            $prod = new Product($db, $_POST['idprod']);
contrat/contrat.class.php:                $prod = new Product($this->db,
$fk_product);
expedition/expedition.class.php:                            $product =
new Product($this->db,$obj->fk_product);
facture.class.php:                  $prod = new Product($this->db,
$_facrec->lignes[$i]->produit_id);
fourn/fournisseur.commande.class.php:       $prod = new
Product($this->db, $fk_product);
fourn/fournisseur.commande.class.php:   $prod = new Product($this->db,
$comclient->lignes[$i]->fk_product);
product/sousproduits/fiche.php:   $prod_arbo=new Product($db,$objp->rowid);
product/fiche.php:  $prod = new Product($db, $_GET['id']);
product.class.php:                 $product = new Product($this->db,
$value[1]);
propal.class.php:                               $prod = new
Product($this->db, $idproduct);

Dans les cas suivants ces lignes sont suivi d'un fetch:

categories/categorie.class.php:365:                             $prod =
new Product ($this->db, $rec['fk_product']);
categories/categorie.class.php-366-
$prod->fetch ($prod->id);

comm/propal.php:538:            $prod = new Product($db, $_POST['idprod']);
comm/propal.php-539-            $prod->fetch($_POST['idprod']);

commande/fiche.php:277:   $prod = new Product($db, $_POST['idprod']);
commande/fiche.php-278-   $prod->fetch($_POST['idprod']);

compta/facture.php:637:            $prod = new Product($db,
$_POST['idprod']);
compta/facture.php-638-            $prod->fetch($_POST['idprod']);

contrat/contrat.class.php:619:                $prod = new
Product($this->db, $fk_product);
contrat/contrat.class.php-620-                if
($prod->fetch($fk_product) > 0)

facture.class.php:249:              $prod = new Product($this->db,
$_facrec->lignes[$i]->produit_id);
facture.class.php-250-
$res=$prod->fetch($_facrec->lignes[$i]->produit_id);

fourn/fournisseur.commande.class.php:647:           $prod = new
Product($this->db, $fk_product);
fourn/fournisseur.commande.class.php-648-           if
($prod->fetch($fk_product) > 0)

fourn/fournisseur.commande.class.php:1049:      $prod = new
Product($this->db, $comclient->lignes[$i]->fk_product);
fourn/fournisseur.commande.class.php-1050-      if
($prod->fetch($comclient->lignes[$i]->fk_product) > 0)

product/fiche.php:279:  $prod = new Product($db, $_GET['id']);
product/fiche.php-280-  $result=$prod->fetch($_GET['id']);


Alors, pouvez-vous me dire quelle manière est la bonne?


Franky


Franky Van Liedekerke wrote:
> Rodolphe Quiedeville wrote:
>> 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++
>>
>>
> 
> Bonjour,
> 
> tu peut me dire dans quelle page le bug serait si on applique cette
> patch? Je veux bien comprendre ce que je fais de mal là.
> Et le 2ième patch?
> 
> Franky
> 
> 
> _______________________________________________
> Dolibarr-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
> 
> 


_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev