hoist exports declarations too

fixes #3972
webpack · Jan 16, 2017 · a44694a · a44694a
1 parent 9796696
commit a44694a
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 1 deletion.
diff --git a/lib/Parser.js b/lib/Parser.js
@@ -421,6 +421,8 @@ Parser.prototype.walkStatements = function walkStatements(statements) {
 Parser.prototype.isHoistedStatement = function isHoistedStatement(statement) {
 	switch(statement.type) {
 		case "ImportDeclaration":
+		case "ExportAllDeclaration":
+		case "ExportNamedDeclaration":
 			return true;
 	}
 	return false;
@@ -621,7 +623,7 @@ Parser.prototype.walkExportDefaultDeclaration = function walkExportDefaultDeclar
 Parser.prototype.walkExportAllDeclaration = function walkExportAllDeclaration(statement) {
 	var source = statement.source.value;
 	this.applyPluginsBailResult("export import", statement, source);
-	this.applyPluginsBailResult("export import specifier", statement, source, null, null);
+	this.applyPluginsBailResult("export import specifier", statement, source, null, null, 0);
 };
 
 Parser.prototype.walkVariableDeclaration = function walkVariableDeclaration(statement) {

diff --git a/test/cases/parsing/harmony-commonjs/e.js b/test/cases/parsing/harmony-commonjs/e.js
@@ -0,0 +1,2 @@
+exports.a = "a";
+exports.b = "b";
diff --git a/test/cases/parsing/harmony-commonjs/index.js b/test/cases/parsing/harmony-commonjs/index.js
@@ -20,3 +20,11 @@ it("should double reexport from non-harmony modules correctly", function() {
 	y.should.be.eql("y");
 	x.should.be.eql("x");
 });
+
+
+import { a, b } from "./reexport"
+
+it("should be possible to reexport a module with unknown exports", function() {
+	a.should.be.eql("a");
+	b.should.be.eql("b");
+});
diff --git a/test/cases/parsing/harmony-commonjs/reexport.js b/test/cases/parsing/harmony-commonjs/reexport.js
@@ -0,0 +1,2 @@
+export * from "./e";
+import "./e";