Skip to content

Commit 8bc1718

Browse files
authored
Merge pull request #4 from sdinteractive/hotfix/jquery-mixins
Hotfix/jquery mixins
2 parents 6cd03c7 + f64d4d1 commit 8bc1718

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

RequireJsLoaderPlugin.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,9 @@ function gatherRequireJsImports(modules) {
1010
for (var module of modules) {
1111
// If the requirejs-loader was used, then we need to wrap and import this module.
1212
// TODO: Clean up this check.
13-
if (module.request && module.request.indexOf('requirejs-loader') !== -1) {
13+
if (module.request && String(module.request).indexOf('jquery.js') !== -1) {
14+
needsImport.push('mixins!' + module.rawRequest);
15+
} else if (module.request && module.request.indexOf('requirejs-loader') !== -1) {
1416
needsImport.push(module.rawRequest);
1517
}
1618
}

index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@ module.exports.pitch = function(remainingRequest) {
1111

1212
// Route through window.require.
1313
// TODO: We use rawRequest to grab the original request (including text! or etc.)
14-
const jsonName = JSON.stringify(this._module.rawRequest);
14+
const prefix = (this._module.rawRequest == 'jquery') ? 'mixins!' : '';
15+
const jsonName = JSON.stringify(prefix + this._module.rawRequest);
1516
return `module.exports = window.require(${jsonName});`;
1617
};
1718

0 commit comments

Comments
 (0)