Skip to content

Commit 8940f6d

Browse files
timseTheLarkInn
authored andcommitted
refactor of RequireIncludeDependency to es6 (webpack#3807)
1 parent d534603 commit 8940f6d

1 file changed

Lines changed: 24 additions & 16 deletions

File tree

lib/dependencies/RequireIncludeDependency.js

Lines changed: 24 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2,24 +2,32 @@
22
MIT License http://www.opensource.org/licenses/mit-license.php
33
Author Tobias Koppers @sokra
44
*/
5-
var ModuleDependency = require("./ModuleDependency");
5+
"use strict";
6+
const ModuleDependency = require("./ModuleDependency");
67

7-
function RequireIncludeDependency(request, range) {
8-
ModuleDependency.call(this, request);
9-
this.range = range;
8+
class RequireIncludeDependency extends ModuleDependency {
9+
constructor(request, range) {
10+
super(request);
11+
this.range = range;
12+
}
13+
14+
get type() {
15+
return "require.include";
16+
}
1017
}
11-
module.exports = RequireIncludeDependency;
1218

13-
RequireIncludeDependency.prototype = Object.create(ModuleDependency.prototype);
14-
RequireIncludeDependency.prototype.constructor = RequireIncludeDependency;
15-
RequireIncludeDependency.prototype.type = "require.include";
19+
RequireIncludeDependency.Template = class RequireIncludeDependencyTemplate {
20+
apply(dep, source, outputOptions, requestShortener) {
21+
const comment = this.getOptionalComment(outputOptions.pathinfo && dep.module, requestShortener.shorten(dep.request));
22+
source.replace(dep.range[0], dep.range[1] - 1, `undefined${comment}`);
23+
}
1624

17-
RequireIncludeDependency.Template = function RequireIncludeDependencyTemplate() {};
25+
getOptionalComment(shouldHaveComment, shortenedRequest) {
26+
if(shouldHaveComment) {
27+
return "";
28+
}
29+
return `/*! require.include ${shortenedRequest} */`;
30+
}
31+
}
1832

19-
RequireIncludeDependency.Template.prototype.apply = function(dep, source, outputOptions, requestShortener) {
20-
var comment = "";
21-
if(outputOptions.pathinfo && dep.module)
22-
comment = "/*! require.include " + requestShortener.shorten(dep.request) + " */";
23-
source.replace(dep.range[0], dep.range[1] - 1,
24-
"undefined" + comment);
25-
};
33+
module.exports = RequireIncludeDependency;

0 commit comments

Comments
 (0)