MS. RML Extension - save test results is failed when user add new machine.

1 files changed, 19 insertions, 23 deletions

diff --git a/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/TestResultsViewVM.cs b/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/TestResultsViewVM.cs
index c63a34394..34ceb9ce8 100644
--- a/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/TestResultsViewVM.cs
+++ b/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/TestResultsViewVM.cs
@@ -234,8 +234,10 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
                 {
                     var tab = CreateNewTestResultVM(name, (ResultTabs.Count + 1));
                     ResultTabs.Add(tab);
+                    SelectedTestResults.Add(tab.TestResult);
                     _active_context.RmlExtensionTestResults.Add(tab.TestResult);
-                     _active_context.SaveChanges();
+                    
+                    _active_context.SaveChanges();
                     SelectedTab = ResultTabs.FirstOrDefault(x=> x.TestResult.Guid == tab.TestResult.Guid);
                     
                     return true;
@@ -288,31 +290,28 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
         {
             if (String.IsNullOrEmpty(SelectedMachineGUID))
             {
-             //   _notification.ShowWarning(LogManager.Log($" Please, select machine.", LogCategory.Warning));
                 return;
             }
 
             try
             {
                 IsFree = false;
-            if (_active_context != null)
-            {
-                _active_context.Dispose();
-            }
-            _active_context = ObservablesContext.CreateDefault();
-            ResultTabs.Clear();
-
-            if(WashingTestMaterials == null)
-                WashingTestMaterials =  _active_context.WashingTestMaterials.OrderBy(x=> x.Name).ToList();
-
                 LogManager.Log("Loading selected test results...");
 
+                if (_active_context != null)
+                {
+                    _active_context.Dispose();
+                }
+
+                _active_context = ObservablesContext.CreateDefault();
+                ResultTabs.Clear();
+                
                 using (_notification.PushTaskItem("Loading Test Results Parameters ..."))
                 {
                     
                     BtsrApplicationTypes = _active_context.BtsrApplicationTypes.ToObservableCollection();
-                    
                     BtsrYarnTypes = _active_context.BtsrYarnTypes.ToObservableCollection();
+                    WashingTestMaterials =  _active_context.WashingTestMaterials.OrderBy(x=> x.Name).ToList();
 
                     var testResults = await new RMLExtensionTestResultsCollectionBuilder(_active_context).SetAll().ForRMLExtension(RMLExtemtionGUID).ForMachine(SelectedMachineGUID).WithRubbingAndTensileResults().WithTestResultsFiles().WithWashingTestResults().WithBtsrApplicationTypes().WithBtsrYarnTypes().BuildAsync();
                     SelectedTestResults = testResults.OrderBy(x => x.ResultIndex).ToSynchronizedObservableCollection();
@@ -321,24 +320,20 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
                         var testResultViewVM = new TestResultViewVM(_notification, _actionLogManager) { TestResult = result, ThreadName = ThreadName };
                         CreateWashingResult(testResultViewVM);
                         ResultTabs.Add(testResultViewVM);
-                        if (result.ResultIndex == 1)
-                        {
-                            SelectedTab = ResultTabs[ResultTabs.Count - 1];
-                        }
+                        
                     }
                     if (ResultTabs.Count == 0)
                     {
-                        SelectedTab = CreateNewTestResultVM("Untitled", 1);
-                        ResultTabs.Add(SelectedTab);
-                        _active_context.RmlExtensionTestResults.Add(SelectedTab.TestResult);
-                        await _active_context.SaveChangesAsync();
+                        AddNewTab("Untitled");
                     }
+                    if(ResultTabs.Count > 0)
+                        SelectedTab = ResultTabs[ResultTabs.Count - 1];
                 }
                 IsFree = true;
             }
             catch (Exception ex)
             {
-                LogManager.Log(ex, $"Error loading TestResults.\n{ex.FlattenMessage()}");
+                LogManager.Log(ex, $"Error loading TestResults.\n{ex.FlattenMessage()}\n Inner Exception: {ex.InnerException.InnerException.ToString()}");
             }
             finally
             {
@@ -352,6 +347,8 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
             newtab.TestResult = new RmlExtensionTestResult() { RmlsExtensionsGuid = RMLExtemtionGUID, MachineGuid = SelectedMachineGUID, ResultIndex = index, Name = name, BtsrMax = 0.0, BtsrMin = 0.0, DryerTemperature = 0, TunnelTemperature = 0, TunnelFlow = 0.0, TunnelAvgTemperature = 0.0, TensionHeadMax = 0.0,
             TensionHeadMin = 0.0, TensioinAfterDryerMax = 0.0, TensionAfterDryerMin = 0.0, TensionWinderMax = 0.0, TensionWinderMin = 0.0, PullerTensionMax = 0.0, PullerTensionMin = 0.0, ExitTensionMax = 0.0, ExitTensionMin = 0.0, SeverityZone1Max = 0.0, SeverityZone1Min = 0.0, SeverityZone2Max = 0.0, SeverityZone2Min = 0.0,
             RefLubVersion="", RefCof = 0.0, RefLub = 0.0, ThreadLubVersion = "", ThreadCof = 0.0, ThreadLub = 0.0, Conclusions="", Comment=""};
+
+            
             var rubbingresults = new SynchronizedObservableCollection<RubbingResult>();
             rubbingresults.Add(new RubbingResult() { RmlExtensionTestResultsGuid = newtab.TestResult.Guid, TestResultColor = TestResultColors.CYAN });
             rubbingresults.Add(new RubbingResult() { RmlExtensionTestResultsGuid = newtab.TestResult.Guid, TestResultColor = TestResultColors.MAGENTA });
@@ -407,7 +404,6 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
                         model.AddMaterial(rmlExtensionTestWashingResult.Guid, material.Name, null);
                         rmlExtensionTestWashingResult.WashingTestMaterials = material;
                         whashingresults.Add(rmlExtensionTestWashingResult);
-
                     }
                     collecction.Add( model);
                 }