support exporting classes

sokra · sokra · commit 2edd1ad5eb7f · 2016-06-04T15:06:10.000+02:00
fixed webpack#2523
diff --git a/lib/Parser.js b/lib/Parser.js
@@ -579,6 +579,8 @@ Parser.prototype.walkVariableDeclaration = function walkVariableDeclaration(stat
 };
 
 Parser.prototype.walkClassDeclaration = function walkClassDeclaration(statement) {
+	this.scope.renames["$" + statement.id.name] = undefined;
+	this.scope.definitions.push(statement.id.name);
 	this.walkClass(statement);
 };
 
diff --git a/lib/dependencies/HarmonyExportDependencyParserPlugin.js b/lib/dependencies/HarmonyExportDependencyParserPlugin.js
@@ -35,7 +35,7 @@ module.exports = AbstractPlugin.create({
 		this.state.module.strict = true;
 		return true;
 	},
-	"export declaration": function() {},
+	"export declaration": function(statement) {},
 	"export specifier": function(statement, id, name) {
 		var rename = this.scope.renames["$" + id];
 		var dep;
@@ -62,11 +62,12 @@ module.exports = AbstractPlugin.create({
 function isImmutableStatement(statement) {
 	if(statement.type === "FunctionDeclaration") return true;
 	if(statement.type === "ClassDeclaration") return true;
+	if(statement.type === "VariableDeclaration" && statement.kind === "const") return true;
 	return false;
 }
 
 function isHoistedStatement(statement) {
 	if(statement.type === "FunctionDeclaration") return true;
-	if(statement.type === "VariableDeclarators" && statement.kind === "var") return true;
+	if(statement.type === "VariableDeclaration" && statement.kind === "var") return true;
 	return false;
 }
diff --git a/test/cases/parsing/issue-2523/index.js b/test/cases/parsing/issue-2523/index.js
@@ -0,0 +1,9 @@
+import A from "./module";
+import { B } from "./module";
+import { c } from "./module";
+
+it("should allow to export a class", function() {
+	(typeof A).should.be.eql("function");
+	(typeof B).should.be.eql("function");
+	c.should.be.eql("c");
+})
diff --git a/test/cases/parsing/issue-2523/module.js b/test/cases/parsing/issue-2523/module.js
@@ -0,0 +1,4 @@
+export default class A {}
+export class B {}
+
+export const c = 'c';
diff --git a/test/cases/parsing/issue-2523/test.filter.js b/test/cases/parsing/issue-2523/test.filter.js
@@ -0,0 +1,5 @@
+var supportsES6 = require("../../../helpers/supportsES6");
+
+module.exports = function(config) {
+	return !config.minimize && supportsES6();
+};
