Re: [Dolibarr-dev] Grosse incomprehension

[Franky Van Liedekerke]

Rodolphe Quiedeville wrote:
> Salut,
> 
> Dans la classe Produit htdocs/product.class.php j'ai trouvé le code
> suivant :
> 
> 
> function clone_price($fromId, $toId)
>  {
>     global $db;
> 
>     $db->begin();
> 
>     // les prix
>     $sql = "insert "    .MAIN_DB_PREFIX."product_price ("
>       . " fk_product, date_price, price, tva_tx, fk_user_author, envente )"
>       . " select ".$toId . ", date_price, price, tva_tx, fk_user_author,
> envente "
>       . " from ".MAIN_DB_PREFIX."product_price "
>       . " where fk_product = ". $fromId . ";" ;
>     if ( ! $db->query($sql ) ) {
>       $db->rollback();
>       return -1;
>     }
>     $db->commit();
>     return 1;
>   }
> 
> 
> 
> Je suis surpris de voir une utilisation de $db en tant que globale, cela
> vient à mon sens d'un problème de compréhension du code, je ne vais pas
> chercher qui à fait cette modif, mais il faut que l'on discute ici de ce
> problème de compréhension car on ne peut laisser du code comme cela dans
> Dolibarr.
> 
> A++
>       

Eh bien, le code est déjà dedans depuis plus d'un ans. Je pense que
celui qui a fait le commit n'a pas fait attention à ça. Remplacer $db
par $this->db et enlever le ligne global le corrige naturellement.
Btw: c'est aussi dans la fonction clone_fournisseurs.
Peut-être aussi ajouter dolibarr_print_error($this->db) quand ça marche pas?

Franky