Make the status bar behave more reasonable when there are no platform configurations or debug configurations.

If there are no debug configurations, don't show the debug configuration, debug, or run status items.

If there are no platform configurations, don't show the platform configuration item.

Author: sterin

src/expand.ts

@@ -50,11 +50,14 @@ export class Expander
 
             const val = this.definitions.get(s);
 
-            for( let k of this.definitions.keys() )
+            if( val !== null )
             {
-                if( val.indexOf(k) !== -1 )
+                for( let k of this.definitions.keys() )
                 {
-                    visit(k);
+                    if( val.indexOf(k) !== -1 )
+                    {
+                        visit(k);
+                    }
                 }
             }
 

src/extension.ts

@@ -201,11 +201,17 @@ class Extension
     private getState<T>(
         key: string, 
         legal:(val:T)=>boolean, 
-        otherwise:(key:string)=>T)
+        otherwise:(key:string)=>T,
+        valid:()=>boolean=()=>true)
     {
+        if( !valid() )
+        {
+            return null;
+        }
+
         let val = this.context.workspaceState.get<T>(key);
 
-        if( ! val || !legal(val) )
+        if( !val || !legal(val) )
         {
             val = otherwise(key);
             this.context.workspaceState.update(key, val);
@@ -233,9 +239,11 @@ class Extension
     {
         return this.getState<string>(
             "debugConfig", 
-            (val:string) => true, 
-            (key:string) => this.config.debugConfigurations.length > 0 ? this.config.debugConfigurations[0].name : null
+            (val:string) => this.config.debugConfigurations.some( (t) => t.name==val ), 
+            (key:string) => this.config.debugConfigurations[0].name,
+            () => this.config.debugConfigurations.length > 0
         );
+        
     }
 
     set debugConfigName(config: string)

src/status.ts

@@ -46,7 +46,7 @@ export class StatusBar
         this.killStatusItem.text = "$(x)";
     }
 
-    public forallItems(f: (item:vscode.StatusBarItem)=>void)
+    public forAllItems(f: (item:vscode.StatusBarItem)=>void)
     {
         f(this.buildStatusItem);
         f(this.buildConfigStatusItem);
@@ -58,24 +58,29 @@ export class StatusBar
 
     public dispose()
     {
-        this.forallItems(i => i.dispose());
-    }
-
-    public show()
-    {
-        this.forallItems(i => i.show());
+        this.forAllItems(i => i.dispose());
     }
 
     public hide()
     {
-        this.forallItems(i => i.hide());
+        this.forAllItems(i => i.hide());
     }
 
     public update(buildConfig: string, debugConfig:string)
     {
         this.buildConfigStatusItem.text = buildConfig;
         this.debugConfigStatusItem.text = debugConfig;
 
-        this.show();
+        this.forAllItems( (i) => 
+        {
+            if( debugConfig === null && ( i===this.debugStatusItem || i===this.runStatusItem || i===this.debugConfigStatusItem ) )
+            {
+                i.hide();
+            }
+            else
+            {
+                i.show();
+            }
+        });
     }
 }