quadra-informatique / CyberMut-Magento

Crédit Mutuel / CIC Community Payment Module for Magento 1.x (1.5+)
http://www.magentocommerce.com/magento-connect/cybermut-extension-2.html
4 stars 11 forks source link

Une version adaptée DSP2 est-elle prévue ? #25

Open zuiko opened 5 years ago

zuiko commented 5 years ago

Cet été les détenteurs d'un TPE virtuel Crédit Mutuel ont reçu un e-mail commençant ainsi : "Madame, Monsieur Les normes techniques de la DSP2 introduisent de nouvelles règles en matière de sécurité des transactions en ligne. Des dispositions qui vont impacter les transactions effectuées sur votre site marchand et qui devront être prises en compte avant le 14 septembre 2019."

La documentation téléchargeable détaille de lourdes modifications de l'interface.

Est-il prévu une version de cette extension en Magento 1 adaptée 3DSecure DSP2 ? J'ai commencé de coder le transfert d'informations client pour l'interface aller mais je n'ai pas touché à l'évolution du calcul du sceau MAC ni de l'interface retour. Si d'autres ont des compétences et sont intéressés, ce serait bien... Monetico_Paiement_documentation_migration_3DSv2_1.0.pdf Monetico_Paiement_documentation_technique_v2.0.pdf

digitalpianism commented 4 years ago

@zuiko bonjour, j'ai un soucis similaire je bosse actuellement sur la version DSP2, peut être pourrions nous échanger à ce propos?

zuiko commented 4 years ago

Bonjour, bien sûr je suis intéressé 👍 j'avoue que comme l'ancienne interface a continué de fonctionner j'ai un peu laissé ça de côté mais il serait en effet plus sain de s'y remettre activement. Je me sentais un peu seul... Pour info, je suis français et j'habite Nantes.

digitalpianism commented 4 years ago

@zuiko j'ai bossé dessus aujourd'hui et j'ai sorti une version qui a l'air de fonctionner correctement. J'ai pas de support du multi paiement en revanche.

zuiko commented 4 years ago

@digitalpianism personnellement, je n'utilise pas le multi paiement... Ta proposition m'intéresse, si tu veux que je vois chez moi ce que ça donne, je veux bien la récupérer et tester. Vois-tu la chose possible et comment ?

digitalpianism commented 4 years ago

Carrément , envoie ton e-mail je t’envoie le package

Raphael Petrini

Le 26 mars 2020 à 23:31, zuiko notifications@github.com a écrit :

 @digitalpianism personnellement, je n'utilise pas le multi paiement... Ta proposition m'intéresse, si tu veux que je vois chez moi ce que ça donne, je veux bien la récupérer et tester. Vois-tu la chose possible et comment ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

zuiko commented 4 years ago

zuiko(at)free.fr Yannick Chotard

Carrément , envoie ton e-mail je t’envoie le package Raphael Petrini Le 26 mars 2020 à 23:31, zuiko @.***> a écrit :  @digitalpianism personnellement, je n'utilise pas le multi paiement... Ta proposition m'intéresse, si tu veux que je vois chez moi ce que ça donne, je veux bien la récupérer et tester. Vois-tu la chose possible et comment ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

digitalpianism commented 4 years ago

@zuiko en fait j'vais procéder différemment, j'pense que j'vais faire un fork de mon côté, qui sera accessible publiquement plutot que de partager des trucs par mail en privé. Ca permettra à ceux qui ont besoin de cloner le repo sur leurs installs et de contribuer vu que Quadra a plus l'air de toucher au module. J'te tiens au jus dès que j'ai fais ca

digitalpianism commented 4 years ago

@zuiko j'ai fais un fork et une PR: https://github.com/quadra-informatique/CyberMut-Magento/pull/27

N'hésites pas a me faire ton retour si tu trouves des trucs qui clochent.

zuiko commented 4 years ago

C'est une très bonne idée de passer par un fork, je vais regarder çà...

zuiko commented 4 years ago

Est-ce que tu pars de la version de Irosset (voir dans Pull requests) ? (qui semblait avoir fait des modifs intéressantes)...

digitalpianism commented 4 years ago

@zuiko pas du tout j'ai juste forké le module de Quadra.

digitalpianism commented 4 years ago

@zuiko ca a pas l'air de marcher totalement correctement mon truc

digitalpianism commented 4 years ago

@zuiko cest good normalement tu peux tester si tu veux j'ai fais un commit a l'instant pr corriger mon dernier soucis.

zuiko commented 4 years ago

@digitalpianism bonjour, en fait je suis ennuyé car je ne suis pas familiarisé avec les mécanismes de Github et je ne sais pas récupérer ton code du fork assemblé en package avec l'extension de base à moins de me faire une intégration laborieuse des deltas des fichiers touchés. Jusqu'à présent j'ai fait mes développements en solo ou participé à des tests de modules mais sans mettre les mains directement dans Github. ça doit pourtant être très simple mais si tu as une indication pour me mettre sur la voie sans que je prenne des risque de mettre la zone dans ce que tu as fait, je serais intéressé. Merci pour ta compréhension. Yannick

digitalpianism commented 4 years ago

@zuiko le plus pratique c'est tout simplement que tu télécharges mon fork de la même façon que tu as probablement téléchargé la version de Quadra içi: https://github.com/digitalpianism/CyberMut-Magento

Ensuite tu fais une sauvegarde du module de Quadra en renommant app/code/community/Quadra et tu envoies ma version sur ton site. Comme ça si mes modifs conviennent pas tu pourras toujours supprimer le dossier Quadra et renommer l'ancien dossier pour récupérer l'ancienne version.

Mais j'ai testé en test et en prod en faisant plusieurs commandes, ca fonctionne correctement de mon côté. N'hésites pas en tout cas

zuiko commented 4 years ago

Merci pour le lien, c'est parfait, j'ai récupéré le package, ensuite je sais faire. C'est juste que je n'avais pas compris par où passer pour accéder au code complet du fork.

JepKoha commented 3 years ago

Bonjour tous,

Merci @digitalpianism pour ce fork du module de Quadra. comme @zuiko çà me sauve la mise ;-) mis en place sans problème dans un vieux magento 1.9.4.5... Un seul mini-problème dans l'envoi des données à l'interface de paiement, avec une erreur dans le JSon du paramètre "contexte_commande"

Tous les champs requis sont présents. Format invalide pour le(s) champ(s) : /contexte_commande/billing/stateOrProvince, /contexte_commande/shipping/stateOrProvince

En fait ces deux champs "stateOrProvince" ont un format précis, ISO etc, et ne s'appliquent qu'aux US et au CA J'envoyais un truc du genre "FR_75". Pafff !

J'ai modifié /app/code/community/Quadra/Cybermut/Model/Abstract.php à getContext($order) lors de la création des données $data['billing']['stateOrProvince'] et $data['shipping']['stateOrProvince'] en testant d'abord si $countryId est égal à 'US' ou 'CA' Vers les lignes 272 et 317 (çà c'est du commit !-)

Pour le $data['billing']['stateOrProvince'] (l272)

// ici erreur Monetico // optionnel sauf CA ou USA if ($regionId = $billingAddress->getRegionId()) { $region = Mage::getModel('directory/region')->load($billingAddress->getRegionId()); $countryId = $region->getCountryId(); $code = $region->getCode(); if ($countryId && $region && ($countryId == 'US' || $countryId == 'CA')) { $data['billing']['stateOrProvince'] = $countryId . "_" . $code; } }

et pour le $data['shipping']['stateOrProvince'] (l317)

// ici erreur Monetico // optionnel sauf CA ou USA if ($regionId = $shippingAddress->getRegionId()) { $region = Mage::getModel('directory/region')->load($shippingAddress->getRegionId()); $countryId = $region->getCountryId(); $code = $region->getCode(); if ($countryId && $region && ($countryId == 'US' || $countryId == 'CA')) { $data['shipping']['stateOrProvince'] = $countryId . "_" . $code; } }

Voilà si çà peut aider Merci encore ! JP