fix: skip handle empty url

webpack-contrib · Feb 15, 2018 · 9713ed6 · 9713ed6
1 parent 91e0316
commit 9713ed6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/lib/urls.js b/lib/urls.js
@@ -62,8 +62,8 @@ module.exports = function (css) {
 			.replace(/^'(.*)'$/, function(o, $1){ return $1; });
 
 		// already a full url? no change
-		if (/^(#|data:|http:\/\/|https:\/\/|file:\/\/\/)/i.test(unquotedOrigUrl)) {
-		  return fullMatch;
+		if (/^(#|data:|http:\/\/|https:\/\/|file:\/\/\/|\s*$)/i.test(unquotedOrigUrl)) {
+			return fullMatch;
 		}
 
 		// convert the url to a full url

diff --git a/test/fixUrls.test.js b/test/fixUrls.test.js
@@ -143,6 +143,17 @@ describe("fix urls tests", function() {
         assertUrl("body { background-image:url(#bg.jpg); }");
     });
 
+    // empty urls
+    it("Empty url should be skipped", function() {
+      assertUrl("body { background-image:url(); }");
+      assertUrl("body { background-image:url( ); }");
+      assertUrl("body { background-image:url(\n); }");
+      assertUrl("body { background-image:url(''); }");
+      assertUrl("body { background-image:url(' '); }");
+      assertUrl("body { background-image:url(\"\"); }");
+      assertUrl("body { background-image:url(\" \"); }");
+    });
+
     // rooted urls
     it("Rooted url", function() {
       assertUrl(