From ac3190506290a687b80d01b846f28b636cfa0ca3 Mon Sep 17 00:00:00 2001
From: Dan Gardiner <dangardiner@gmail.com>
Date: Wed, 20 May 2020 10:45:17 -0500
Subject: [PATCH] Add DeleteMany action and organize code

---
 .../FieldsInUse/Views/FieldDefListView.xaml   |   1 +
 .../Views/FieldDefListView.xaml.cs            | 255 ++++++++++--------
 .../Views/ManageUnusedFieldsDialog.xaml.cs    | 137 ++++++++--
 3 files changed, 255 insertions(+), 138 deletions(-)

diff --git a/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml b/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml
index 8d8ae40..2663a8d 100644
--- a/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml
+++ b/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml
@@ -83,6 +83,7 @@
             </StackPanel>
             <StackPanel Orientation="Horizontal" Grid.Column="1">
                 <Button Margin="0,3,10,3" Content="Refresh"  Style="{StaticResource LinkButton}" Click="RefreshRuleApplicationList_Click"/>
+                <Button Margin="0,3,10,3" Content="Delete All"  Style="{StaticResource LinkButton}" Click="DeleteAll_Click"/>
                 <Button Margin="0,3,10,3" Content="Close"  Style="{StaticResource LinkButton}" Click="Close_Click"/>
             </StackPanel>
         </Grid>
diff --git a/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml.cs b/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml.cs
index a93dc37..913da2f 100644
--- a/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml.cs
+++ b/FieldsInUse/FieldsInUse/Views/FieldDefListView.xaml.cs
@@ -12,6 +12,8 @@
 using InRule.Authoring.Extensions;
 using InRule.Repository;
 using InRuleLabs.AuthoringExtensions.FieldsInUse.Extensions;
+using System.Xml.Linq;
+using InRule.Repository.Infos;
 
 namespace InRuleLabs.AuthoringExtensions.FieldsInUse.Views
 {
@@ -20,8 +22,6 @@ namespace InRuleLabs.AuthoringExtensions.FieldsInUse.Views
     /// </summary>
     public partial class FieldDefListView : UserControl, IDisposable
     {
-        public event EventHandler<object> OnCloseView;
-        
         [Flags]
         public enum Options
         {
@@ -30,23 +30,23 @@ public enum Options
             TypeNameColumn = 32,
         }
 
-        public void SetTitleBarText(string title)
-        {
-            this.TitleBar.Content = title;
-        }
-        private bool TryGetUnusedFieldList(out IList<UnusedFieldReference> unusedFields, out string errorMsg)
-        {
-            unusedFields = this.TargetRuleApplicationDef.GetUnusedFields()
-                .Select(t => new UnusedFieldReference(t)).ToList();
-            errorMsg = null;
-            return true;
-        }
-
+        public event EventHandler<object> OnCloseView;
         public RuleApplicationDef TargetRuleApplicationDef { get; set; }
+        public ListViewSortManager ListViewSortManager { get; private set; }
         public string DefaultSortColumnName { get; set; }
-        public FieldDefListView(RuleApplicationDef ruleAppDef, Options includedOptions, List<UnusedFieldAction> actions, string defaultSortColumnName)
+        private List<UnusedFieldAction> _actions;
+        private Action<FieldDefListView, IEnumerable<UnusedFieldReference>> _deleteAllAction;
+        private bool _hasLoaded;
+
+        public FieldDefListView(RuleApplicationDef ruleAppDef, 
+                                Options includedOptions, 
+                                List<UnusedFieldAction> actions, 
+                                Action<FieldDefListView, IEnumerable<UnusedFieldReference>> deleteAllAction, 
+                                string defaultSortColumnName)
         {
             this.Background = Brushes.White;
+            _actions = actions;
+            _deleteAllAction = deleteAllAction;
             TargetRuleApplicationDef = ruleAppDef;
             DefaultSortColumnName = defaultSortColumnName;
         
@@ -56,19 +56,13 @@ public FieldDefListView(RuleApplicationDef ruleAppDef, Options includedOptions,
             InitializeComponent();
 
             DataTemplate cardLayout = new DataTemplate();
-            //cardLayout.DataType = typeof(CreditCardPayment);
-
-            //set up the stack panel
-            
             
             if (includedOptions.HasFlag(Options.NameColumn)) { AddColumn("NameColumn");}
             if (includedOptions.HasFlag(Options.TypeNameColumn)) { AddColumn("TypeNameColumn"); }
 
-            
             actions = new List<UnusedFieldAction>(actions);
             
             FrameworkElementFactory spFactory = new FrameworkElementFactory(typeof(StackPanel));
-            //spFactory.Name = "myComboFactory";
             spFactory.SetValue(StackPanel.OrientationProperty, Orientation.Horizontal);
 
             int count = 0;
@@ -92,12 +86,10 @@ public FieldDefListView(RuleApplicationDef ruleAppDef, Options includedOptions,
                 spFactory.AppendChild(buttonFactory);
             }
             
-            
             //set the visual tree of the data template
             cardLayout.VisualTree = spFactory;
 
             var actionColumn = AddColumn("ActionColumn");
-            //set the item template to be our shiny new data template
             actionColumn.CellTemplate = cardLayout;
             
             ListViewSortManager = new ListViewSortManager(this.lstViewValues);
@@ -107,7 +99,12 @@ public FieldDefListView(RuleApplicationDef ruleAppDef, Options includedOptions,
             HideHistoryBorder();
 
             this.OnCloseView += OnClosingView;
+        }
 
+        #region Helpers
+        public void SetTitleBarText(string title)
+        {
+            this.TitleBar.Content = title;
         }
 
         private GridViewColumn AddColumn(string resourceName)
@@ -121,31 +118,96 @@ private GridViewColumn GetColumn(string resourceName)
         {
             return this.Resources[resourceName] as GridViewColumn;
         }
-        
-        private void OnClosingView(object sender, object e)
+
+        private void HideHistoryBorder()
         {
-            
+            Dispatcher.Invoke(() =>
+            {
+                OuterHistoryBorder.Visibility = Visibility.Hidden;
+            });
         }
 
+        private static ListSortDirection GetColumnDefaultSortDirection(string columnName)
+        {
+            switch (columnName)
+            {
+                case "LastModifiedDate":
+                    {
+                        return ListSortDirection.Descending;
+                    }
+            }
+            return ListSortDirection.Ascending;
+        }
+        #endregion
 
-        public ListViewSortManager ListViewSortManager { get; private set; }
-
+        #region Basic Event Handlers
         private void FieldDefListView_Initialized(object sender, EventArgs e)
         {
             _hasLoaded = true;
             RefreshRuleAppListAsync();
         }
 
-      
+        private void RefreshRuleApplicationList_Click(object sender, RoutedEventArgs e)
+        {
+            RefreshRuleAppListAsync();
+        }
 
-        private bool _hasLoaded;
+        private void CheckApplyDefaultSorting()
+        {
+            if (!ListViewSortManager.HasSortApplied)
+            {
+                ListViewSortManager.SortByColumnNameAscending(DefaultSortColumnName);
+            }
+        }
 
-        private void RefreshRuleAppListAsync()
+        private void HideErrorMessage()
         {
+            Dispatcher.Invoke(() =>
+            {
+                ErrorMessageBorder.Visibility = Visibility.Hidden;
+            });
+        }
 
-            if (!_hasLoaded) return;
+        private void txtFilter_OnKeyUp(object sender, KeyEventArgs e)
+        {
+            if (e.Key == Key.Return)
+            {
+            }
+        }
 
-           
+        private void TxtFilter_OnTextChanged(object sender, TextChangedEventArgs e)
+        {
+            var view = CollectionViewSource.GetDefaultView(lstViewValues.ItemsSource);
+            view.Refresh();
+        }
+
+        private void Close_Click(object sender, RoutedEventArgs e)
+        {
+            try
+            {
+
+                OnCloseView?.Invoke(this, null);
+            }
+            catch (Exception ex)
+            {
+                MessageBox.Show(ex.Message);
+            }
+        }
+
+        public void Dispose()
+        {
+            //RuleAppDBService.OnDatabaseUpdated -= OnDatabaseUpdated;
+        }
+
+        private void OnClosingView(object sender, object e)
+        {
+        }
+        #endregion
+
+        #region Unused Field Loading Logic
+        private void RefreshRuleAppListAsync()
+        {
+            if (!_hasLoaded) return;
 
             try
             {
@@ -158,7 +220,6 @@ private void RefreshRuleAppListAsync()
                 MessageBox.Show(ex.Message);
                 waitSpinner.StopSpinning();
             }
-
         }
         public void LoadAllRuleApps()
         {
@@ -170,27 +231,36 @@ public void LoadAllRuleApps()
             }
             else
             {
-                Dispatcher.Invoke(() => { OnUnusedFieldsRetrieved(unusedFieldList); }
-                );
+                Dispatcher.Invoke(() => { OnUnusedFieldsRetrieved(unusedFieldList); });
             }
         }
-       
-        private void HideErrorMessage()
-        {
-            Dispatcher.Invoke(() =>
-            {
-                ErrorMessageBorder.Visibility = Visibility.Hidden;
-            });
-
-        }
 
-        private void HideHistoryBorder()
+        private bool TryGetUnusedFieldList(out IList<UnusedFieldReference> unusedFields, out string errorMsg)
         {
-            Dispatcher.Invoke(() =>
+            unusedFields = this.TargetRuleApplicationDef
+                               .GetUnusedFields()
+                               .Select(t => new UnusedFieldReference(t))
+                               .ToList();
+
+            var usageNetwork = DefUsageNetwork.Create(this.TargetRuleApplicationDef);
+            var allEntities = this.TargetRuleApplicationDef.GetChildDefsByType<EntityDef>();
+            foreach (var entity in allEntities)
             {
-                OuterHistoryBorder.Visibility = Visibility.Hidden;
-            });
+                var thisUsages = usageNetwork.GetDefUsages(entity.Guid);
+                if (!thisUsages.Any(usage => usage.UsageType == DefUsageType.ConsumedBy
+                                            || usage.UsageType == DefUsageType.InvalidatedBy
+                                            || usage.UsageType == DefUsageType.UpdatedBy
+                                            || usage.UsageType == DefUsageType.TypeUsedBy
+                                            || usage.UsageType == DefUsageType.ParentOf  //This forces all children to be removed before this can be considered unused.  Alternative would be to auto-select children
+                                            )
+                    && entity.Fields.Count == 0)
+                {
+                    unusedFields.Add(new UnusedFieldReference(entity));
+                }
+            }
 
+            errorMsg = null;
+            return true;
         }
 
         private void OnUnusedFieldsRetrieved(IList<UnusedFieldReference> ruleAppDescriptors)
@@ -211,33 +281,6 @@ private void OnUnusedFieldsRetrieved(IList<UnusedFieldReference> ruleAppDescript
             }
         }
 
-        private void OnError(string errorMsg)
-        {
-            Dispatcher.Invoke(() =>
-            {
-                ErrorMessageBorder.Visibility = Visibility.Visible;
-                var displayErrorMessage =
-                    $"An error occured connecting to the rule application database.{Environment.NewLine}{Environment.NewLine}Please check your connection string settings in irAuthor under File->Options->UBF Options{Environment.NewLine}{Environment.NewLine}Error Message:{Environment.NewLine}{errorMsg}";
-                ErrorMessageTextBlock.Text = displayErrorMessage;
-                //MessageBox.Show(displayErrorMessage);
-                waitSpinner.StopSpinning();
-            });
-        }
-
-        private void CheckApplyDefaultSorting()
-        {
-            if (!ListViewSortManager.HasSortApplied)
-            {
-                ListViewSortManager.SortByColumnNameAscending(DefaultSortColumnName);
-            }
-        }
-
-        private void RefreshRuleApplicationList_Click(object sender, RoutedEventArgs e)
-        {
-            RefreshRuleAppListAsync();
-        }
-
-        
         private bool TestRuleAppAgainstFilter(object item)
         {
             var dbRuleApp = item as UnusedFieldReference;
@@ -250,52 +293,35 @@ private bool TestRuleAppAgainstFilter(object item)
             if (dbRuleApp.TypeName.ContainsIgnoreCase(filter)) return true;
             return false;
         }
+        #endregion
 
-
-        private void txtFilter_OnKeyUp(object sender, KeyEventArgs e)
-        {
-            if (e.Key == Key.Return)
-            {
-              
-            }
-        }
-
-        private void TxtFilter_OnTextChanged(object sender, TextChangedEventArgs e)
-        {
-            var view = CollectionViewSource.GetDefaultView(lstViewValues.ItemsSource);
-            view.Refresh();
-        }
-
-        private static ListSortDirection GetColumnDefaultSortDirection(string columnName)
+        private void OnError(string errorMsg)
         {
-            switch (columnName)
+            Dispatcher.Invoke(() =>
             {
-                case "LastModifiedDate":
-                {
-                    return ListSortDirection.Descending;
-                }
-            }
-            return ListSortDirection.Ascending;
+                ErrorMessageBorder.Visibility = Visibility.Visible;
+                var displayErrorMessage =
+                    $"An error occured connecting to the rule application database.{Environment.NewLine}{Environment.NewLine}Please check your connection string settings in irAuthor under File->Options->UBF Options{Environment.NewLine}{Environment.NewLine}Error Message:{Environment.NewLine}{errorMsg}";
+                ErrorMessageTextBlock.Text = displayErrorMessage;
+                //MessageBox.Show(displayErrorMessage);
+                waitSpinner.StopSpinning();
+            });
         }
 
-      
-        private void Close_Click(object sender, RoutedEventArgs e)
+        private void DeleteAll_Click(object sender, RoutedEventArgs e)
         {
             try
             {
-
-                OnCloseView?.Invoke(this, null);
+                waitSpinner.StartSpinning();
+                var allItems = this.lstViewValues.ItemsSource.ToList<UnusedFieldReference>();
+                _deleteAllAction(this, allItems);
             }
             catch (Exception ex)
             {
                 MessageBox.Show(ex.Message);
             }
-        }
-      
-        
-        public void Dispose()
-        {
-            //RuleAppDBService.OnDatabaseUpdated -= OnDatabaseUpdated;
+            RefreshRuleAppListAsync();
+            waitSpinner.StopSpinning();
         }
 
         public void RemoveItem(UnusedFieldReference target)
@@ -303,6 +329,13 @@ public void RemoveItem(UnusedFieldReference target)
             var source = lstViewValues.ItemsSource as ObservableCollection<UnusedFieldReference>;
             source.Remove(target);
         }
+        public void RemoveRange(IEnumerable<UnusedFieldReference> target)
+        {
+            var source = lstViewValues.ItemsSource as ObservableCollection<UnusedFieldReference>;
+            var sourceList = source.ToList();
+            sourceList.RemoveAll(s => target.Contains(s));
+            lstViewValues.ItemsSource = new ObservableCollection<UnusedFieldReference>(sourceList);
+        }
     }
 
     public class UnusedFieldAction
@@ -321,8 +354,8 @@ public void ExecuteAction_Click(object sender, RoutedEventArgs e)
             try
             {
                 var button = sender as Button;
-                var ruleApp = button.DataContext as UnusedFieldReference;
-                Action(ListView, ruleApp);
+                var unusedRef = button.DataContext as UnusedFieldReference;
+                Action(ListView, unusedRef);
             }
             catch (Exception ex)
             {
diff --git a/FieldsInUse/FieldsInUse/Views/ManageUnusedFieldsDialog.xaml.cs b/FieldsInUse/FieldsInUse/Views/ManageUnusedFieldsDialog.xaml.cs
index 3f6d2a0..ac1d575 100644
--- a/FieldsInUse/FieldsInUse/Views/ManageUnusedFieldsDialog.xaml.cs
+++ b/FieldsInUse/FieldsInUse/Views/ManageUnusedFieldsDialog.xaml.cs
@@ -2,7 +2,12 @@
 using System.Windows;
 using InRule.Authoring.Services;
 using InRule.Repository;
+using InRule.Authoring.Extensions;
+using System.Linq;
+using System.Xml.Linq;
 using InRuleLabs.AuthoringExtensions.FieldsInUse.Extensions;
+using InRule.Repository.EndPoints;
+using System.Diagnostics;
 
 namespace InRuleLabs.AuthoringExtensions.FieldsInUse.Views
 {
@@ -10,29 +15,90 @@ internal partial class ManageUnusedFieldsDialog : Window
 	{
 	    public FieldDefListView FieldDefListView;
 	    public RuleApplicationController _controller;
+        private IEnumerable<IRuleRepositoryDefBase> _schemas;
         public ManageUnusedFieldsDialog(RuleApplicationDef ruleAppDef, RuleApplicationController controller)
 	    {
 	        InitializeComponent();
 	        _controller = controller;
 
             var actions = new List<UnusedFieldAction>();
-	        actions.Add(new UnusedFieldAction("Delete", (list, target) =>
-	        {
-	            _controller.RemoveDef(target.InnerDef);
-	            list.RemoveItem(target);
-	        }));
+            _schemas = ruleAppDef.SchemasRoot.AsEnumerable();
+
+            actions.Add(new UnusedFieldAction("Delete", DeleteAction));
             
 	        var columns = FieldDefListView.Options.NameColumn;
 	        columns |= FieldDefListView.Options.TypeNameColumn;
-	        
             
-	        FieldDefListView = new FieldDefListView(ruleAppDef, columns, actions, "Name");
+	        FieldDefListView = new FieldDefListView(ruleAppDef, columns, actions, DeleteManyAction, "Name");
 	        FieldDefListView.OnCloseView += FieldDefListViewOnCloseView;
             
             this.grdMain.Children.Add(FieldDefListView);
 	    }
+        private void DeleteAction(FieldDefListView list, UnusedFieldReference target)
+        {
+            //If the field is bound to a schema, the field must be removed from the schema as well, otherwise there will be validation errors
+            if (target.ExternalSchemaFieldDef != null)
+            {
+                var info = target.ExternalSchemaFieldDef;
+                var parentName = info.AuthoringElementPath.Substring(0, info.AuthoringElementPath.LastIndexOf('.'));
+                var elementName = info.AuthoringElementPath.Replace(parentName, "").Substring(1);
 
-	    private void FieldDefListViewOnCloseView(object sender, object e)
+                var schema = _schemas.FirstOrDefault(s => s.Name == info.AuthoringElementPath.Split('.')[0]);
+                if (schema is XmlSchemaDef xmlSchema)
+                {
+                    if (target.ExternalSchemaFieldDef is EntityDefInfo)
+                    {
+                        var schemaItem = xmlSchema.EntityDefsInfo.EntityDefInfos.FirstOrDefault(i => i.Name == elementName);
+                        xmlSchema.EntityDefsInfo.EntityDefInfos.Remove(schemaItem);
+                    }
+                    else if (target.ExternalSchemaFieldDef is FieldDefInfo)
+                    {
+                        var schemaItem = xmlSchema.EntityDefsInfo.EntityDefInfos.FirstOrDefault(i => i.AuthoringElementPath == parentName);
+                        var fieldItem = schemaItem.Fields[elementName];
+                        schemaItem.Fields.Remove(fieldItem);
+                    }
+                }
+            }
+
+            _controller.RemoveDef(target.InnerDef);
+            list.RemoveItem(target);
+        }
+        public void DeleteManyAction(FieldDefListView list, IEnumerable<UnusedFieldReference> targets)
+        {
+            foreach (var target in targets)
+            {
+                //If the field is bound to a schema, the field must be removed from the schema as well, otherwise there will be validation errors
+                if (target.ExternalSchemaFieldDef != null)
+                {
+                    var info = target.ExternalSchemaFieldDef;
+                    var parentName = info.AuthoringElementPath.Substring(0, info.AuthoringElementPath.LastIndexOf('.'));
+                    var elementName = info.AuthoringElementPath.Replace(parentName, "").Substring(1);
+
+                    var schema = _schemas.FirstOrDefault(s => s.Name == info.AuthoringElementPath.Split('.')[0]);
+                    if (schema is XmlSchemaDef xmlSchema)
+                    {
+                        if (target.ExternalSchemaFieldDef is EntityDefInfo)
+                        {
+                            var schemaItem = xmlSchema.EntityDefsInfo.EntityDefInfos.FirstOrDefault(i => i.Name == elementName);
+                            xmlSchema.EntityDefsInfo.EntityDefInfos.Remove(schemaItem);
+                        }
+                        else if (target.ExternalSchemaFieldDef is FieldDefInfo)
+                        {
+                            var schemaItem = xmlSchema.EntityDefsInfo.EntityDefInfos.FirstOrDefault(i => i.AuthoringElementPath == parentName);
+                            var fieldItem = schemaItem.Fields[elementName];
+                            schemaItem.Fields.Remove(fieldItem);
+                        }
+                    }
+                }
+
+                //This is the bit that takes the 99% of the time 
+                _controller.RemoveDef(target.InnerDef);
+            }
+
+            list.RemoveRange(targets);
+        }
+
+        private void FieldDefListViewOnCloseView(object sender, object e)
 	    {
 	        this.Close();
 	    }
@@ -51,8 +117,14 @@ public class UnusedFieldReference
         public UnusedFieldReference(RuleRepositoryDefBase innerDef)
         {
             this.InnerDef = innerDef;
+
+            if(InnerDef is FieldDef fd)
+                this.ExternalSchemaFieldDef = fd.ExternalSchemaFieldDef;
+            else if(InnerDef is EntityDef ed)
+                this.ExternalSchemaFieldDef = ed.ExternalSchemaEntityDef;
         }
         public RuleRepositoryDefBase InnerDef { get; set; }
+        public RuleRepositoryDefBase ExternalSchemaFieldDef { get; set; }
 
 
         public string Name { get { return InnerDef.AuthoringElementPath; } }
@@ -61,29 +133,40 @@ public string TypeName
         {
             get
             {
-                var def = InnerDef as FieldDef;
-                if (def == null) return "Unknown";
+                if (InnerDef == null) return "Unknown";
 
-                switch (def.DataType)
+                if (InnerDef is FieldDef def)
                 {
-                    case DataType.Entity:
-                    {
-                        if (def.IsCollection)
-                        {
-                            return $"{def.DataTypeEntityName}[]";
-                        }
-                        return $"{def.DataTypeEntityName}";
-                    }
-                    case DataType.Complex:
+
+                    switch (def.DataType)
                     {
-                        if (def.IsCollection)
-                        {
-                            return $"{def.DataType.ToString()}[]";
-                        }
-                        return $"{def.DataType.ToString()}";
+                        case DataType.Entity:
+                            {
+                                if (def.IsCollection)
+                                {
+                                    return $"{def.DataTypeEntityName}[]";
+                                }
+                                return $"{def.DataTypeEntityName}";
+                            }
+                        case DataType.Complex:
+                            {
+                                if (def.IsCollection)
+                                {
+                                    return $"{def.DataType.ToString()}[]";
+                                }
+                                return $"{def.DataType.ToString()}";
+                            }
+                        default:
+                            return def.DataType.ToString();
                     }
-                    default:
-                        return def.DataType.ToString();
+                }
+                else if (InnerDef is EntityDef)
+                {
+                    return "Entity";
+                }
+                else
+                {
+                    return "Error";
                 }
             }
         }