Skip to content

Commit

Permalink
MAGETWO-55115: [GitHub][IE]Checkout cart totals.js throws an error wh…
Browse files Browse the repository at this point in the history
…en estimating shipping
  • Loading branch information
omiroshnichenko committed Jul 20, 2016
1 parent bf0bf18 commit f3d7952
Showing 1 changed file with 23 additions and 26 deletions.
49 changes: 23 additions & 26 deletions app/code/Magento/Checkout/view/frontend/web/js/view/cart/totals.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,31 +2,28 @@
* Copyright © 2016 Magento. All rights reserved.
* See COPYING.txt for license details.
*/
define(
[
'uiComponent',
'Magento_Checkout/js/model/totals',
'Magento_Checkout/js/model/shipping-service'
],
function (Component, totalsService, shippingService) {
'use strict';
define([
'jquery',
'uiComponent',
'Magento_Checkout/js/model/totals',
'Magento_Checkout/js/model/shipping-service'
], function ($, Component, totalsService, shippingService) {
'use strict';

return Component.extend({
return Component.extend({
isLoading: totalsService.isLoading,

isLoading: totalsService.isLoading,

/**
* @override
*/
initialize: function () {
this._super();
totalsService.totals.subscribe(function () {
window.dispatchEvent(new Event('resize'));
});
shippingService.getShippingRates().subscribe(function () {
window.dispatchEvent(new Event('resize'));
});
}
});
}
);
/**
* @override
*/
initialize: function () {
this._super();
totalsService.totals.subscribe(function () {
$(window).trigger('resize');
});
shippingService.getShippingRates().subscribe(function () {
$(window).trigger('resize');
});
}
});
});

3 comments on commit f3d7952

@sshymko
Copy link

@sshymko sshymko commented on f3d7952 Dec 21, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@omiroshnichenko,
What release is the fix going to be included in?

@omiroshnichenko
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sshymko, it is going to be included in 2.1.4.

@LucScu
Copy link

@LucScu LucScu commented on f3d7952 Nov 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx for the fix

Please sign in to comment.