Skip to content

Refactor loader's code #448

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
alexander-akait merged 8 commits into from
Sep 16, 2019
Merged

Refactor loader's code #448

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7e6fbf2
refactor: remove excessive constants
goganchic Sep 13, 2019
49199bf
refactor: remove outdated comment
goganchic Sep 13, 2019
93724d0
refactor: move dependency adding function into loader
goganchic Sep 13, 2019
0929269
refactor: functions style unification
goganchic Sep 13, 2019
fc64716
refactor: rename variables names to reduce misunderstanding
goganchic Sep 13, 2019
53aeda8
refactor: remove unnecessary webpack comment
goganchic Sep 13, 2019
26fbea0
refactor: more clear names for local variables of addDependencies func
goganchic Sep 13, 2019
b134f00
refactor: update eslint-utils
goganchic Sep 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
refactor: remove excessive constants
  • Loading branch information
@goganchic
goganchic committed Sep 13, 2019
commit 7e6fbf2c26ec1db79d64c859863b4dbf8e15dafe
41 changes: 11 additions & 30 deletions package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

43 changes: 23 additions & 20 deletions src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,29 +148,32 @@ class MiniCssExtractPlugin {

apply(compiler) {
compiler.hooks.thisCompilation.tap(pluginName, (compilation) => {
compilation.hooks.normalModuleLoader.tap(pluginName, (lc, m) => {
const loaderContext = lc;
const module = m;

loaderContext[MODULE_TYPE] = (content) => {
if (!Array.isArray(content) && content != null) {
throw new Error(
`Exported value was not extracted as an array: ${JSON.stringify(
content
)}`
);
}
compilation.hooks.normalModuleLoader.tap(
pluginName,
(loaderContext, module) => {
// eslint-disable-next-line no-param-reassign
loaderContext[MODULE_TYPE] = (content) => {
if (!Array.isArray(content) && content != null) {
throw new Error(
`Exported value was not extracted as an array: ${JSON.stringify(
content
)}`
);
}

const identifierCountMap = new Map();
const identifierCountMap = new Map();

for (const line of content) {
const count = identifierCountMap.get(line.identifier) || 0;
for (const line of content) {
const count = identifierCountMap.get(line.identifier) || 0;

module.addDependency(new CssDependency(line, m.context, count));
identifierCountMap.set(line.identifier, count + 1);
}
};
});
module.addDependency(
new CssDependency(line, module.context, count)
);
identifierCountMap.set(line.identifier, count + 1);
}
};
}
);

compilation.dependencyFactories.set(
CssDependency,
Expand Down