fix for first time rename and goToDef returning 'no result' (#15747)

* fix for first time rename and goToDef returning 'no result' * adding news
microsoft · Mar 23, 2021 · b13243b · b13243b
1 parent 888333e
commit b13243b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/news/2 Fixes/15727.md b/news/2 Fixes/15727.md
@@ -0,0 +1,2 @@
+Fix for Go to definition needs to be pressed twice
+(thanks [djplt](https://github.com/djplt))
diff --git a/src/client/activation/languageClientMiddleware.ts b/src/client/activation/languageClientMiddleware.ts
@@ -431,6 +431,7 @@ function captureTelemetryForLSPMethod(
                     };
                 }
                 sendTelemetryEvent(eventName, measures, properties);
+                return result;
             };
 
             let result = originalMethod.apply(this, args);
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Fix for Go to definition needs to be pressed twice
		(thanks [djplt](https://github.com/djplt))
-Original file line number
+Diff line change
@@ Expand Up / @@ -431,6 +431,7 @@ function captureTelemetryForLSPMethod( @@
                         };
                     }
                     sendTelemetryEvent(eventName, measures, properties);
+                    return result;
                 };
                 let result = originalMethod.apply(this, args);
@@ Expand Down @@