Merge pull request #1287 from olsonpm/fix-warn-on-namespace-conflict

fix incorrect namespace conflict warning
rollup · Mar 9, 2017 · ae54071 · ae54071
2 parents a6aa96e + 6c0048d
commit ae54071
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/Bundle.js b/src/Bundle.js
@@ -327,10 +327,11 @@ export default class Bundle {
 							if ( name in module.exportsAll ) {
 								this.warn({
 									code: 'NAMESPACE_CONFLICT',
-									message: `Conflicting namespaces: ${relativeId( module.id )} re-exports '${name}' from both ${relativeId( module.exportsAll[ name ] )} (will be ignored) and ${relativeId( exportAllModule.exportsAll[ name ] )}`
+									message: `Conflicting namespaces: ${relativeId( module.id )} re-exports '${name}' from both ${relativeId( module.exportsAll[ name ] )} and ${relativeId( exportAllModule.exportsAll[ name ] )} (will be ignored)`
 								});
+							} else {
+							  module.exportsAll[ name ] = exportAllModule.exportsAll[ name ];
 							}
-							module.exportsAll[ name ] = exportAllModule.exportsAll[ name ];
 						});
 					});
 					return module;

diff --git a/test/function/warn-on-namespace-conflict/_config.js b/test/function/warn-on-namespace-conflict/_config.js
@@ -3,7 +3,7 @@ module.exports = {
 	warnings: [
 		{
 			code: 'NAMESPACE_CONFLICT',
-			message: `Conflicting namespaces: main.js re-exports 'foo' from both foo.js (will be ignored) and deep.js`
+			message: `Conflicting namespaces: main.js re-exports 'foo' from both foo.js and deep.js (will be ignored)`
 		}
 	]
 };