Skip to content

Commit

Permalink
Removed spConfig check from configurable-mixin since the double ajax …
Browse files Browse the repository at this point in the history
…call issue does not appear here - only with swatches
  • Loading branch information
peterjaap committed Jan 23, 2018
1 parent b347a61 commit 5d31505
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion etc/module.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?xml version="1.0"?>
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
<module name="Elgentos_LargeConfigProducts" setup_version="0.1.2">
<module name="Elgentos_LargeConfigProducts" setup_version="0.1.3">
</module>
</config>
5 changes: 1 addition & 4 deletions view/frontend/web/js/configurable-mixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@ define([
$.widget('mage.configurable', widget, {
// Load jsonConfig through AJAX call instead of in-line
_create: function () {
if (!_.isEmpty(this.options.spConfig)) {
return;
}
var that = this;
var productData = this._determineProductData();
$.ajax({
Expand Down Expand Up @@ -74,4 +71,4 @@ define([

return $.mage.configurable;
}
});
});

0 comments on commit 5d31505

Please sign in to comment.