Don't collect open-file telemetry on the syntax server

amcasey · amcasey · commit 0b89460c18ee · 2018-05-16T13:40:24.000-07:00
It's throwing an exception (since there's no program from which to
retrieve the SourceFile) and there's no reason to fix it since the
same files are open in the semantic server.
diff --git a/src/server/editorServices.ts b/src/server/editorServices.ts
@@ -2123,7 +2123,7 @@ namespace ts.server {
         }
 
         private telemetryOnOpenFile(scriptInfo: ScriptInfo): void {
-            if (!this.eventHandler || !scriptInfo.isJavaScript() || !addToSeen(this.allJsFilesForOpenFileTelemetry, scriptInfo.path)) {
+            if (this.syntaxOnly || !this.eventHandler || !scriptInfo.isJavaScript() || !addToSeen(this.allJsFilesForOpenFileTelemetry, scriptInfo.path)) {
                 return;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -2123,7 +2123,7 @@ namespace ts.server {`
`2123`	`2123`	`}`
`2124`	`2124`
`2125`	`2125`	`private telemetryOnOpenFile(scriptInfo: ScriptInfo): void {`
`2126`		`- if (!this.eventHandler \|\| !scriptInfo.isJavaScript() \|\| !addToSeen(this.allJsFilesForOpenFileTelemetry, scriptInfo.path)) {`
	`2126`	`+ if (this.syntaxOnly \|\| !this.eventHandler \|\| !scriptInfo.isJavaScript() \|\| !addToSeen(this.allJsFilesForOpenFileTelemetry, scriptInfo.path)) {`
`2127`	`2127`	`return;`
`2128`	`2128`	`}`
`2129`	`2129`