pr feedback updates

Author: David Kline

HoloLensCommander/HoloLensCommander/Controls/DeviceMonitorControlViewModel.cs

@@ -632,7 +632,7 @@ private void Device_HeartbeatReceived(DeviceMonitor sender)
             // Did we recover from a heartbeat loss?
             if (this.StatusMessage == HeartbeatLostMessage)
             {
-                this.StatusMessage = string.Empty;
+                this.ClearStatusMessage();
             }
 
             // Handle whether or not we were previously selected

HoloLensCommander/HoloLensCommander/Dialogs/SettingsDialog.xaml.cs

@@ -36,7 +36,7 @@ private void ContentDialog_OkClick(
                 // Return the data
                 ((SettingsDialogViewModel)this.DataContext).UpdateUserData(this.appSettings);
                 this.appSettings.SettingsUpdated = true;
-                this.appSettings.StatusMessage = "";
+                this.appSettings.StatusMessage = string.Empty;
             }
             catch(Exception e)
             {

HoloLensCommander/HoloLensCommander/MainWindowViewModel.cs

@@ -199,7 +199,7 @@ private void RegisterCommands()
             this.ClearStatusMessageCommand = new Command(
                 (parameter) =>
                 {
-                    this.StatusMessage = string.Empty;
+                    this.ClearStatusMessage();
                 });
 
             this.CloseAllAppsCommand = new Command(
@@ -435,20 +435,29 @@ await this.ConnectToDeviceAsync(
             this.UninstallAppCommand = new Command(
                 (parameter) =>
                 {
-                    this.UninstallApp();
+                    try
+                    {
+                        this.UninstallApp();
+                    }
+                    catch (Exception e)
+                    {
+                        this.StatusMessage = string.Format(
+                            "Failed to uninstall an app on one or more devices ({0})",
+                            e.Message);
+                    }
                 });
 
             this.UninstallAllAppsCommand = new Command(
                 (parameter) =>
                 {
                     try
                     {
-                        this.UninstallApp();
+                        this.UninstallAllApps();
                     }
                     catch (Exception e)
                     {
                         this.StatusMessage = string.Format(
-                            "Failed to uninstall an app on one or more devices ({0})",
+                            "Failed to uninstall all apps on one or more devices ({0})",
                             e.Message);
                     }
                 });

HoloLensCommander/HoloLensCommander/MainWindowViewModelCommands.cs

@@ -60,6 +60,14 @@ private async Task ClearDeviceStatus()
         public ICommand ClearStatusMessageCommand
         { get; private set; }
 
+        /// <summary>
+        /// Implementation of the clear status message command.
+        /// </summary>
+        private void ClearStatusMessage()
+        {
+            this.StatusMessage = string.Empty;
+        }
+        
         /// <summary>
         /// Command used to close all applications on the selected devices.
         /// </summary>
@@ -471,7 +479,7 @@ public ICommand SaveSessionFileCommand
         /// </summary>
         private async Task SaveSessionFile()
         {
-            this.StatusMessage = "";
+            this.ClearStatusMessage();
 
             try
             {

HoloLensCommander/HoloLensCommander/Utilities/Settings.cs

@@ -102,13 +102,13 @@ public Settings() :
         /// <param name="interval">Seconds between heartbeat checks.</param>
         public Settings(float interval) :  
             this(
-            false, 
+            false,          // autoReconnect
             interval,
-            false,
-            false,
-            false,
-            string.Empty,
-            string.Empty)
+            false,          // expandCredentials
+            false,          // expandNetworkSettigns
+            false,          // useInstalledCertificate
+            string.Empty,   // defaultSsid
+            string.Empty)   // defaultNetworkKey
         { }
 
         /// <summary>