Trims whitespace in import urls for unquoted imports

Fixes #781
webpack-contrib · Oct 10, 2018 · a57f8ab · a57f8ab
1 parent 67b2f20
commit a57f8ab
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/lib/loader.js b/lib/loader.js
@@ -49,7 +49,11 @@ module.exports = function(content, map) {
 		var importUrlPrefix = getImportPrefix(this, query);
 
 		var alreadyImported = {};
-		var importJs = result.importItems.filter(function(imp) {
+		var importJs = result.importItems.map(function(imp) {
+			// fixes #781 when importing `url(filename.css )`
+			imp.url = imp.url.trim();
+			return imp;
+		}).filter(function(imp) {
 			if(!imp.mediaQuery) {
 				if(alreadyImported[imp.url])
 					return false;

diff --git a/test/importTest.js b/test/importTest.js
@@ -13,6 +13,12 @@ describe("import", function() {
 	], "", {
 		"./test.css": [[2, ".test{a: b}", ""]]
 	});
+	test("import", "@import url(test.css );\n.class { a: b c d; }", [
+		[2, ".test{a: b}", ""],
+		[1, ".class { a: b c d; }", ""]
+	], "", {
+		"./test.css": [[2, ".test{a: b}", ""]]
+	});
 	test("import camelcase", "@IMPORT url(test.css);\n.class { a: b c d; }", [
 		[2, ".test{a: b}", ""],
 		[1, ".class { a: b c d; }", ""]