Merge pull request microsoft#11893 from Microsoft/truthy_enum

Andy · web-flow · commit f307948af01b · 2016-10-27T09:03:16.000-07:00
Fix bug: We want to test for existence of the enum value, not whether it's non-zero
diff --git a/src/compiler/program.ts b/src/compiler/program.ts
@@ -333,7 +333,7 @@ namespace ts {
         if (host.resolveModuleNames) {
             resolveModuleNamesWorker = (moduleNames, containingFile) => host.resolveModuleNames(moduleNames, containingFile).map(resolved => {
                 // An older host may have omitted extension, in which case we should infer it from the file extension of resolvedFileName.
-                if (!resolved || resolved.extension) {
+                if (!resolved || resolved.extension !== undefined) {
                     return resolved;
                 }
                 resolved = clone(resolved);

Original file line number	Diff line number	Diff line change
`@@ -333,7 +333,7 @@ namespace ts {`
`333`	`333`	`if (host.resolveModuleNames) {`
`334`	`334`	`resolveModuleNamesWorker = (moduleNames, containingFile) => host.resolveModuleNames(moduleNames, containingFile).map(resolved => {`
`335`	`335`	`// An older host may have omitted extension, in which case we should infer it from the file extension of resolvedFileName.`
`336`		`- if (!resolved \|\| resolved.extension) {`
	`336`	`+ if (!resolved \|\| resolved.extension !== undefined) {`
`337`	`337`	`return resolved;`
`338`	`338`	`}`
`339`	`339`	`resolved = clone(resolved);`