MS. Performance when open RML extensions. Bug during delete tab - remove objects before save.

Related Work Items: #7357

2 files changed, 34 insertions, 19 deletions

diff --git a/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/MainViewVM.cs b/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/MainViewVM.cs
index 458bc913a..686d95ccc 100644
--- a/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/MainViewVM.cs
+++ b/Software/Visual_Studio/MachineStudio/Modules/Tango.MachineStudio.ThreadExtensions/ViewModels/MainViewVM.cs
@@ -992,16 +992,7 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
             YarnTypes = _active_context.YarnTypes.ToObservableCollection();
             IndustrySector = _active_context.YarnIndustrysectors.ToObservableCollection();
 
-            var machines = new MachinesCollectionBuilder(_active_context).SetAll().WithConfiguration().Build();
-            Machines = machines.Select(x => new MachineModel()
-            {
-                Guid = x.Guid,
-                Name = x.Name,
-                SerialNumber = x.SerialNumber,
-                IdsPacks = x.Configuration.IdsPacks.Where(z => !z.IsEmpty),
-                LastUpdated = x.LastUpdated,
-                HasRMLTest = false
-            }).ToObservableCollection();
+            
 
         }
 
@@ -1058,6 +1049,16 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
 
                     _active_context = ObservablesContext.CreateDefault();
                     LoadRmlProperties();
+                    var machines = await new MachinesCollectionBuilder(_active_context).SetAll().WithConfiguration().BuildListAsync();
+                    Machines = machines.Select(x => new MachineModel()
+                    {
+                        Guid = x.Guid,
+                        Name = x.Name,
+                        SerialNumber = x.SerialNumber,
+                        IdsPacks = x.Configuration.IdsPacks.Where(z => !z.IsEmpty),
+                        LastUpdated = x.LastUpdated,
+                        HasRMLTest = false
+                    }).ToObservableCollection();
 
                     if (guid == null)
                     {
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 ba7ae7238..c63a34394 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
@@ -198,9 +198,13 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
                 
                 _active_context.TensileResults.RemoveRange(tab.TestResult.TensileResults);
                 tab.TestResult.TensileResults = null;
-                
+
+                _active_context.RmlExtensionTestWashingResults.RemoveRange(tab.TestResult.RmlExtensionTestWashingResults);
+                tab.TestResult.RmlExtensionTestWashingResults = null;
+
                 _active_context.RmlExtensionTestResults.Remove(tab.TestResult);
 
+                
                 ResultTabs.Remove(tab);
                 SelectedTab = ResultTabs.LastOrDefault();
                 _active_context.SaveChanges();
@@ -211,7 +215,7 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
         /// <summary>
         /// Adds a new tab.
         /// </summary>
-        private bool AddNewTab(String name = null)
+        private  bool AddNewTab(String name = null)
         {
             if (ResultTabs.Count > 7)
             {
@@ -226,11 +230,21 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
             
             if (!String.IsNullOrWhiteSpace(name))
             {
-                var tab = CreateNewTestResultVM(name, (ResultTabs.Count + 1));
-                ResultTabs.Add(tab);
-                _active_context.RmlExtensionTestResults.Add(tab.TestResult);
-                SelectedTab = tab;
-                return true;
+                try
+                {
+                    var tab = CreateNewTestResultVM(name, (ResultTabs.Count + 1));
+                    ResultTabs.Add(tab);
+                    _active_context.RmlExtensionTestResults.Add(tab.TestResult);
+                     _active_context.SaveChanges();
+                    SelectedTab = ResultTabs.FirstOrDefault(x=> x.TestResult.Guid == tab.TestResult.Guid);
+                    
+                    return true;
+                }
+                catch(Exception ex)
+                {
+                    LogManager.Log(ex, $"Error add new Tab TestResults.\n{ex.FlattenMessage()}");
+                    return false;
+                }
             }
             else
             {
@@ -472,7 +486,7 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
             try
             {
                 IsFree = false;
-                await Task.Factory.StartNew(() =>
+                await Task.Run(() =>
                 {
                     foreach (var tab in ResultTabs)
                     {
@@ -482,7 +496,7 @@ namespace Tango.MachineStudio.ThreadExtensions.ViewModels
                     _active_context.SaveChanges();
                 });
                
-              //  LoadTestResults();
+                //LoadTestResults();
             }
             catch (Exception ex)
             {