search

alma / alma-monthlypayments-magento2

Magento 2 Payment Gateway integration for Alma
Other
6 stars 9 forks source link

Prevent loop on reorderMethods() #3

Closed KLea closed 3 years ago

KLea commented 4 years ago

Problème remonté par un dev :

Le soucis vient du javascript sur la page de checkout, qui a l'air de déclenché de manière aléatoire (en fonction de la vitesse de chargement de la page ?), l'erreur suivante : 

at ko.computed.disposeWhenNodeIsRemoved (knockout.js:20)
at Function.evaluateImmediate_CallReadThenEndDependencyDetection (knockout.js:9)
at Function.evaluateImmediate_CallReadWithDependencyDetection (knockout.js:9)
at Function.evaluateImmediate (knockout.js:9)
at Function.evaluatePossiblyAsync (knockout.js:9)
at Function.notifySubscribers (knockout.js:6)
at Function.valueHasMutated (knockout.js:7)
at ko.subscription.reorderMethods [as callback] (alma_monthly_payments.js:1)
at Function.notifySubscribers (knockout.js:6)
at Function.valueHasMutated (knockout.js:7)
at ko.subscription.reorderMethods [as callback] (alma_monthly_payments.js:1)
at Function.notifySubscribers (knockout.js:6)
at Function.valueHasMutated (knockout.js:7)
at ko.subscription.reorderMethods [as callback] (alma_monthly_payments.js:1)
at Function.notifySubscribers (knockout.js:6)
at Function.valueHasMutated (knockout.js:7)

Lorsque l'erreur est déclenchée, le paiement Alma n'apparait pas dans la liste des choix. (l'erreurr vient d'une section de code qui cherche à réordonner les modes de paiement dans la liste de choix). J'ai changé la configuration du mode de paiement Alma pour le passer en position 2 (je l'avais mis à 10 à l'origine). Le problème n'a pas l'air de se reproduire.

Voir discussion sur #dev-support.

olance commented 3 years ago

👍 Tested (finally -_-) and approved! Included manually