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

[anthony . poiret]

Dernière chose, j'avais fait une petite erreur dans le patch contact.

Le bon patch en PJ.

(désolé pour le multiPost)


address@hidden a écrit :

> Hmmmm...
>
> Avec les PJ c'est quand même mieux...
>
>
>
> address@hidden a écrit :
>
> > Bonjour à tous,
> >
> > J'ai terminé les modifications pour le module d'import en 3.1   
> > alpha.   Cela a impliqué la modification de deux classes (command   
> > et compta)   pour lesquels j'ai produit un patch chacun que vous   
> > trouverez   ci-joint.
> >
> > Pour commande, il s'agit d'avantage d'une rustine que d'un patch -   
> >   la mise en commentaire des deux lignes fautives citées   
> > précédemment.   Je n'ai pas poussé d'avantage sur celle-ci,   
> > j'imagine que la   correction dont parlait Régis est bien plus   
> > élaborée que mes deux   ligne commentées, donc pas de réinvention   
> > de la roue pour moi là   dessus ;)
> >
> > Pour contact, j'ai simplement ajouté une fonction afin d'éviter une  
> >    démultiplication des contacts (magento stocke, comme je le  
> > disais    précédement, séparément les adresses de  
> > commande/facturation et  les   adresses de livraisons, pour  
> > lesquelles il créer une nouvelle  entrée   à chaque commande, même  
> > si l'adresse est identique). Comme  cette   fonction effectue un  
> > accès SQL, il me parraissait plus  juste de   l'intégrée dans le  
> > DAO concerné.
> >
> > Le troisième prérequis est le patch catégories, que j'ai déjà   
> > fourni   dans un thread parallèle.
> >
> >
> > En attendant un retour, merci de votre lecture,
> >
> > Anthony Poiret
> >
> >
> > address@hidden a écrit :
> >
> > > Une 3.1 alpha à priori, chargé il y'a une semaine environ.      
> > > D'ailleurs, en revérifiant, je viens de le trouver en double: il    
> > >  est  à la fois dans le fetch static et dans le fetch en méthode    
> > >  d'instance.
> > >
> > > Je viens de replace pour vérifier, le problème est effectivement   
> > > là    dans la dernière version (l 1445 et l 2865) dans les deux   
> > > fetchs,    static et d'instance.
> > >
> > >
> > > Régis Houssin <address@hidden> a écrit :
> > >
> > > > > 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/
> > > > ---------------------------------------------------------

patchContact31alpha
Description: Binary data