Bugfix in Dart Analysis Server completion support, don't call complet…

…ion_setSubscriptions() in startServer()

close #860

GitOrigin-RevId: 07ae189d451b2360ddf73aaaff7c89972f5d5b14

Dart/src/com/jetbrains/lang/dart/analyzer/DartAnalysisServerService.java

@@ -300,6 +300,11 @@ public void computedSearchResults(String searchId, List<SearchResult> results, b
  @Override
  public void serverConnected(@Nullable String version) {
  myServerVersion = version != null ? version : "";
+ // completion_setSubscriptions() are handled here instead of in startServer() as the server version isn't known until this
+ // serverConnected() call.
+ if (myServer != null && !shouldUseCompletion2()) {
+ myServer.completion_setSubscriptions(List.of(CompletionService.AVAILABLE_SUGGESTION_SETS));
+ }
  }
 
  @Override
@@ -2088,10 +2093,6 @@ else if (!dasSnapshotFile.canRead()) {
  startedServer.start();
  server_setSubscriptions(startedServer);
 
- if (!shouldUseCompletion2()) {
- startedServer.completion_setSubscriptions(List.of(CompletionService.AVAILABLE_SUGGESTION_SETS));
- }
-
  if (!myInitializationOnServerStartupDone) {
  myInitializationOnServerStartupDone = true;