Changeset 9839 for trunk/sources/HeuristicLab.Core.Views/3.3/CheckedItemCollectionView.cs

Timestamp:

08/01/13 16:56:57 (11 years ago)

Author:

abeham

Message:

#2044:

• Updated checked state of listview items in CheckedItemListView when elements were moved in the CheckedItemList
• Cosmetic change in CheckedItemCollectionView

File:

• trunk/sources/HeuristicLab.Core.Views/3.3/CheckedItemCollectionView.cs (modified) (3 diffs)

trunk/sources/HeuristicLab.Core.Views/3.3/CheckedItemCollectionView.cs

@@ -21 +21 @@
 
 using System;
+using System.Collections.Generic;
 using System.Drawing;
 using System.Linq;
@@ -97 +98 @@
 
     #region Content Events
-    protected virtual void Content_CheckedItemsChanged(object sender, HeuristicLab.Collections.CollectionItemsChangedEventArgs<T> e) {
+    protected virtual void Content_CheckedItemsChanged(object sender, CollectionItemsChangedEventArgs<T> e) {
       if (InvokeRequired)
         Invoke(new CollectionItemsChangedEventHandler<T>(Content_CheckedItemsChanged), sender, e);
       else {
-        foreach (T item in e.Items) {
-          foreach (ListViewItem listViewItem in GetListViewItemsForItem(item)) {
-            if (listViewItem.Checked != Content.ItemChecked(item))
-              listViewItem.Checked = Content.ItemChecked(item);
-          }
-        }
+        UpdateCheckedItemState(e.Items);
         SetNumberOfCheckItems();
       }
@@ -131 +127 @@
       }
     }
+
+    private void UpdateCheckedItemState(IEnumerable<T> items) {
+      foreach (T item in items) {
+        foreach (ListViewItem listViewItem in GetListViewItemsForItem(item)) {
+          var isChecked = Content.ItemChecked(item);
+          if (listViewItem.Checked != isChecked)
+            listViewItem.Checked = isChecked;
+        }
+      }
+    }
   }
 }