Corrections module So Colissimo

[gootsy]

Il semblerait qu'il y ait un problème avec le module so colissimo; avec le transporteur activé, on obtient un message de type "hack attempt" avec un message "address->address1 is empty" et ce même en sélectionnant un autre transporteur (en l'occurence, mon transporteur habituel)... En désactivant le transporteur "so colissimo", tout rentre dans l'ordre... j'ai tourné un moment pour voir que ça venait de là, je voyais pas pourquoi il interviendrait dans le traitement alors que j'avais choisi un autre transporteur... Si ça peut éviter à d'autres de s'arracher les cheveux...

[Vincent Augagneur]

Bonjour,

Le bug viens d'être reglé.
Il suffit de rajouter apres la ligne 436 du fichier socolissimo.php le bout de code suivant :

Avant :

    public function hooknewOrder($params)
    {
        global $cookie;    
         if ($params['objOrder']->module != $this->name) 
                 return;

[foxtoto2000]

est ce que la version téléchargeable a ce bug de corrigé?
perso, j'ai systématiquement l'erreur 002 et pourton mon ID Fo est bien communiqué et il est valide
Je pense qu'il y a une erreur dans ton message Vincent car je ne vois pas ce qu'il faut ajouter

[foxtoto2000]

y a t il un moyen de voir les infos envoyées à la poste car je ne vois vraiment pas pourquoi cela ne fonctionne pas.
est ce que quelqu'un a reussi a le faire fonctionner car j'ai essayé sur un autre site et j'ai le meme probleme

[Vincent Augagneur]

Bonjour,

Je viens d'avoir une personne de la Poste au téléphone, pour ce qui est de l'inscription au service So Colissimo, vous pouvez normalement des maintenant vous inscrire en appelant le 36 34.

Pour ce qui est du problème de retour d'erreur 002 ID FO incorrecte : il faut dans le module remplacer dans la configuration les 2 paramètres suivants :
Adresse FO : https://ws.colissimo...fo/storeCall.do
Adresse de verification : http://ws.colissimo....supervision.jsp

[foxtoto2000]

merci pour cette information: cela fonctionne pour le module maintenant.
Par contre, pour le reste...
Prestashop n'est pas encore adapté à ce module:
les BL n'intègrent pas les adresses de So colissimo
Expeditor ne gère pas non plus So Colissimo

[Broceliande]

From 1283515638:

Bonjour,

Le bug viens d'être reglé.
Il suffit de rajouter apres la ligne 436 du fichier socolissimo.php le bout de code suivant :

Avant :

    public function hooknewOrder($params)
    {
        global $cookie;    
         if ($params['objOrder']->module != $this->name) 
                 return;

Bonjour,
J'ai justement posté au sujet de ce correctif qui n'en est pas un , ici :
http://www.prestasho...d/73715/#318360

[Broceliande]

Pour être plus précis , voici ce que j'ai mis , moi :

public function hooknewOrder($params)
    {
        global $cookie;
        
        $order = $params['order'];
        $soCarrierId = intval(Configuration::get('SOCOLISSIMO_CARRIER_ID'));
        if($order->id_carrier == $soCarrierId)
        {

Dans le détail , par rapport au correctif proposé et intégré en 1.1 :

$params['objOrder']

est null , objOrder est la variable utilisée par le hook orderConfirmation ...
pour avoir l'objet order c'est donc $params['order']; dans ce hook ci

$params[‘order’]->module 

est le module choisi pour le paiement et ne peut pas être comparé à

$this->name

, d'ou :

if($order->id_carrier == $soCarrierId)

qui me semble - t - il correspond à ce que l'on souhaite

[Broceliande]

Bon on nous demande de regrouper les posts ici mais ça ne vit pas bien plus.

Alors pour ceux qui utilisent déja socolissimo en prod et rencontrent quelques bizarreries, voici un correctif de plus :

Nous avons eu plusieurs contacts clients nous indiquant ne pas pouvoir commander, suite à une erreur (en retour d'interface so colissimo).
Dans le même moment , notre taux de réalisation des paniers a particulièrement chuté .

Il s'avère que si le client a entré dans son numéro de portable, lors de son inscription sur votre shop, des espaces pour séparer les chiffre , ou des points , des tirets encore... et bien tout bêtement c'est une erreur à la clé .

Beaucoup ne comprennent pas et tous ne remontent pas l'erreur.

Voici un correctif , qui bien sûr peut être amélioré , mais qui a le mérite d'améliorer grandement le fonctionnement chez mes clients :

            $bads = array(" ", ".", "-", ",", ";", "+", "/", "\", "+", "(", ")");
            $params['address']->phone_mobile = str_replace($bads,"",$params['address']->phone_mobile);

Et donc ici même la mise à jour (qui inclue aussi mes précédents correctifs ) , en attendant des nouvelles de la team.

Edit :

Fichier re-mis à jour . Cette fois le module inclut les améliorations d'enveloppebulle (cf. autres posts)

Il eut été idiot de ne pas regrouper le tout . J'espère que tu passeras par là enveloppebulle , et pourras tester l'intégration de tes modifs.

En référence à des posts dans d'autres thread , j'ajoute que l'adresse de livraison so colissimo est belle et bien affichée dans la facture, sans modifs du pdf . Merci de vos retours

Attached File(s)

•  socolissimo.zip (1.42K)
  Number of downloads: 102

[Patric Codron]

Le développeur en charge de ce module a presque fini d'intégrer les modifications et améliorations proposées. :)

[Broceliande]

Super nouvelle , c'est important de danser tous sur le même pied :)

[quality22]

Super ! on a hate de voir ca !!!

[Gregory Roussac]

Bonjour,

Sur http://www.prestasho...m/fr/downloads/ on voit compatible 1.2 en bas mais dans le module socolissimo on voit une constante _MYSQL_ENGINE_ qui n'est dispo qu'à partir de la 1.4. Le module utilise aussi la table ps_carrier_group aussi qui n'est disponible qu'a partir de la 1.3 pour "Autorisations de groupe:" sur un transporteur.

Pour corriger pour ces versions

dans settings.inc.php rajouter

define('_MYSQL_ENGINE_', 'MyISAM');

Rajouter la table

CREATE TABLE IF NOT EXISTS ps_carrier_group (

    id_carrier int(10) unsigned NOT NULL,
    id_group int(10) unsigned NOT NULL,
    UNIQUE KEY id_carrier (id_carrier,id_group)

) ENGINE=MyISAM DEFAULT CHARSET=utf8;

ensuite il faut y insérer pour chaque identifiant de transporteur le groupe par défaut au moins (1).

Par exemple

    INSERT INTO ps_carrier_group (

id_carrier ,
id_group
)
VALUES ('1', '1')
, ('2', '1'), ('3', '1'
);

Cordialement,

PS: Et il manque de jolis intval() sur certains SELECT

[Julien Breux]

Peux-t'on demander à Prestashop un module final compatible avec la version 1.3.x ou faut-il le re-développer de façon fonctionnel ?

[Gregory Roussac]

Pour le define c'est juste une ligne à rajouter en tout cas.

[Broceliande]

Quote

_MYSQL_ENGINE_ ....

Très juste , si l'on re télécharge la 1.1 ,

Sauf que , la version sur laquelle je me suis appuyé , et que j'ai corrigé est bien une 1.1 ... et ne contient pas cette constante mais MyISAM .
Je précise que bien sûr cette version provient http://www.prestashop.com/downloads

Y aurait-il plusieurs versions 1.1 ?

[Julien Breux]

From 1287683558:

Quote

_MYSQL_ENGINE_ ....

Très juste , si l'on re télécharge la 1.1 ,

Sauf que , la version sur laquelle je me suis appuyé , et que j'ai corrigé est bien une 1.1 ... et ne contient pas cette constante mais MyISAM .
Je précise que bien sûr cette version provient http://www.prestashop.com/downloads

Y aurait-il plusieurs versions 1.1 ?

Effectivement ^^

[quality22]

rolala cest quoi se patacaisse !

Sil vous plait la team, pouvez vous nous dire CLAIREMENT :
- la version officielle en téléchargement sur presta peut tourner sous quelle version prestashop : 1.1, 1.2, 1.3, 1.4 ?
- ou en êtes vous de la résolution des multiples bugs du module qui le rendent inutilisable ?

parce que la on va pas sen sortir

Merci !

[Broceliande]

Je peux en tout cas assurer que la version corrigée , postée ici :

http://www.prestasho...ewreply/319396/

est pleinement compatible 1.3 (je ne sais pas pour la 1.2.5 , mais je ne vois pas en quoi elle ne le serait pas , sur ce que je sais du code )

Elle résout pas mal de bugs, en tout cas tous ceux rencontrés en production, y compris l'enregistrement de l'adresse choisie, bureau de poste ou point relais , affichée donc dans la facture .

Même si désormais on nous a dit que le responsable du module se penchait sur l'intégration de ces correctifs , ainsi que ceux de enveloppebulle , je trouverais intéressant que ces derniers soient testés.
Cette version me semble assez stable, mais rien ne dit qu'il n'y a pas de bugs restant . Dans la foulée , on pourrait tenter de corriger ceux là aussi et finir par avoir un vrai module stable, en attendant celui de prestashop.

Enfin tout est dit , n'hésitez pas à vous servir et tester cette version.

[Patric Codron]

Il semble qu'il y ait eu un petit problème dans les développements récents.
Une nouvelle version du module 1.2 sera normalement disponible début de semaine prochaine et intègrera les modifications retenues sur le forum et aussi la compatibilité avec Expeditor.