Points d'accroche nécessaires ? Fichiers js.stripe.com chargés sur toutes les pages

[vz777]

Bonjour, i can translate if needed but it's not so easy for me ;)

je me permet de tagguer les derniers contributeurs au module car je ne crois pas qu'il y avait ce souci dans les précedentes versions @lopes-vincent @zawaze @AnthonyMeedle

Avec la dernière version 2.0.4., je me retrouve avec les fichiers javascript de stripe chargés sur toutes les pages, ceci ne me semble pas trop normal, à moins qu'il y ait une bonne raison ? Normalement les fichiers stripe ne devraient être chargés que lors de la sélection de stripe sur order/invoice ?

Ceci est du au fait que dans les points d'accroche du module, il y en avait 3 d'activés dont 2 je ne vois pas bien pourquoi : main.head-bottom main.after-javascript-include

J'ai donc désactivé ces deux points d'accroches mais je ne sais pas si c'est une bonne idée. Je n'ai donc plus que order-invoice.after-javascript-include comme point d'accroche.

Pour illustrer le propos, voir copie écran en PJ.

Merci d'avance

[vz777]

Bon déja j'ai l'impression qu'il serait possible de modifier la ligne 11 du fichier config.xml : <tag name="hook.event_listener" event="main.after-javascript-include" type="front" method="includeStripeJsV3"/> par : <tag name="hook.event_listener" event="order-invoice.after-javascript-include" type="front" method="includeStripeJsV3"/>

Il faudrait sans doute modifier Hook/StripePaymentHook.php

Pour main-head-bottom, je vois pas.

[vz777]

Bon je suis désolé de vous avoir dérangés, mais je n'arrive même pas à reproduire mon problème ... ;) Sans doute encore une histoire de cache.

Pb résolu

[EDIT] non résolu : je crois comprendre dans la doc stripe que stripe veut son javascript sur tout le site ?

[lopes-vincent]

Bonjour,

Désolé pour le temps de réponse. Je ne sais plus exactement mais oui il me semble que c'est stripe qui nous force a le mettre sur toutes les pages car ils veulent qu'on leur envoie le panier au plus tôt et qu'on le mette à jour a chaque modification.

[AnthonyMeedle]

Bonjour,

oui de mémoire c'était recommandé à l'époque par stripe de mettre ce bout de code partout, je ne sais plus la raison, vérifier le parcour client, si c'est bien un humain, évaluer les risques de fraude des robot ... ça peut très bien fonctionner juste sur la page de paiement mais ça fait remonter mais d'information.

le module stripe n'est pas autant complet que ce que stripe propose. En effet stripe propose une gestion client, panier avec article, abonnement, multi compte pour dispatcher l'argent reçu via le site. Et je crois stripe à depuis le module fait deux mise à jours code qui permet peut-être de simplifier certaines partie. Pour le moment le défaut du module est qu'il crée un peu vite la demande de paiement. Et remplie stripe de paiement incomplet, dans un sens c'est intéressant de voir les panier abandonnés mais d'un autre côté ça pollu un peu la liste des paiements.

Le modules est à modifier/améliorer aussi bien au niveau code technique que visuel mais ça dépend du site sur lequel on veut l'installer.

Les versions précédente n'avait en effet pas le même fonctionnement mais la mise à jours stripe était obligatoire par rapport aux dernières normes de sécurité, ça été fait au dernier moment, ce qui n'a pas vraiment laissé le temps de peaufiner comme il se doit le module.

Est-ce que au final le module fonctionne ? ou est-ce qu'il y a des bugs ?

Bien cordialement,

Anthony CHEVRIER

Responsable technique et développementMobile : *06 37 38 47 25*Site web https://www.meedle.fr/ | Facebook https://www.facebook.com/agencemeedle | Twitter https://twitter.com/meedleweb | Linkedin https://www.linkedin.com/in/anthony-chevrier-41b30936

Le lun. 31 août 2020 à 09:39, Vincent Lopes-Vicente < notifications@github.com> a écrit :

Bonjour,

Désolé pour le temps de réponse. Je ne sais plus exactement mais oui il me semble que c'est stripe qui nous force a le mettre sur toutes les pages car ils veulent qu'on leur envoie le panier au plus tôt et qu'on le mette à jour a chaque modification.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/thelia-modules/StripePayment/issues/18#issuecomment-683619473, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA3U52OTQIZHMYTJ3ON756DSDNHRLANCNFSM4QN5EBUQ .

[vz777]

Bonjour, pas de pb pour le temps de réponse, chacun fait comme il peut ;)

j'ai été incapable de le faire fonctionner tout seul, il a fallu l'intervention d'un vrai dev pour que ce soit ok ;) J'ai pas eu le temps de mettre un message pour le forum pour expliquer la solution.

Dans les logs j'avais l'erreur stripe-paiement.html cannot be found ou un truc comme ça. Voila ce qui à été fait pour que ça fonctionne : Duplication de ../local/modules/StripePayment/templates/frontOffice/default et renommer le dossier en "lenomdemontemplate"

Pour ce qui est des volontés de stripe, je ne vois pas pourquoi charger 6 scripts sur toutes les pages qui ralentissent bien le site vont améliorer la sécurité. Mais peut être que je me trompe. Dites moi si c'est le cas. J'ai donc désactivé ces deux hooks:

main.head-bottom qui n'a l'air d'appeler que du css qui concerne stripe element uniquement

main.after-javascript-include et j'ai appelé dans order/invoice au début du {block name="javascript-initialization"} La aussi, je suis peut être dans l'erreur, faut me dire.

Du reste, personnellement le module me convient en l'état, je n'ai pas besoin des dernières fonctionnalités de stripe, du genre paiement en 18,5 fois. ;)

Le truc qui m'a fait bizzare au début, c'est le passage d'une sorte de popup/modal en une page externe. Et au final, ce n'est que mieux ça évite au changer de cliquer 50 fois sur actualiser