OPEN-DSI / ecommerceng_woosync

WooSync, synchronize Woocommerce and Dolibarr
GNU General Public License v3.0
72 stars 46 forks source link

woosync n'utilise pas le champ ugs de woocommerce pour la référence #31

Open Cyrix126 opened 3 years ago

Cyrix126 commented 3 years ago

Bonjour, J'ai mis en place ce module sur dolibarr. La syncronisation des produits de Woocommerce à Dolibarr rencontre une erreur:

Des erreurs se sont produites lors de la synchronisation des produits Erreur lors de la synchronisation du produit (ID distant: 42477) La référence du produit doit etre renseigné sur votre produit distant

J'ai bien vérifié que le champ UGS dans woocommerce est bien remplis.

La synchronisation de Dolibarr à Woocommerce s'effectue sans erreur mais le produit créé sur woocommerce ne possède pas de valeur dans le champ UGS.

Je suspecte Woosync de ne pas utiliser correctement le champ UGS de Woocommerce en tant que référence.

Voici les versions utilisés: woosync: 4.0.50 dolibarr: 12.0.3 woocommerce: 4.7.0 wordpress: 5.5.3

Sur Woocomerce j'ai activé l'ancienne API Rest car je n'arrivais pas à effectuer la connexion sans ça.

mkdgs commented 3 years ago

Il y a un mélange entre l'ancienne et la nouvelle API, les deux sont requises. Effectivement, il y a un soucis également avec les références. J'ai fait des correctifs assez nombreux mais je n'ai pas le temps de faire des PR propres ici, cf #30

Ma branche fonctionne correctement dans le sens Woo vers Dolibarr avec uniquement l'API V2 de woocommerce, dans l'autre sens il y a sans doute pas mal de chose à debugger.

Cyrix126 commented 3 years ago

Il y a un mélange entre l'ancienne et la nouvelle API, les deux sont requises. Effectivement, il y a un soucis également avec les références. J'ai fait des correctifs assez nombreux mais je n'ai pas le temps de faire des PR propres ici, cf #30

Ma branche fonctionne correctement dans le sens Woo vers Dolibarr avec uniquement l'API V2 de woocommerce, dans l'autre sens il y a sans doute pas mal de chose à debugger.

Merci, j'ai testé votre branche et en effet pas de problème woo ver dolibarr, par contre pas de syncronisation effectué de doli ver woo. Je vais essayer d'approfondir ça lorsque j'en aurai le temps.

mkdgs commented 3 years ago

Comme je l'ai indiqué c'est testé dans le sens Woocommerce vers Dolibarr uniquement. Je n'ai pas besoin d'envoyer des données ( et il me semble trop dangereux d'utiliser cette fonctionnalité en l'état ).

swiffysmith commented 1 year ago

Il y a un mélange entre l'ancienne et la nouvelle API, les deux sont requises. Effectivement, il y a un soucis également avec les références. J'ai fait des correctifs assez nombreux mais je n'ai pas le temps de faire des PR propres ici, cf #30 Ma branche fonctionne correctement dans le sens Woo vers Dolibarr avec uniquement l'API V2 de woocommerce, dans l'autre sens il y a sans doute pas mal de chose à debugger.

Merci, j'ai testé votre branche et en effet pas de problème woo ver dolibarr, par contre pas de syncronisation effectué de doli ver woo. Je vais essayer d'approfondir ça lorsque j'en aurai le temps.

Idem, une synchro dans les 2 sens m'intéresse. Je vais creuser dans l'année. N'hésitez pas si vous voulez travailler en binôme / trinôme.