Changeset 3068 for branches/DeploymentServer Prototype/HeuristicLab.Services/HeuristicLab.DeploymentService.AdminClient/PluginListView.cs

Timestamp:

03/16/10 17:14:35 (15 years ago)

Author:

gkronber

Message:

Worked on multi-select behavior of list views in deployment server interfaces. #891 (Refactor GUI for plugin management)

File:

• branches/DeploymentServer Prototype/HeuristicLab.Services/HeuristicLab.DeploymentService.AdminClient/PluginListView.cs (modified) (6 diffs)

branches/DeploymentServer Prototype/HeuristicLab.Services/HeuristicLab.DeploymentService.AdminClient/PluginListView.cs

@@ -10 +10 @@
 
 namespace HeuristicLab.DeploymentService.AdminClient {
+  /// <summary>
+  /// Wraps a ListView and adds functionality to automatically check and uncheck dependencies of plugins.
+  /// </summary>
   public partial class PluginListView : UserControl {
     public event ItemCheckedEventHandler ItemChecked;
@@ -18 +21 @@
       set {
         plugins = value;
+        checkedPlugins.Clear();
         UpdateControls();
       }
     }
 
+    private List<IPluginDescription> checkedPlugins = new List<IPluginDescription>();
     public IEnumerable<IPluginDescription> CheckedPlugins {
       get {
-        return from item in listView.Items.OfType<ListViewItem>()
-               where item.Checked
-               select item.Tag as IPluginDescription;
+        return checkedPlugins;
       }
     }
-
-    private bool suppressItemCheckedEvents = false;
 
     public PluginListView() {
@@ -37 +38 @@
 
     public void CheckPlugin(IPluginDescription plugin) {
-      var items = FindItemsForPlugin(plugin);
-      if (items.Count() == 0) throw new ArgumentException();
-      var item = items.Single();
-      if (!item.Checked) {
-        suppressItemCheckedEvents = true;
-        item.Checked = true;
-        suppressItemCheckedEvents = false;
-      }
+      MarkPluginChecked(plugin);
+      listView.CheckItems(new IPluginDescription[] { plugin }.Select(x => FindItemsForPlugin(x).Single()));
     }
+
 
     private void UpdateControls() {
       if (plugins != null) {
         listView.Items.Clear();
-        suppressItemCheckedEvents = true;
+        listView.SuppressItemCheckedEvents = true;
         foreach (var plugin in plugins) {
           listView.Items.Add(CreateListViewItem(plugin));
         }
-        suppressItemCheckedEvents = false;
+        listView.SuppressItemCheckedEvents = false;
       }
     }
@@ -60 +56 @@
     private ListViewItem CreateListViewItem(IPluginDescription plugin) {
       var item = new ListViewItem(new string[] { plugin.Name, plugin.Version.ToString() });
+      item.Checked = (from p in checkedPlugins where p.Name == plugin.Name where p.Version == plugin.Version select p).Any();
       item.Tag = plugin;
       return item;
@@ -65 +62 @@
 
     private void listView_ItemChecked(object sender, ItemCheckedEventArgs e) {
-      if (suppressItemCheckedEvents) return;
+      List<IPluginDescription> modifiedPlugins = new List<IPluginDescription>();
       if (e.Item.Checked) {
-        // also check all dependencies
-        suppressItemCheckedEvents = true;
-        var dependencies = GetAllDependencies(e.Item.Tag as IPluginDescription);
-        foreach (var dep in dependencies) {
-          var item = FindItemsForPlugin(dep).Single();
-          if (!item.Checked) item.Checked = true;
+        foreach (ListViewItem item in listView.SelectedItems) {
+          var plugin = (IPluginDescription)item.Tag;
+          // also check all dependencies
+          MarkPluginChecked(plugin);
+          modifiedPlugins.Add(plugin);
+          foreach (var dep in GetAllDependencies(plugin)) {
+            MarkPluginChecked(dep);
+            modifiedPlugins.Add(dep);
+          }
         }
-        suppressItemCheckedEvents = false;
+        listView.CheckItems(modifiedPlugins.Select(x => FindItemsForPlugin(x).Single()));
+        OnItemChecked(e);
       } else {
-        // also uncheck all dependencies
-        suppressItemCheckedEvents = true;
-        var dependents = GetAllDependents(e.Item.Tag as IPluginDescription);
-        foreach (var dep in dependents) {
-          var item = FindItemsForPlugin(dep).Single();
-          if (item.Checked) item.Checked = false;
+        foreach (ListViewItem item in listView.SelectedItems) {
+          var plugin = (IPluginDescription)item.Tag;
+          // also uncheck all dependent plugins
+          MarkPluginUnchecked(plugin);
+          modifiedPlugins.Add(plugin);
+          foreach (var dep in GetAllDependents(plugin)) {
+            MarkPluginUnchecked(dep);
+            modifiedPlugins.Add(dep);
+          }
+
         }
-        suppressItemCheckedEvents = false;
+        listView.UncheckItems(modifiedPlugins.Select(x => FindItemsForPlugin(x).Single()));
+        OnItemChecked(e);
       }
-      OnItemChecked(e);
+    }
+
+    private void MarkPluginChecked(IPluginDescription plugin) {
+      var matching = from p in checkedPlugins
+                     where p.Name == plugin.Name
+                     where p.Version == plugin.Version
+                     select p;
+      if (!matching.Any()) checkedPlugins.Add(plugin);
+    }
+
+    private void MarkPluginUnchecked(IPluginDescription plugin) {
+      checkedPlugins = (from p in checkedPlugins
+                        where p.Name != plugin.Name ||
+                              p.Version != plugin.Version
+                        select p).ToList();
     }
 
@@ -116 +136 @@
 
     private void OnItemChecked(ItemCheckedEventArgs e) {
-      if (ItemChecked != null && !suppressItemCheckedEvents) ItemChecked(this, e);
+      if (ItemChecked != null) ItemChecked(this, e);
     }
   }