fix: source maps path on windows.

Author: evilebottnawi

lib/loader.js

@@ -20,8 +20,17 @@ module.exports = function(content, map) {
 	var resolve = createResolver(query.alias);
 
 	if(sourceMap) {
-		if (map && typeof map !== "string") {
-			map = JSON.stringify(map);
+		if (map) {
+			if (typeof map !== "string") {
+				map = JSON.stringify(map);
+			}
+
+			if (map.sources) {
+				map.sources = map.sources.map(function (source) {
+					return source.replace(/\\/g, '/');
+				});
+				map.sourceRoot = '';
+			}
 		}
 	} else {
 		// Some loaders (example `"postcss-loader": "1.x.x"`) always generates source map, we should remove it
@@ -30,8 +39,8 @@ module.exports = function(content, map) {
 
 	processCss(content, map, {
 		mode: moduleMode ? "local" : "global",
-		from: loaderUtils.getRemainingRequest(this),
-		to: loaderUtils.getCurrentRequest(this),
+		from: loaderUtils.getRemainingRequest(this).split("!").pop(),
+		to: loaderUtils.getCurrentRequest(this).split("!").pop(),
 		query: query,
 		minimize: this.minimize,
 		loaderContext: this,
@@ -105,11 +114,11 @@ module.exports = function(content, map) {
 			map = result.map;
 			if(map.sources) {
 				map.sources = map.sources.map(function(source) {
-					return source.split("!").pop();
+					return source.split("!").pop().replace(/\\/g, '/');
 				}, this);
 				map.sourceRoot = "";
 			}
-			map.file = map.file.split("!").pop();
+			map.file = map.file.split("!").pop().replace(/\\/g, '/');
 			map = JSON.stringify(map);
 			moduleJs = "exports.push([module.id, " + cssAsString + ", \"\", " + map + "]);";
 		} else {

test/sourceMapTest.js

@@ -10,27 +10,52 @@ describe("source maps", function() {
 	testWithMap("falsy: undefined map doesn't cause an error", ".class { a: b c d; }", undefined, [
 		[1, ".class { a: b c d; }", ""]
 	]);
-    testWithMap("should don't generate sourceMap when `sourceMap: false` and map exist",
+	testWithMap("should don't generate sourceMap when `sourceMap: false` and map exist",
 		".class { a: b c d; }",
-        {
-            file: 'test.css',
-            mappings: 'AAAA,SAAS,SAAS,EAAE',
-            names: [],
-            sourceRoot: '',
-            sources: [ '/folder/test.css' ],
-            sourcesContent: [ '.class { a: b c d; }' ],
-            version: 3
-        },
+		{
+			file: 'test.css',
+			mappings: 'AAAA,SAAS,SAAS,EAAE',
+			names: [],
+			sourceRoot: '',
+			sources: [ '/folder/test.css' ],
+			sourcesContent: [ '.class { a: b c d; }' ],
+			version: 3
+		},
 		[
 			[1, ".class { a: b c d; }", ""]
 		],
 		{
-            query: "?sourceMap=false"
+			sourceMap: false
+		}
+	);
+	testWithMap("should don't generate sourceMap when `sourceMap: true` and map exist",
+		".class { a: b c d; }",
+		{
+			file: 'test.css',
+			mappings: 'AAAA,SAAS,SAAS,EAAE',
+			names: [],
+			sourceRoot: '',
+			sources: [ '/folder/test.css' ],
+			sourcesContent: [ '.class { a: b c d; }' ],
+			version: 3
+		},
+		[
+			[1, ".class { a: b c d; }", "", {
+				file: 'test.css',
+				mappings: 'AAAA,SAAS,SAAS,EAAE',
+				names: [],
+				sourceRoot: '',
+				sources: [ '/folder/test.css' ],
+				sourcesContent: [ '.class { a: b c d; }' ],
+				version: 3
+			}]
+		],
+		{
+			sourceMap: true
 		}
 	);
 	testMap("generate sourceMap (1 loader)", ".class { a: b c d; }", undefined, {
 		loaders: [{request: "/path/css-loader"}],
-		options: { context: "/" },
 		resource: "/folder/test.css",
 		request: "/path/css-loader!/folder/test.css",
 		query: "?sourceMap"
@@ -47,7 +72,6 @@ describe("source maps", function() {
 	]);
 	testMap("generate sourceMap (1 loader, relative)", ".class { a: b c d; }", undefined, {
 		loaders: [{request: "/path/css-loader"}],
-		options: { context: "/other-folder/sub" },
 		resource: "/folder/test.css",
 		request: "/path/css-loader!/folder/test.css",
 		query: "?sourceMap"
@@ -64,7 +88,6 @@ describe("source maps", function() {
 	]);
 	testMap("generate sourceMap (1 loader, data url)", ".class { background-image: url(\"data:image/svg+xml;charset=utf-8,<svg xmlns='http://www.w3.org/2000/svg' viewBox='0 0 42 26' fill='%23007aff'><rect width='4' height='4'/><rect x='8' y='1' width='34' height='2'/><rect y='11' width='4' height='4'/><rect x='8' y='12' width='34' height='2'/><rect y='22' width='4' height='4'/><rect x='8' y='23' width='34' height='2'/></svg>\"); }", undefined, {
 		loaders: [{request: "/path/css-loader"}],
-		options: { context: "/" },
 		resource: "/folder/test.css",
 		request: "/path/css-loader!/folder/test.css",
 		query: "?sourceMap"
@@ -81,7 +104,6 @@ describe("source maps", function() {
 	]);
 	testMap("generate sourceMap (1 loader, encoded data url)", ".class { background-image: url(\"data:image/svg+xml;charset=utf-8,%3Csvg%20xmlns%3D%27http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg%27%20viewBox%3D%270%200%2042%2026%27%20fill%3D%27%23007aff%27%3E%3Crect%20width%3D%274%27%20height%3D%274%27%2F%3E%3Crect%20x%3D%278%27%20y%3D%271%27%20width%3D%2734%27%20height%3D%272%27%2F%3E%3Crect%20y%3D%2711%27%20width%3D%274%27%20height%3D%274%27%2F%3E%3Crect%20x%3D%278%27%20y%3D%2712%27%20width%3D%2734%27%20height%3D%272%27%2F%3E%3Crect%20y%3D%2722%27%20width%3D%274%27%20height%3D%274%27%2F%3E%3Crect%20x%3D%278%27%20y%3D%2723%27%20width%3D%2734%27%20height%3D%272%27%2F%3E%3C%2Fsvg%3E\"); }", undefined, {
 		loaders: [{request: "/path/css-loader"}],
-		options: { context: "/" },
 		resource: "/folder/test.css",
 		request: "/path/css-loader!/folder/test.css",
 		query: "?sourceMap"
@@ -98,7 +120,30 @@ describe("source maps", function() {
 	]);
 	testMap("generate sourceMap (2 loaders)", ".class { a: b c d; }", undefined, {
 		loaders: [{request: "/path/css-loader"}, {request: "/path/sass-loader"}],
-		options: { context: "/" },
+		resource: "/folder/test.scss",
+		request: "/path/css-loader!/path/sass-loader!/folder/test.scss",
+		query: "?sourceMap"
+	}, [
+		[1, ".class { a: b c d; }", "", {
+			file: 'test.scss',
+			mappings: 'AAAA,SAAS,SAAS,EAAE',
+			names: [],
+			sourceRoot: '',
+			sources: [ '/folder/test.scss' ],
+			sourcesContent: [ '.class { a: b c d; }' ],
+			version: 3
+		}]
+	]);
+	testMap("generate sourceMap (2 loaders) and map exist", ".class { a: b c d; }", {
+		file: 'test.scss',
+		mappings: 'AAAA,SAAS,SAAS,EAAE',
+		names: [],
+		sourceRoot: '',
+		sources: [ '/folder/test.scss' ],
+		sourcesContent: [ '.class { a: b c d; }' ],
+		version: 3
+	}, {
+		loaders: [{request: "/path/css-loader"}, {request: "/path/sass-loader"}],
 		resource: "/folder/test.scss",
 		request: "/path/css-loader!/path/sass-loader!/folder/test.scss",
 		query: "?sourceMap"
@@ -115,7 +160,6 @@ describe("source maps", function() {
 	]);
 	testMap("don't generate sourceMap (1 loader)", ".class { a: b c d; }", undefined, {
 		loaders: [{request: "/path/css-loader"}],
-		options: { context: "/" },
 		resource: "/folder/test.css",
 		request: "/path/css-loader!/folder/test.css",
 		query: "?sourceMap=false"