Fix for microsoft#50792: empty pre post scripts not rendered properly

egamma · egamma · commit 8b9b0d358256 · 2018-05-30T17:31:02.000+02:00
diff --git a/extensions/npm/src/tasks.ts b/extensions/npm/src/tasks.ts
@@ -58,7 +58,7 @@ function getPrePostScripts(scripts: any): Set<string> {
 		const script = keys[i];
 		const prepost = ['pre' + script, 'post' + script];
 		prepost.forEach(each => {
-			if (scripts[each]) {
+			if (scripts[each] !== undefined) {
 				prePostScripts.add(each);
 			}
 		});

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ function getPrePostScripts(scripts: any): Set<string> {`
`58`	`58`	`const script = keys[i];`
`59`	`59`	`const prepost = ['pre' + script, 'post' + script];`
`60`	`60`	`prepost.forEach(each => {`
`61`		`- if (scripts[each]) {`
	`61`	`+ if (scripts[each] !== undefined) {`
`62`	`62`	`prePostScripts.add(each);`
`63`	`63`	`}`
`64`	`64`	`});`