Re: [Dolibarr-dev] Module d'import Magento->Dolibarr

[Régis Houssin]

> Pour commande, le problème est dans le fetch: à la fin de celui-ci, on
> appelle $this->fetch_lines(), fonction dont le code m'a quelque peu étonné:
>> while ($i < $num)
>> {
>> $objp = $this->db->fetch_object($result);
>> $line = new OrderLine($this->db);
>> $line->rowid            = $objp->rowid;
>> ...
>> $line->fk_parent_line    = $objp->fk_parent_line;
>>
>> $this->ref          = $objp->product_ref;    // TODO deprecated
>> $this->product_ref  = $objp->product_ref;
>> ...
>> }

ceci avait été corrigé, un copier collé malheureux, as-tu une version
cvs à jour ?


> $this étant ici la commande (à laquelle je comprend très bien qu'on
> assigne les lignes). Cependant, la ligne $this->ref [...] // TODO
> deprecated pose un réel problème, d'un part parce qu'on est dans un
> while, je ne vois donc pas l'utilité d'affecter des attributs d'une
> commande composée de multiples lignes, d'autre part parce que la ref de
> la commande n'est pas censé être celle du dernier produit de ses lignes
> (cette manip' purge tout simplement la première référence de commande
> updaté pour un type donné si la dernière ligne est un frais de port, et
> cause ensuite une erreur sql, la ref de la commande faisant partie avec
> le type d'une contrainte unique). J'ai pour ma part passé cette ligne en
> commentaire afin de supprimer le bug, cependant les autres assignations
> à $this me parraissent douteuses dans ce contexte.
> 



Cordialement,
-- 
Régis Houssin
---------------------------------------------------------
Cap-Networks
30, quai de Verdun
71700 Tournus
FRANCE
VoIP: +33 1 83 62 40 03
GSM: +33 6 33 02 07 97
Web: http://www.cap-networks.com/
Email: address@hidden

Dolibarr developer: address@hidden
Web Portal: http://www.dolibarr.fr/
SaaS offers: http://www.dolibox.fr/
Shop: http://www.dolistore.com/
Development platform: https://doliforge.org/
---------------------------------------------------------

regis_houssin.vcf
Description: Vcard