Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
zuiko opened this issue Sep 12, 2019 · 17 comments
Open

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

zuiko opened this issue Sep 12, 2019 · 17 comments

Comments

@zuiko
Copy link

zuiko commented Sep 12, 2019

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
Copy link

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

@zuiko
Copy link
Author

zuiko commented Mar 26, 2020

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
Copy link

@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
Copy link
Author

zuiko commented Mar 26, 2020

@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
Copy link

digitalpianism commented Mar 26, 2020 via email

@zuiko
Copy link
Author

zuiko commented Mar 26, 2020

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
Copy link

@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
Copy link

@zuiko j'ai fais un fork et une PR: #27

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

@zuiko
Copy link
Author

zuiko commented Mar 27, 2020

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

@zuiko
Copy link
Author

zuiko commented Mar 27, 2020

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

@digitalpianism
Copy link

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

@digitalpianism
Copy link

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

@digitalpianism
Copy link

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

@zuiko
Copy link
Author

zuiko commented Mar 30, 2020

@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
Copy link

@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
Copy link
Author

zuiko commented Mar 30, 2020

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
Copy link

JepKoha commented Dec 9, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants