Changeset 16870

Timestamp:

04/24/19 10:59:39 (6 years ago)

Author:

chaider

Message:

#2971 changed content events in ParsedConstraintView

File:

• branches/2971_named_intervals/HeuristicLab.Problems.DataAnalysis.Views/3.4/ParsedConstraintView.cs (modified) (6 diffs)

branches/2971_named_intervals/HeuristicLab.Problems.DataAnalysis.Views/3.4/ParsedConstraintView.cs

@@ -40 +40 @@
       errorOutput.Text = "";
       intervalConstraints = new CheckedItemList<IntervalConstraint>();
-      intervalConstraints.CheckedItemsChanged += constraint_CheckedItemChanged;
-      //intervalConstraints.ItemsRemoved
     }
 
@@ -48 +46 @@
       UpdateControl();
     }
-    //ParameterCollectionView
-    //Remove method directly at parsebutton event
+
+    protected override void RegisterContentEvents() {
+      base.RegisterContentEvents();
+      intervalConstraints.CheckedItemsChanged += constraint_CheckedItemChanged;
+      intervalConstraints.ItemsAdded += constraints_Updated;
+      intervalConstraints.ItemsRemoved += constraints_Updated; 
+    }
+
+    protected override void DeregisterContentEvents() {
+      intervalConstraints.CheckedItemsChanged -= constraint_CheckedItemChanged;
+      intervalConstraints.ItemsAdded -= constraints_Updated;
+      intervalConstraints.ItemsRemoved -= constraints_Updated;
+      base.DeregisterContentEvents();
+    }
+
+
     private IEnumerable<IntervalConstraint> ParseConstraints(string input) {
       return IntervalConstraintsParser.ParseInput(input, Content.ProblemData.TargetVariable,
@@ -57 +69 @@
     private void parseBtn_Click(object sender, EventArgs e) {
       if (constraintsInput.Text != null) {
-        ClearConstraints();
+        intervalConstraints.Clear();
         var parsedConstraints = ParseConstraints(constraintsInput.Text);
         foreach (var constraint in parsedConstraints) {
-          constraint.Changed += constraint_Changed;
           intervalConstraints.Add(constraint, constraint.Enabled);
         }
@@ -74 +85 @@
     private void UpdateControl() {
       if (Content == null) {
-        ClearConstraints();
+        intervalConstraints.Clear();
         constraintsOutput.Content = intervalConstraints.AsReadOnly();
       } else {
-        ClearConstraints();
+        intervalConstraints.Clear();
         constraintsInput.Text = Content.Input;
         foreach (var constraint in Content.Constraints) {
-          constraint.Changed += constraint_Changed;
           intervalConstraints.Add(constraint, constraint.Enabled);
         }
@@ -91 +101 @@
       var constraint = (IntervalConstraint) sender;
       intervalConstraints.SetItemCheckedState(constraint, constraint.Enabled);
+    }
+
+    private void constraints_Updated(object sender, CollectionItemsChangedEventArgs<IndexedItem<IntervalConstraint>> e) {
+      foreach (var addedItem in e.Items) {
+        addedItem.Value.Changed += constraint_Changed;
+      }
     }
 
@@ -105 +121 @@
     }
 
-    private void ClearConstraints() {
-      foreach (var constraint in intervalConstraints) {
-        constraint.Changed -= constraint_Changed;
-      }
-      intervalConstraints.Clear();
-    }
-
   }
 }