use decodeURIComponent for getFilenameFromUrl final output (#173)

Author: Dima Paloskin

lib/GetFilenameFromUrl.js

@@ -3,7 +3,6 @@ var urlParse = require("url").parse;
 
 function getFilenameFromUrl(publicPath, outputPath, url) {
 	var filename;
-	url = decodeURIComponent(url);
 
 	// localPrefix is the folder our bundle should be in
 	var localPrefix = urlParse(publicPath || "/", false, true);
@@ -30,7 +29,7 @@ function getFilenameFromUrl(publicPath, outputPath, url) {
 		return false;
 	}
 	// and if not match, use outputPath as filename
-	return filename ? pathJoin(outputPath, filename) : outputPath;
+	return decodeURIComponent(filename ? pathJoin(outputPath, filename) : outputPath);
 
 }
 

test/GetFilenameFromUrl.test.js

@@ -89,7 +89,12 @@ describe("GetFilenameFromUrl", function() {
 				outputPath: "/",
 				publicPath: "//test.domain/protocol/relative/",
 				expected: "/sample.js"
-			}
+			}, {
+				url: "/pathname%20with%20spaces.js",
+				outputPath: "/",
+				publicPath: "/",
+				expected: "/pathname with spaces.js"
+			},
 		];
 		results.forEach(testUrl);
 	});