Skip to content

Commit 60d89c1

Browse files
author
Anna Bukatar
committed
ACP2E-876: Google Recaptcha on Checkout not working
1 parent 84d43a6 commit 60d89c1

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

ReCaptchaCheckout/view/frontend/web/js/model/place-order-mixin.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,7 @@
88
define([
99
'jquery',
1010
'mage/utils/wrapper',
11-
'Magento_ReCaptchaWebapiUi/js/webapiReCaptchaRegistry',
12-
'Magento_Checkout/js/action/redirect-on-success'
11+
'Magento_ReCaptchaWebapiUi/js/webapiReCaptchaRegistry'
1312
], function ($, wrapper, recaptchaRegistry, redirectOnSuccessAction) {
1413
'use strict';
1514

@@ -25,7 +24,6 @@ define([
2524
payload.xReCaptchaValue = token;
2625
originalAction(serviceUrl, payload, messageContainer).done(function () {
2726
recaptchaDeferred.resolve.apply(recaptchaDeferred, arguments);
28-
redirectOnSuccessAction.execute();
2927
}).fail(function () {
3028
recaptchaDeferred.reject.apply(recaptchaDeferred, arguments);
3129
});

dev/tests/js/jasmine/tests/app/code/Magento/ReCaptchaCheckout/frontend/js/model/place-order-mixin.test.js

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,7 @@ define(['squire'
1111

1212
defaultContext = require.s.contexts._,
1313
mixin,
14-
registry,
15-
redirect;
14+
registry;
1615

1716
beforeEach(function (done) {
1817
window.checkoutConfig = {
@@ -21,12 +20,10 @@ define(['squire'
2120

2221
injector.require([
2322
'Magento_ReCaptchaCheckout/js/model/place-order-mixin',
24-
'Magento_ReCaptchaWebapiUi/js/webapiReCaptchaRegistry',
25-
'Magento_Checkout/js/action/redirect-on-success'
26-
], function (Mixin, Registry, Redirect) {
23+
'Magento_ReCaptchaWebapiUi/js/webapiReCaptchaRegistry'
24+
], function (Mixin, Registry) {
2725
mixin = Mixin;
2826
registry = Registry;
29-
redirect = Redirect;
3027
done();
3128
});
3229
});
@@ -97,10 +94,7 @@ define(['squire'
9794
registry._listeners[id] = func;
9895
};
9996
registry.removeListener = jasmine.createSpy();
100-
101-
redirect.execute = jasmine.createSpy();
10297
mixin()(action, serviceUrl, payload, messageContainer);
103-
expect(redirect.execute).toHaveBeenCalled();
10498
expect(registry.removeListener).toHaveBeenCalledWith(recaptchaId);
10599
});
106100
});

0 commit comments

Comments
 (0)