Manage use of vscode.window.showInputBox

Fixes #302 by only showing one input box at a time, deduplicating per server

Author: timleavitt

src/extension.ts

@@ -71,6 +71,7 @@ import {
   terminalWithDocker,
   notNull,
   currentFile,
+  InputBoxManager,
 } from "./utils";
 import { ObjectScriptDiagnosticProvider } from "./providers/ObjectScriptDiagnosticProvider";
 import { DocumentRangeFormattingEditProvider } from "./providers/DocumentRangeFormattingEditProvider";
@@ -262,21 +263,23 @@ export function checkConnection(clearCookies = false, uri?: vscode.Uri): void {
               disableConnection(configName);
             }
           } else {
-            vscode.window
-              .showInputBox({
+            InputBoxManager.showInputBox(
+              {
                 password: true,
                 placeHolder: `Not Authorized. Enter password to connect as user '${username}' to ${connInfo}`,
                 prompt: !api.externalServer ? "If no password is entered the connection will be disabled." : "",
                 ignoreFocusOut: true,
-              })
-              .then((password) => {
+              },
+              (password) => {
                 if (password) {
                   workspaceState.update(configName + ":password", password);
                   checkConnection(false, uri);
                 } else if (!api.externalServer) {
                   disableConnection(configName);
                 }
-              });
+              },
+              connInfo
+            );
           }
         }, 1000);
         message = "Not Authorized";

src/utils/index.ts

@@ -13,6 +13,9 @@ export function outputConsole(data: string[]): void {
   });
 }
 
+import { InputBoxManager } from "./inputBoxManager";
+export { InputBoxManager };
+
 // tslint:disable-next-line: interface-name
 export interface CurrentFile {
   name: string;

src/utils/inputBoxManager.ts

@@ -0,0 +1,59 @@
+import vscode = require("vscode");
+
+import { InputBoxOptions } from "vscode";
+
+// Used in situations where multiple input boxes may be displayed to ensure that we don't stomp on
+// the box that's already shown.
+export class InputBoxManager {
+  private static shown = false;
+  private static specs = new Array<InputBoxSpec>();
+  private static currentKey: string;
+
+  public static showInputBox(options: InputBoxOptions, callback: (answer: string) => void, key?: string): void {
+    const spec: InputBoxSpec = {
+      options: options,
+      callback: callback,
+    };
+    if (key) {
+      if (
+        InputBoxManager.currentKey === key ||
+        InputBoxManager.specs.find((spec) => {
+          return spec.key === key;
+        })
+      ) {
+        return;
+      }
+      spec.key = key;
+    }
+    this.specs.push(spec);
+    if (!this.shown) {
+      this.shown = true;
+      this.next();
+    }
+  }
+
+  private static next() {
+    if (this.specs.length === 0) {
+      this.shown = false;
+      this.currentKey = undefined;
+      return;
+    }
+    const spec = this.specs.shift();
+    this.currentKey = spec.key;
+    vscode.window.showInputBox(spec.options).then(
+      (result) => {
+        spec.callback(result);
+        InputBoxManager.next();
+      },
+      (ignored) => {
+        InputBoxManager.next();
+      }
+    );
+  }
+}
+
+interface InputBoxSpec {
+  key?: string;
+  options: InputBoxOptions;
+  callback: (answer: string) => void;
+}