- Issue created by @hodev
- Status changed to Needs review
almost 2 years ago 7:13pm 21 February 2023 Thank you! It is working now!
How about to add this fix to production?
- 🇮🇱Israel jsacksick
What about the attached patch? Usually recommended to do strict type checking.
- 🇮🇱Israel jsacksick
If somebody confirms that the patch from #6 also fixes the issue, I'll go ahead and commit it.
- Status changed to Needs work
almost 2 years ago 8:18am 8 March 2023 - 🇺🇦Ukraine taraskorpach Lutsk 🇺🇦
The patch from #6 just avoids the error, but does not solve the problem. As far as I understand, arrow functions do not bind their own this, instead, they inherit the one from the parent scope, so instead of the expected address field, we get a global "this" object. The easiest solution is to replace the arrow function with "function (i, item)" in the "each" loop. Someone will add a patch with this suggestion, or I will do it myself later.
- First commit to issue fork.
- @tbkot opened merge request.
- Status changed to Needs review
over 1 year ago 3:49pm 23 March 2023 - Status changed to RTBC
over 1 year ago 8:21am 24 March 2023 - 🇺🇦Ukraine taraskorpach Lutsk 🇺🇦
It looks good. I tested locally and the payment checkout was successful.
-
jsacksick →
committed 320e4ddb on 8.x-1.x authored by
tBKoT →
Issue #3343285 by tBKoT, _pratik_, jsacksick: jQuery error on checkout.
-
jsacksick →
committed 320e4ddb on 8.x-1.x authored by
tBKoT →
- Status changed to Fixed
over 1 year ago 1:02pm 29 March 2023 - 🇺🇸United States drumm NY, US
(Updating to resolve Drupal.org issue indexing issue, please disregard.)
Automatically closed - issue fixed for 2 weeks with no activity.