Merge branch 'master' of https://twinetfs.visualstudio.com/_git/Tango

3 files changed, 94 insertions, 41 deletions

diff --git a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ConnectedMachineViewVM.cs b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ConnectedMachineViewVM.cs
index 35a4b0e9d..5467c53a0 100644
--- a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ConnectedMachineViewVM.cs
+++ b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ConnectedMachineViewVM.cs
@@ -20,6 +20,7 @@ namespace Tango.MachineStudio.UI.ViewModels
             Disconnect,
             UploadHardwareConfig,
             Reset,
+            TurnOffHeaters
         }
 
         private IStudioApplicationManager _applicationManager;
@@ -45,31 +46,22 @@ namespace Tango.MachineStudio.UI.ViewModels
 
         public RelayCommand ResetCommand { get; set; }
 
+        public RelayCommand TurnOffHeatersCommand { get; set; }
+
         public ConnectedMachineViewVM(IStudioApplicationManager application, IDiagnosticsFrameProvider frameProvider)
         {
             ApplicationManager = application;
-            DisconnectCommand = new RelayCommand(Disconnect);
-            UploadHardwareConfigurationCommand = new RelayCommand(UploadHardwareConfiguration);
-            ResetCommand = new RelayCommand(Reset);
+            DisconnectCommand = new RelayCommand(() => AcceptResult(ConnectedMachineVMResult.Disconnect));
+            UploadHardwareConfigurationCommand = new RelayCommand(() => AcceptResult(ConnectedMachineVMResult.UploadHardwareConfig));
+            ResetCommand = new RelayCommand(() => AcceptResult(ConnectedMachineVMResult.Reset));
+            TurnOffHeatersCommand = new RelayCommand(() => AcceptResult(ConnectedMachineVMResult.TurnOffHeaters));
 
             DiagnosticsFrameProvider = frameProvider;
         }
 
-        private void Reset()
-        {
-            Result = ConnectedMachineVMResult.Reset;
-            Accept();
-        }
-
-        private void UploadHardwareConfiguration()
-        {
-            Result = ConnectedMachineVMResult.UploadHardwareConfig;
-            Accept();
-        }
-
-        private void Disconnect()
+        private void AcceptResult(ConnectedMachineVMResult result)
         {
-            Result = ConnectedMachineVMResult.Disconnect;
+            Result = result;
             Accept();
         }
     }
diff --git a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/MainViewVM.cs b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/MainViewVM.cs
index e53ec0b39..df1ea5f7e 100644
--- a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/MainViewVM.cs
+++ b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/MainViewVM.cs
@@ -37,6 +37,7 @@ using Tango.Settings;
 using Tango.SharedUI;
 using Tango.SharedUI.Controls;
 using Tango.SharedUI.Helpers;
+using Tango.Transport;
 using Tango.Transport.Adapters;
 
 namespace Tango.MachineStudio.UI.ViewModels
@@ -449,27 +450,33 @@ namespace Tango.MachineStudio.UI.ViewModels
                                             x.SelectedMachine.EnableEmbeddedDebugging = true;
                                         }
 
-                                        await x.SelectedMachine.Connect();
-
-                                        var authenticated = await x.SelectedMachine.As<IExternalBridgeSecureClient>().Authenticate(login.Password, Environment.MachineName, AuthenticationProvider.CurrentUser.Guid, "Machine Studio");
-                                        if (!authenticated)
+                                        await x.SelectedMachine.As<IExternalBridgeSecureClient>().Connect(new PMR.Integration.ExternalBridgeLoginRequest()
                                         {
-                                            _notificationProvider.ShowError("It seems like you are not authorized to access the selected machine.");
-                                        }
-                                        else
+                                            AppID = "Machine Studio",
+                                            HostName = Environment.MachineName,
+                                            Password = login.Password,
+                                            UserGuid = AuthenticationProvider.CurrentUser.Guid,
+                                            Intent = PMR.Integration.ExternalBridgeLoginIntent.Override,
+                                        });
+
+                                        ApplicationManager.ConnectedMachine = x.SelectedMachine;
+                                        (x.SelectedMachine as IExternalBridgeSecureClient).SessionClosed += (_, __) =>
                                         {
-                                            ApplicationManager.ConnectedMachine = x.SelectedMachine;
-                                            (x.SelectedMachine as IExternalBridgeSecureClient).SessionClosed += (_, __) =>
+                                            InvokeUI(() =>
                                             {
-                                                InvokeUI(() =>
-                                                {
-                                                    _notificationProvider.ShowError("The remote machine has closed the current session. Machine disconnected.");
-                                                    ApplicationManager.ConnectedMachine = null;
-                                                });
-                                            };
-                                            PostMessage(new MachineConnectionChangedMessage() { Machine = x.SelectedMachine });
-                                            _eventLogger.Log(String.Format("Successfully connected to machine {0} via TCP", x.SelectedMachine.SerialNumber));
-                                        }
+                                                _notificationProvider.ShowError("The remote machine has closed the current session. Machine disconnected.");
+                                                ApplicationManager.ConnectedMachine = null;
+                                            });
+                                        };
+                                        PostMessage(new MachineConnectionChangedMessage() { Machine = x.SelectedMachine });
+                                        _eventLogger.Log(String.Format("Successfully connected to machine {0} via TCP", x.SelectedMachine.SerialNumber));
+
+                                    }
+                                    catch (ResponseErrorException ex)
+                                    {
+                                        LogManager.Log(ex);
+                                        _eventLogger.Log(ex, "Error connecting to machine " + x.SelectedMachine.SerialNumber);
+                                        _notificationProvider.ShowError("Could not connect to the selected machine." + Environment.NewLine + ex.Container.ErrorMessage);
                                     }
                                     catch (Exception ex)
                                     {
@@ -547,8 +554,8 @@ namespace Tango.MachineStudio.UI.ViewModels
                                 }
                                 catch (Exception ex)
                                 {
-                                    LogManager.Log(ex, "Error upload hardware configuration.");
-                                    NotificationProvider.ShowError("Error upload hardware configuration." + Environment.NewLine + ex.Message);
+                                    LogManager.Log(ex, "Error uploading hardware configuration.");
+                                    NotificationProvider.ShowError("Error uploading hardware configuration." + Environment.NewLine + ex.Message);
                                 }
                             }
                         }
@@ -557,7 +564,7 @@ namespace Tango.MachineStudio.UI.ViewModels
                     {
                         if (NotificationProvider.ShowQuestion("This will reset the embedded device. Are you sure?"))
                         {
-                            using (NotificationProvider.PushTaskItem("Uploading hardware configuration..."))
+                            using (NotificationProvider.PushTaskItem("Resetting the embedded device..."))
                             {
                                 try
                                 {
@@ -572,6 +579,25 @@ namespace Tango.MachineStudio.UI.ViewModels
                             }
                         }
                     }
+                    else if (x.Result == ConnectedMachineViewVM.ConnectedMachineVMResult.TurnOffHeaters)
+                    {
+                        if (NotificationProvider.ShowQuestion("This will reset the process parameters. Are you sure?"))
+                        {
+                            using (NotificationProvider.PushTaskItem("Resetting process parameters..."))
+                            {
+                                try
+                                {
+                                    await ApplicationManager.ConnectedMachine.UploadProcessParameters(new ProcessParametersTable());
+                                    NotificationProvider.ShowInfo("Heaters are turned off.");
+                                }
+                                catch (Exception ex)
+                                {
+                                    LogManager.Log(ex, "Error resetting process parameters.");
+                                    NotificationProvider.ShowError("Error resetting process parameters." + Environment.NewLine + ex.Message);
+                                }
+                            }
+                        }
+                    }
                 });
             }
 
@@ -737,7 +763,9 @@ namespace Tango.MachineStudio.UI.ViewModels
 
         private void ReportIssue()
         {
-            _notificationProvider.ShowModalDialog<ReportIssueViewVM, ReportIssueView>(new ReportIssueViewVM(TFSClient.Project, TFSClient.CreateBug()), async (vm) =>
+            var bug = TFSClient.CreateBug();
+
+            _notificationProvider.ShowModalDialog<ReportIssueViewVM, ReportIssueView>(new ReportIssueViewVM(TFSClient.Project, bug), async (vm) =>
              {
                  using (_notificationProvider.PushTaskItem("Uploading bug report..."))
                  {
diff --git a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ReportIssueViewVM.cs b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ReportIssueViewVM.cs
index 2935ff81b..96800d9cf 100644
--- a/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ReportIssueViewVM.cs
+++ b/Software/Visual_Studio/MachineStudio/Tango.MachineStudio.UI/ViewModels/ReportIssueViewVM.cs
@@ -3,8 +3,10 @@ using System.Collections.Generic;
 using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
+using Tango.MachineStudio.Common;
 using Tango.MachineStudio.Common.Notifications;
 using Tango.MachineStudio.UI.TFS;
+using Tango.Settings;
 using Tango.SharedUI;
 using Tango.SharedUI.Components;
 using Tango.TFS;
@@ -13,6 +15,8 @@ namespace Tango.MachineStudio.UI.ViewModels
 {
     public class ReportIssueViewVM : DialogViewVM
     {
+        private MachineStudioSettings _settings;
+
         public SelectedObjectCollection<Tag> SelectedTags { get; set; }
 
         public Project Project { get; set; }
@@ -22,14 +26,33 @@ namespace Tango.MachineStudio.UI.ViewModels
 
         public ReportIssueViewVM() : base()
         {
-
+            _settings = SettingsManager.Default.GetOrCreate<MachineStudioSettings>();
         }
 
         public ReportIssueViewVM(Project project, WorkItem workItem) : this()
         {
             Project = project;
             WorkItem = workItem;
-            SelectedTags = new SelectedObjectCollection<Tag>(Project.Tags.ToObservableCollection(), new System.Collections.ObjectModel.ObservableCollection<Tag>());
+
+            var area = project.Areas.FirstOrDefault(x => x.Name == _settings.DefaultIssueReportArea);
+            if (area != null)
+            {
+                workItem.Area = area;
+            }
+
+            var assignedTo = project.Members.FirstOrDefault(x => x.Email == _settings.DefaultIssueReportAssignTo);
+            if (assignedTo != null)
+            {
+                workItem.AssignedTo = assignedTo;
+            }
+
+            var tags = project.Tags.Where(x => _settings.DefaultIssueReportTags.Contains(x.Name)).ToList();
+            if (tags != null)
+            {
+                workItem.Tags = tags;
+            }
+
+            SelectedTags = new SelectedObjectCollection<Tag>(Project.Tags.ToObservableCollection(), workItem.Tags.ToObservableCollection());
         }
 
         protected override void Accept()
@@ -37,6 +60,16 @@ namespace Tango.MachineStudio.UI.ViewModels
             if (Validate())
             {
                 WorkItem.Tags = SelectedTags.SynchedSource.ToList();
+
+                try
+                {
+                    _settings.DefaultIssueReportArea = WorkItem.Area.Name;
+                    _settings.DefaultIssueReportAssignTo = WorkItem.AssignedTo.Email;
+                    _settings.DefaultIssueReportTags = WorkItem.Tags.Select(x => x.Name).ToList();
+                    _settings.Save();
+                }
+                catch { }
+
                 base.Accept();
             }
         }