Changeset 15422

Timestamp:

10/17/17 13:44:16 (7 years ago)

Author:

jkarder

Message:

#2839: worked on resources and projects views

Location:

branches/HiveProjectManagement

Files:

• HeuristicLab.Clients.Access.Administration/3.3/Views/UserGroupView.cs (modified) (2 diffs)
• HeuristicLab.Clients.Access/3.3/ServiceClients/User.cs (modified) (1 diff)
• HeuristicLab.Clients.Hive.Administrator/3.3/HeuristicLab.Clients.Hive.Administrator-3.3.csproj (modified) (3 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectPermissionsView.Designer.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectPermissionsView.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectResourcesView.Designer.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectResourcesView.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectView.Designer.cs (modified) (4 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectView.cs (modified) (5 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectsView.Designer.cs (modified) (9 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectsView.cs (modified) (13 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourceView.Designer.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourceView.cs (added)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourcesView.Designer.cs (modified) (10 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourcesView.cs (modified) (15 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/SlaveView.Designer.cs (modified) (2 diffs)
• HeuristicLab.Clients.Hive.Administrator/3.3/Views/SlaveView.cs (modified) (3 diffs)

branches/HiveProjectManagement/HeuristicLab.Clients.Access.Administration/3.3/Views/UserGroupView.cs

@@ -40 +40 @@
     protected override void RegisterContentEvents() {
       base.RegisterContentEvents();
-      Access.AccessClient.Instance.Refreshing += new EventHandler(Content_Refreshing);
+      AccessClient.Instance.Refreshing += new EventHandler(Content_Refreshing);
       refreshableLightweightUserView.StorableStateChanged += new EventHandler(refreshableLightweightUserView_StorableStateChanged);
     }
 
     protected override void DeregisterContentEvents() {
-      Access.AccessClient.Instance.Refreshing -= new EventHandler(Content_Refreshing);
+      AccessClient.Instance.Refreshing -= new EventHandler(Content_Refreshing);
       refreshableLightweightUserView.StorableStateChanged -= new EventHandler(refreshableLightweightUserView_StorableStateChanged);
       base.DeregisterContentEvents();
@@ -63 +63 @@
         idTextBox.Text = Content.Id.ToString();
 
-        refreshableLightweightUserView.Content = Content.Id != Guid.Empty ? Access.AccessClient.Instance : null;
-        refreshableLightweightUserView.FetchSelectedUsers = Content.Id != Guid.Empty ? new Func<List<Guid>>(delegate { return AccessAdministrationClient.CallAccessService<List<Guid>>(s => s.GetUserGroupIdsOfGroup(Content.Id)); }) : null;
+        refreshableLightweightUserView.Content = Content.Id != Guid.Empty ? AccessClient.Instance : null;
+        refreshableLightweightUserView.FetchSelectedUsers = Content.Id != Guid.Empty ? new Func<List<Guid>>(delegate { return AccessAdministrationClient.CallAccessService(s => s.GetUserGroupIdsOfGroup(Content.Id)); }) : null;
       }
     }
 
-    private void groupNameTextBox_TextChanged(object sender, System.EventArgs e) {
-      if (Content.Name != this.groupNameTextBox.Text)
-        Content.Name = this.groupNameTextBox.Text;
+    private void groupNameTextBox_TextChanged(object sender, EventArgs e) {
+      if (Content != null && Content.Name != groupNameTextBox.Text)
+        Content.Name = groupNameTextBox.Text;
     }
 

branches/HiveProjectManagement/HeuristicLab.Clients.Access/3.3/ServiceClients/User.cs

@@ -51 +51 @@
 
     public override string ToString() {
-      return UserName + "(" + FullName + ")";
+      return UserName + " (" + FullName + ")";
     }
   }

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/HeuristicLab.Clients.Hive.Administrator-3.3.csproj

@@ -132 +132 @@
     </Compile>
     <Compile Include="MenuItems\AdministratorMenuItem.cs" />
+    <Compile Include="Views\ProjectPermissionsView.cs">
+      <SubType>UserControl</SubType>
+    </Compile>
+    <Compile Include="Views\ProjectPermissionsView.Designer.cs">
+      <DependentUpon>ProjectPermissionsView.cs</DependentUpon>
+    </Compile>
+    <Compile Include="Views\ProjectResourcesView.cs">
+      <SubType>UserControl</SubType>
+    </Compile>
+    <Compile Include="Views\ProjectResourcesView.Designer.cs">
+      <DependentUpon>ProjectResourcesView.cs</DependentUpon>
+    </Compile>
     <Compile Include="Views\ProjectsView.cs">
       <SubType>UserControl</SubType>
@@ -162 +174 @@
     <Compile Include="Views\ScheduleView.Designer.cs">
       <DependentUpon>ScheduleView.cs</DependentUpon>
+    </Compile>
+    <Compile Include="Views\ResourceView.cs">
+      <SubType>UserControl</SubType>
+    </Compile>
+    <Compile Include="Views\ResourceView.Designer.cs">
+      <DependentUpon>ResourceView.cs</DependentUpon>
     </Compile>
     <Compile Include="Views\SlaveView.cs">
@@ -178 +196 @@
   </ItemGroup>
   <ItemGroup>
+    <ProjectReference Include="..\..\HeuristicLab.Clients.Access.Administration\3.3\HeuristicLab.Clients.Access.Administration-3.3.csproj">
+      <Project>{C9CF6AE8-4637-4A75-9141-042F479B5D50}</Project>
+      <Name>HeuristicLab.Clients.Access.Administration-3.3</Name>
+    </ProjectReference>
     <ProjectReference Include="..\..\HeuristicLab.Clients.Access.Views\3.3\HeuristicLab.Clients.Access.Views-3.3.csproj">
       <Project>{FE856595-64CD-46DA-9CD2-FFF3E6B0D4F2}</Project>

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectView.Designer.cs

@@ -57 +57 @@
       this.nameTextBox.Size = new System.Drawing.Size(464, 20);
       this.nameTextBox.TabIndex = 1;
+      this.nameTextBox.TextChanged += new System.EventHandler(this.nameTextBox_TextChanged);
       this.nameTextBox.Validating += new System.ComponentModel.CancelEventHandler(this.nameTextBox_Validating);
       // 
@@ -77 +78 @@
       this.descriptionTextBox.Size = new System.Drawing.Size(464, 98);
       this.descriptionTextBox.TabIndex = 3;
-      this.descriptionTextBox.Validating += new System.ComponentModel.CancelEventHandler(this.descriptionTextBox_Validating);
+      this.descriptionTextBox.TextChanged += new System.EventHandler(this.descriptionTextBox_TextChanged);
       // 
       // ownerComboBox
@@ -118 +119 @@
       this.startDateTimePicker.Size = new System.Drawing.Size(200, 20);
       this.startDateTimePicker.TabIndex = 9;
-      this.startDateTimePicker.Validating += new System.ComponentModel.CancelEventHandler(this.startDateTimePicker_Validating);
+      this.startDateTimePicker.ValueChanged += new System.EventHandler(this.startDateTimePicker_ValueChanged);
       // 
       // startLabel
@@ -138 +139 @@
       this.endDateTimePicker.Size = new System.Drawing.Size(200, 20);
       this.endDateTimePicker.TabIndex = 11;
-      this.endDateTimePicker.Validating += new System.ComponentModel.CancelEventHandler(this.endDateTimePicker_Validating);
+      this.endDateTimePicker.ValueChanged += new System.EventHandler(this.endDateTimePicker_ValueChanged);
       // 
       // endLabel

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectView.cs

@@ -34 +34 @@
   public partial class ProjectView : ItemView {
     private readonly object locker = new object();
-    private bool updatingUsers = false;
 
     public new Project Content {
@@ -91 +90 @@
     protected override void SetEnabledStateOfControls() {
       base.SetEnabledStateOfControls();
-      bool enabled = Content != null;
+      bool enabled = Content != null && !ReadOnly;
       nameTextBox.Enabled = enabled;
       descriptionTextBox.Enabled = enabled;
@@ -138 +137 @@
     private async void refreshButton_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (updatingUsers) return;
-        updatingUsers = true;
+        if (!refreshButton.Enabled) return;
+        refreshButton.Enabled = false;
       }
 
@@ -149 +148 @@
           ownerComboBox.SelectedItem = AccessClient.Instance.UsersAndGroups.FirstOrDefault(x => x.Id == Content.OwnerUserId);
           ownerComboBox.SelectedIndexChanged += ownerComboBox_SelectedIndexChanged;
-          updatingUsers = false;
+          refreshButton.Enabled = true;
         });
     }
@@ -161 +160 @@
           MessageBoxIcon.Error);
         e.Cancel = true;
-      } else Content.Name = nameTextBox.Text;
-    }
-
-    private void descriptionTextBox_Validating(object sender, CancelEventArgs e) {
-      Content.Description = descriptionTextBox.Text;
+      }
+    }
+
+    private void nameTextBox_TextChanged(object sender, EventArgs e) {
+      if (Content != null && Content.Name != nameTextBox.Text)
+        Content.Name = nameTextBox.Text;
+    }
+
+    private void descriptionTextBox_TextChanged(object sender, EventArgs e) {
+      if (Content != null && Content.Description != descriptionTextBox.Text)
+        Content.Description = descriptionTextBox.Text;
     }
 
     private void ownerComboBox_SelectedIndexChanged(object sender, EventArgs e) {
       var selectedItem = (LightweightUser)ownerComboBox.SelectedItem;
-      Content.OwnerUserId = selectedItem != null ? selectedItem.Id : Guid.Empty;
-    }
-
-    private void startDateTimePicker_Validating(object sender, CancelEventArgs e) {
+      var selectedOwnerUserId = selectedItem != null ? selectedItem.Id : Guid.Empty;
+      if (Content != null && Content.OwnerUserId != selectedOwnerUserId)
+        Content.OwnerUserId = selectedOwnerUserId;
+    }
+
+    private void startDateTimePicker_ValueChanged(object sender, EventArgs e) {
+      if (Content == null) return;
       if (!Content.EndDate.HasValue || startDateTimePicker.Value > Content.EndDate)
         endDateTimePicker.Value = startDateTimePicker.Value;
-      Content.StartDate = startDateTimePicker.Value;
-    }
-
-    private void endDateTimePicker_Validating(object sender, CancelEventArgs e) {
+      if (Content.StartDate != startDateTimePicker.Value)
+        Content.StartDate = startDateTimePicker.Value;
+    }
+
+    private void endDateTimePicker_ValueChanged(object sender, EventArgs e) {
+      if (Content == null) return;
       if (endDateTimePicker.Value < startDateTimePicker.Value)
         endDateTimePicker.Value = startDateTimePicker.Value;
-      Content.EndDate = endDateTimePicker.Value;
+      if (Content.EndDate != endDateTimePicker.Value)
+        Content.EndDate = endDateTimePicker.Value;
     }
 
     private void indefiniteCheckBox_CheckedChanged(object sender, EventArgs e) {
-      Content.EndDate = indefiniteCheckBox.Checked ? (DateTime?)null : Content.StartDate;
+      if (Content == null) return;
+      var newEndDate = indefiniteCheckBox.Checked ? (DateTime?)null : Content.StartDate;
+      if (Content.EndDate != newEndDate)
+        Content.EndDate = newEndDate;
     }
     #endregion

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectsView.Designer.cs

@@ -36 +36 @@
       this.projectView = new HeuristicLab.Clients.Hive.Administrator.Views.ProjectView();
       this.permissionsTabPage = new System.Windows.Forms.TabPage();
-      this.savePermissionsButton = new System.Windows.Forms.Button();
-      this.permissionsView = new HeuristicLab.Clients.Access.Views.RefreshableLightweightUserView();
+      this.projectPermissionsView = new HeuristicLab.Clients.Hive.Administrator.Views.ProjectPermissionsView();
+      this.resourcesTabPage = new System.Windows.Forms.TabPage();
+      this.projectResourcesView = new HeuristicLab.Clients.Hive.Administrator.Views.ProjectResourcesView();
       ((System.ComponentModel.ISupportInitialize)(this.splitContainer)).BeginInit();
       this.splitContainer.Panel1.SuspendLayout();
@@ -45 +46 @@
       this.detailsTabPage.SuspendLayout();
       this.permissionsTabPage.SuspendLayout();
+      this.resourcesTabPage.SuspendLayout();
       this.SuspendLayout();
       // 
@@ -82 +84 @@
       // saveProjectButton
       // 
-      this.saveProjectButton.Image = HeuristicLab.Common.Resources.VSImageLibrary.PublishToWeb;
+      this.saveProjectButton.Image = HeuristicLab.Common.Resources.VSImageLibrary.Save;
       this.saveProjectButton.Location = new System.Drawing.Point(93, 3);
       this.saveProjectButton.Name = "saveProjectButton";
@@ -143 +145 @@
       this.tabControl.Controls.Add(this.detailsTabPage);
       this.tabControl.Controls.Add(this.permissionsTabPage);
+      this.tabControl.Controls.Add(this.resourcesTabPage);
       this.tabControl.Location = new System.Drawing.Point(3, 3);
       this.tabControl.Name = "tabControl";
@@ -148 +151 @@
       this.tabControl.Size = new System.Drawing.Size(585, 541);
       this.tabControl.TabIndex = 0;
-      this.tabControl.SelectedIndexChanged += new System.EventHandler(this.tabControl_SelectedIndexChanged);
       // 
       // detailsTabPage
@@ -176 +178 @@
       // permissionsTabPage
       // 
-      this.permissionsTabPage.Controls.Add(this.savePermissionsButton);
-      this.permissionsTabPage.Controls.Add(this.permissionsView);
+      this.permissionsTabPage.Controls.Add(this.projectPermissionsView);
       this.permissionsTabPage.Location = new System.Drawing.Point(4, 22);
       this.permissionsTabPage.Name = "permissionsTabPage";
@@ -186 +187 @@
       this.permissionsTabPage.UseVisualStyleBackColor = true;
       // 
-      // savePermissionsButton
-      // 
-      this.savePermissionsButton.Enabled = false;
-      this.savePermissionsButton.Image = HeuristicLab.Common.Resources.VSImageLibrary.PublishToWeb;
-      this.savePermissionsButton.Location = new System.Drawing.Point(39, 9);
-      this.savePermissionsButton.Name = "savePermissionsButton";
-      this.savePermissionsButton.Size = new System.Drawing.Size(24, 24);
-      this.savePermissionsButton.TabIndex = 2;
-      this.savePermissionsButton.UseVisualStyleBackColor = true;
-      this.savePermissionsButton.Click += new System.EventHandler(this.savePermissionsButton_Click);
-      // 
-      // permissionsView
-      // 
-      this.permissionsView.Anchor = ((System.Windows.Forms.AnchorStyles)((((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Bottom) 
-            | System.Windows.Forms.AnchorStyles.Left) 
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.permissionsView.Caption = "RefreshableLightweightUser View";
-      this.permissionsView.Content = null;
-      this.permissionsView.FetchSelectedUsers = null;
-      this.permissionsView.Location = new System.Drawing.Point(6, 6);
-      this.permissionsView.Name = "permissionsView";
-      this.permissionsView.ReadOnly = false;
-      this.permissionsView.Size = new System.Drawing.Size(565, 503);
-      this.permissionsView.TabIndex = 0;
+      // projectPermissionsView
+      // 
+      this.projectPermissionsView.Caption = "ProjectView";
+      this.projectPermissionsView.Content = null;
+      this.projectPermissionsView.Dock = System.Windows.Forms.DockStyle.Fill;
+      this.projectPermissionsView.Location = new System.Drawing.Point(3, 3);
+      this.projectPermissionsView.Name = "projectPermissionsView";
+      this.projectPermissionsView.ReadOnly = false;
+      this.projectPermissionsView.Size = new System.Drawing.Size(571, 509);
+      this.projectPermissionsView.TabIndex = 0;
+      // 
+      // resourcesTabPage
+      // 
+      this.resourcesTabPage.Controls.Add(this.projectResourcesView);
+      this.resourcesTabPage.Location = new System.Drawing.Point(4, 22);
+      this.resourcesTabPage.Name = "resourcesTabPage";
+      this.resourcesTabPage.Padding = new System.Windows.Forms.Padding(3);
+      this.resourcesTabPage.Size = new System.Drawing.Size(577, 515);
+      this.resourcesTabPage.TabIndex = 2;
+      this.resourcesTabPage.Text = "Resources";
+      this.resourcesTabPage.UseVisualStyleBackColor = true;
+      // 
+      // projectResourcesView
+      // 
+      this.projectResourcesView.Caption = "ProjectView";
+      this.projectResourcesView.Content = null;
+      this.projectResourcesView.Dock = System.Windows.Forms.DockStyle.Fill;
+      this.projectResourcesView.Location = new System.Drawing.Point(3, 3);
+      this.projectResourcesView.Name = "projectResourcesView";
+      this.projectResourcesView.ReadOnly = false;
+      this.projectResourcesView.Size = new System.Drawing.Size(571, 509);
+      this.projectResourcesView.TabIndex = 0;
       // 
       // ProjectsView
@@ -226 +235 @@
       this.detailsTabPage.ResumeLayout(false);
       this.permissionsTabPage.ResumeLayout(false);
+      this.resourcesTabPage.ResumeLayout(false);
       this.ResumeLayout(false);
 
@@ -243 +253 @@
     private ProjectView projectView;
     private System.Windows.Forms.ImageList imageList;
-    private Access.Views.RefreshableLightweightUserView permissionsView;
-    private System.Windows.Forms.Button savePermissionsButton;
+    private System.Windows.Forms.TabPage resourcesTabPage;
+    private ProjectResourcesView projectResourcesView;
+    private ProjectPermissionsView projectPermissionsView;
   }
 }

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ProjectsView.cs

@@ -42 +42 @@
 
     private readonly object locker = new object();
-    private bool updatingProjects = false;
-    private bool removingProjects = false;
-    private bool savingProjects = false;
 
     public new IItemList<Project> Content {
@@ -89 +86 @@
         projectsTreeView.Nodes.Clear();
         projectView.Content = null;
-        permissionsView.Content = null;
-        permissionsView.FetchSelectedUsers = null;
+        projectPermissionsView.Content = null;
+        projectResourcesView.Content = null;
       } else {
         var top = BuildProjectTree(Content);
         projectView.Content = top;
-        permissionsView.Content = AccessClient.Instance;
+        projectPermissionsView.Content = top;
+        projectResourcesView.Content = top;
       }
     }
@@ -100 +98 @@
     protected override void SetEnabledStateOfControls() {
       base.SetEnabledStateOfControls();
-      bool enabled = Content != null;
+      bool enabled = Content != null && !ReadOnly;
+      refreshButton.Enabled = enabled;
       addButton.Enabled = enabled;
       removeButton.Enabled = enabled;
       saveProjectButton.Enabled = enabled;
-      savePermissionsButton.Enabled = enabled && permissionsView.Content != null && permissionsView.FetchSelectedUsers != null;
-      permissionsView.Enabled = enabled && permissionsView.Content != null && permissionsView.FetchSelectedUsers != null;
+      projectView.Enabled = enabled;
+      projectPermissionsView.Enabled = enabled;
+      projectResourcesView.Enabled = enabled;
     }
     #endregion
@@ -175 +175 @@
     private async void refreshButton_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (updatingProjects) return;
-        updatingProjects = true;
+        if (!refreshButton.Enabled) return;
+        refreshButton.Enabled = false;
       }
 
@@ -183 +183 @@
         finallyCallback: () => {
           Content = HiveAdminClient.Instance.Projects;
-          updatingProjects = false;
+          refreshButton.Enabled = true;
         });
     }
@@ -197 +197 @@
     private async void removeButton_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (removingProjects) return;
-        removingProjects = true;
+        if (!removeButton.Enabled) return;
+        removeButton.Enabled = false;
       }
 
@@ -223 +223 @@
             finallyCallback: () => {
               Content.Remove(project);
-              removingProjects = false;
+              removeButton.Enabled = true;
             });
         }
@@ -231 +231 @@
     private async void saveProjectButton_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (savingProjects) return;
-        savingProjects = true;
+        if (!saveProjectButton.Enabled) return;
+        saveProjectButton.Enabled = false;
       }
 
@@ -241 +241 @@
             project.Store();
         },
-        finallyCallback: () => savingProjects = false);
-    }
-
-    private async void projectsTreeView_AfterSelect(object sender, TreeViewEventArgs e) {
+        finallyCallback: () => saveProjectButton.Enabled = true);
+    }
+
+    private void projectsTreeView_AfterSelect(object sender, TreeViewEventArgs e) {
       var selectedProject = (Project)e.Node.Tag;
 
@@ -251 +251 @@
 
       projectView.Content = selectedProject;
+      projectPermissionsView.Content = selectedProject;
+      projectResourcesView.Content = selectedProject;
 
       if (selectedProject != null)
@@ -258 +260 @@
         if (!tabControl.TabPages.Contains(permissionsTabPage))
           tabControl.TabPages.Add(permissionsTabPage);
-
-        permissionsView.FetchSelectedUsers = () =>
-          HiveServiceLocator.Instance.CallHiveService(s => s.GetProjectPermissions(selectedProject.Id))
-                                     .Select(x => x.GrantedUserId)
-                                     .ToList();
-
-        if (tabControl.SelectedIndex == 1 && permissionsView.Content != null && permissionsView.FetchSelectedUsers != null)
-          await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
-            action: () => UpdatePermissions());
+        if (!tabControl.TabPages.Contains(resourcesTabPage))
+          tabControl.TabPages.Add(resourcesTabPage);
       } else {
         if (tabControl.TabPages.Contains(permissionsTabPage))
           tabControl.TabPages.Remove(permissionsTabPage);
-        permissionsView.FetchSelectedUsers = null;
+        if (tabControl.TabPages.Contains(resourcesTabPage))
+          tabControl.TabPages.Remove(resourcesTabPage);
       }
     }
@@ -317 +313 @@
     private void projectsTreeView_QueryContinueDrag(object sender, QueryContinueDragEventArgs e) {
       e.Action = DragAction.Continue;
-    }
-
-    private async void tabControl_SelectedIndexChanged(object sender, EventArgs e) {
-      if (tabControl.SelectedIndex == 1 && permissionsView.Content != null && permissionsView.FetchSelectedUsers != null)
-        await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
-          action: () => UpdatePermissions());
-    }
-
-    private async void savePermissionsButton_Click(object sender, EventArgs e) {
-      var project = (Project)projectsTreeView.SelectedNode.Tag;
-
-      await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
-        action: () => HiveServiceLocator.Instance.CallHiveService(s => {
-          s.GrantProjectPermissions(project.Id, permissionsView.GetAddedUsers().Select(x => x.Id).ToList());
-          s.RevokeProjectPermissions(project.Id, permissionsView.GetDeletedUsers().Select(x => x.Id).ToList());
-        }));
     }
     #endregion
@@ -401 +381 @@
     }
 
-    private void UpdatePermissions() {
-      try {
-        permissionsView.ManualRefresh();
-      } catch (AnonymousUserException) {
-        ShowHiveInformationDialog();
-      }
-    }
-
     private bool IsAuthorized(Project project) {
       return project != null

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourcesView.Designer.cs

@@ -37 +37 @@
         components.Dispose();
       }
-      HiveAdminClient.Instance.Refreshing -= new EventHandler(HiveAdminClient_Instance_Refreshing);
-      HiveAdminClient.Instance.Refreshed -= new EventHandler(HiveAdminClient_Instance_Refreshed);
-
-      Access.AccessClient.Instance.Refreshing -= new EventHandler(AccessClient_Instance_Refreshing);
-      Access.AccessClient.Instance.Refreshed -= new EventHandler(AccessClient_Instance_Refreshed);
       base.Dispose(disposing);
     }
@@ -59 +54 @@
       this.btnRemoveGroup = new System.Windows.Forms.Button();
       this.btnAddGroup = new System.Windows.Forms.Button();
-      this.treeSlaveGroup = new System.Windows.Forms.TreeView();
+      this.treeView = new System.Windows.Forms.TreeView();
       this.tabSlaveGroup = new System.Windows.Forms.TabControl();
       this.tabDetails = new System.Windows.Forms.TabPage();
+      this.viewHost = new HeuristicLab.MainForm.WindowsForms.ViewHost();
       this.tabSchedule = new System.Windows.Forms.TabPage();
+      this.scheduleView = new HeuristicLab.Clients.Hive.Administrator.Views.ScheduleView();
       this.toolTip = new System.Windows.Forms.ToolTip(this.components);
-      this.slaveView = new HeuristicLab.Clients.Hive.Administrator.Views.SlaveView();
-      this.scheduleView = new HeuristicLab.Clients.Hive.Administrator.Views.ScheduleView();
       ((System.ComponentModel.ISupportInitialize)(this.splitSlaves)).BeginInit();
       this.splitSlaves.Panel1.SuspendLayout();
@@ -95 +90 @@
       this.splitSlaves.Panel1.Controls.Add(this.btnRemoveGroup);
       this.splitSlaves.Panel1.Controls.Add(this.btnAddGroup);
-      this.splitSlaves.Panel1.Controls.Add(this.treeSlaveGroup);
+      this.splitSlaves.Panel1.Controls.Add(this.treeView);
       // 
       // splitSlaves.Panel2
@@ -117 +112 @@
       // btnSave
       // 
-      this.btnSave.Image = HeuristicLab.Common.Resources.VSImageLibrary.PublishToWeb;
+      this.btnSave.Image = HeuristicLab.Common.Resources.VSImageLibrary.Save;
       this.btnSave.Location = new System.Drawing.Point(93, 3);
       this.btnSave.Name = "btnSave";
@@ -148 +143 @@
       this.btnAddGroup.Click += new System.EventHandler(this.btnAddGroup_Click);
       // 
-      // treeSlaveGroup
-      // 
-      this.treeSlaveGroup.AllowDrop = true;
-      this.treeSlaveGroup.Anchor = ((System.Windows.Forms.AnchorStyles)((((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Bottom) 
-            | System.Windows.Forms.AnchorStyles.Left) 
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.treeSlaveGroup.ImageIndex = 0;
-      this.treeSlaveGroup.ImageList = this.imageListSlaveGroups;
-      this.treeSlaveGroup.Location = new System.Drawing.Point(3, 33);
-      this.treeSlaveGroup.Name = "treeSlaveGroup";
-      this.treeSlaveGroup.SelectedImageIndex = 0;
-      this.treeSlaveGroup.Size = new System.Drawing.Size(243, 511);
-      this.treeSlaveGroup.TabIndex = 0;
-      this.treeSlaveGroup.ItemDrag += new System.Windows.Forms.ItemDragEventHandler(this.treeSlaveGroup_ItemDrag);
-      this.treeSlaveGroup.AfterSelect += new System.Windows.Forms.TreeViewEventHandler(this.treeSlaveGroup_AfterSelect);
-      this.treeSlaveGroup.DragDrop += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragDrop);
-      this.treeSlaveGroup.DragEnter += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragEnter);
-      this.treeSlaveGroup.DragOver += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragOver);
-      this.treeSlaveGroup.QueryContinueDrag += new System.Windows.Forms.QueryContinueDragEventHandler(this.treeSlaveGroup_QueryContinueDrag);
+      // treeView
+      // 
+      this.treeView.AllowDrop = true;
+      this.treeView.Anchor = ((System.Windows.Forms.AnchorStyles)((((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Bottom) 
+            | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.treeView.ImageIndex = 0;
+      this.treeView.ImageList = this.imageListSlaveGroups;
+      this.treeView.Location = new System.Drawing.Point(3, 33);
+      this.treeView.Name = "treeView";
+      this.treeView.SelectedImageIndex = 0;
+      this.treeView.Size = new System.Drawing.Size(243, 511);
+      this.treeView.TabIndex = 0;
+      this.treeView.ItemDrag += new System.Windows.Forms.ItemDragEventHandler(this.treeSlaveGroup_ItemDrag);
+      this.treeView.AfterSelect += new System.Windows.Forms.TreeViewEventHandler(this.treeSlaveGroup_AfterSelect);
+      this.treeView.DragDrop += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragDrop);
+      this.treeView.DragEnter += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragEnter);
+      this.treeView.DragOver += new System.Windows.Forms.DragEventHandler(this.treeSlaveGroup_DragOver);
+      this.treeView.QueryContinueDrag += new System.Windows.Forms.QueryContinueDragEventHandler(this.treeSlaveGroup_QueryContinueDrag);
       // 
       // tabSlaveGroup
@@ -180 +175 @@
       this.tabSlaveGroup.Size = new System.Drawing.Size(585, 541);
       this.tabSlaveGroup.TabIndex = 1;
-      this.tabSlaveGroup.SelectedIndexChanged += new System.EventHandler(this.tabSlaveGroup_SelectedIndexChanged);
       // 
       // tabDetails
       // 
-      this.tabDetails.Controls.Add(this.slaveView);
+      this.tabDetails.Controls.Add(this.viewHost);
       this.tabDetails.Location = new System.Drawing.Point(4, 22);
       this.tabDetails.Name = "tabDetails";
@@ -192 +186 @@
       this.tabDetails.Text = "Details";
       this.tabDetails.UseVisualStyleBackColor = true;
+      // 
+      // viewHost
+      // 
+      this.viewHost.Anchor = ((System.Windows.Forms.AnchorStyles)((((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Bottom) 
+            | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.viewHost.Caption = "View";
+      this.viewHost.Content = null;
+      this.viewHost.Enabled = false;
+      this.viewHost.Location = new System.Drawing.Point(6, 6);
+      this.viewHost.Name = "viewHost";
+      this.viewHost.ReadOnly = false;
+      this.viewHost.Size = new System.Drawing.Size(565, 503);
+      this.viewHost.TabIndex = 0;
+      this.viewHost.ViewsLabelVisible = true;
+      this.viewHost.ViewType = null;
       // 
       // tabSchedule
@@ -203 +213 @@
       this.tabSchedule.Text = "Schedule";
       this.tabSchedule.UseVisualStyleBackColor = true;
-      // 
-      // slaveView
-      // 
-      this.slaveView.Anchor = ((System.Windows.Forms.AnchorStyles)((((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Bottom) 
-            | System.Windows.Forms.AnchorStyles.Left) 
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.slaveView.Caption = "SlaveView";
-      this.slaveView.Content = null;
-      this.slaveView.Location = new System.Drawing.Point(6, 6);
-      this.slaveView.Name = "slaveView";
-      this.slaveView.ReadOnly = false;
-      this.slaveView.Size = new System.Drawing.Size(565, 503);
-      this.slaveView.TabIndex = 0;
       // 
       // scheduleView
@@ -253 +250 @@
     private System.Windows.Forms.Button btnRemoveGroup;
     private System.Windows.Forms.Button btnAddGroup;
-    private System.Windows.Forms.TreeView treeSlaveGroup;
+    private System.Windows.Forms.TreeView treeView;
     private System.Windows.Forms.TabControl tabSlaveGroup;
     private System.Windows.Forms.TabPage tabDetails;
     private System.Windows.Forms.TabPage tabSchedule;
-    private SlaveView slaveView;
     private System.Windows.Forms.ImageList imageListSlaveGroups;
     private ScheduleView scheduleView;
@@ -263 +259 @@
     private System.Windows.Forms.Button btnRefresh;
     private System.Windows.Forms.ToolTip toolTip;
+    private MainForm.WindowsForms.ViewHost viewHost;
   }
 }

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/ResourcesView.cs

@@ -21 +21 @@
 
 using System;
+using System.Collections.Generic;
 using System.ComponentModel;
 using System.Drawing;
@@ -37 +38 @@
   [Content(typeof(IItemList<Resource>), false)]
   public partial class ResourcesView : ItemView, IDisposable {
-    private const string UngroupedGroupName = "UNGROUPED";
     private const int slaveImageIndex = 0;
     private const int slaveGroupImageIndex = 1;
+    public const string ungroupedGroupName = "UNGROUPED";
+    public const string ungroupedGroupDescription = "Contains slaves that are not assigned to any group.";
 
     private readonly Color ownedResourceColor = Color.LightGreen;
     private readonly object locker = new object();
-    private bool updatingResources = false;
-    private bool removingResources = false;
-    private bool savingResources = false;
 
     public new IItemList<Resource> Content {
@@ -55 +54 @@
       InitializeComponent();
 
-      treeSlaveGroup.ImageList.Images.Add(VSImageLibrary.MonitorLarge);
-      treeSlaveGroup.ImageList.Images.Add(VSImageLibrary.NetworkCenterLarge);
+      treeView.ImageList.Images.Add(VSImageLibrary.MonitorLarge);
+      treeView.ImageList.Images.Add(VSImageLibrary.NetworkCenterLarge);
 
       HiveAdminClient.Instance.Refreshing += HiveAdminClient_Instance_Refreshing;
@@ -88 +87 @@
       base.OnContentChanged();
       if (Content == null) {
-        slaveView.Content = null;
+        treeView.Nodes.Clear();
+        viewHost.Content = null;
         scheduleView.Content = null;
-        treeSlaveGroup.Nodes.Clear();
       } else {
-        treeSlaveGroup.Nodes.Clear();
-
-        //rebuild
-        TreeNode ungrp = new TreeNode(UngroupedGroupName);
-        ungrp.ImageIndex = slaveGroupImageIndex;
-        ungrp.SelectedImageIndex = ungrp.ImageIndex;
-        var newGroup = new SlaveGroup();
-        newGroup.Name = UngroupedGroupName;
-        newGroup.Id = Guid.NewGuid();
-        newGroup.Description = "Contains slaves which are in no group";
-        ungrp.Tag = newGroup;
-
-        foreach (Resource g in Content.OrderBy(x => x.Name)) {
-          if (g.GetType() == typeof(SlaveGroup)) {
-            //root node
-            if (g.ParentResourceId == null) {
-              TreeNode tn = new TreeNode();
-              tn.ImageIndex = slaveGroupImageIndex;
-              tn.SelectedImageIndex = tn.ImageIndex;
-
-              tn.Tag = g;
-              tn.Text = g.Name;
-              if (g.OwnerUserId == Access.UserInformation.Instance.User.Id) tn.BackColor = ownedResourceColor;
-
-              BuildSlaveGroupTree(g, tn);
-              treeSlaveGroup.Nodes.Add(tn);
-            }
-          } else if (g.GetType() == typeof(Slave)) {
-            if (g.ParentResourceId == null) {
-              var stn = new TreeNode(g.Name);
-              stn.ImageIndex = slaveImageIndex;
-              stn.SelectedImageIndex = stn.ImageIndex;
-              stn.Tag = g;
-              if (g.OwnerUserId == Access.UserInformation.Instance.User.Id) stn.BackColor = ownedResourceColor;
-              ungrp.Nodes.Add(stn);
-            }
-          }
-        }
-        treeSlaveGroup.Nodes.Add(ungrp);
+        var top = BuildResourceTree(Content);
+        viewHost.Content = top;
       }
     }
@@ -140 +102 @@
       btnRemoveGroup.Enabled = enabled;
       btnSave.Enabled = enabled;
-      scheduleView.SetEnabledStateOfSchedule(enabled && IsAuthorized(slaveView.Content));
+      scheduleView.SetEnabledStateOfSchedule(enabled && IsAuthorized((Resource)viewHost.Content));
     }
     #endregion
@@ -164 +126 @@
         OnContentChanged();
         if (e.PropertyName == "HbInterval") {
-          UpdateChildHbIntervall(slaveView.Content);
+          UpdateChildHbIntervall((Resource)viewHost.Content);
         }
       }
@@ -212 +174 @@
     private async void btnRefresh_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (updatingResources) return;
-        updatingResources = true;
+        if (!btnRefresh.Enabled) return;
+        btnRefresh.Enabled = false;
       }
 
       await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
         action: () => UpdateResources(),
-        finallyCallback: () => updatingResources = false);
+        finallyCallback: () => btnRefresh.Enabled = true);
     }
 
@@ -231 +193 @@
     private async void btnRemoveGroup_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (removingResources) return;
-        removingResources = true;
+        if (!btnRemoveGroup.Enabled) return;
+        btnRemoveGroup.Enabled = false;
       }
 
       await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
         action: () => RemoveResource(),
-        finallyCallback: () => removingResources = false);
+        finallyCallback: () => btnRemoveGroup.Enabled = true);
     }
 
     private async void btnSave_Click(object sender, EventArgs e) {
       lock (locker) {
-        if (savingResources) return;
-        savingResources = true;
+        if (!btnSave.Enabled) return;
+        btnSave.Enabled = false;
       }
 
@@ -252 +214 @@
             resource.Store();
         },
-        finallyCallback: () => savingResources = false);
+        finallyCallback: () => btnSave.Enabled = true);
     }
 
@@ -258 +220 @@
       var selectedResource = (Resource)e.Node.Tag;
 
-      if (slaveView.Content != null && slaveView.Content is SlaveGroup)
-        slaveView.Content.PropertyChanged -= SlaveViewContent_PropertyChanged;
-
-      slaveView.Content = selectedResource;
+      if (viewHost.Content != null && viewHost.Content is SlaveGroup)
+        ((SlaveGroup)viewHost.Content).PropertyChanged -= SlaveViewContent_PropertyChanged;
+
+      viewHost.Content = selectedResource;
+      HiveAdminClient.Instance.DowntimeForResourceId = selectedResource != null ? selectedResource.Id : Guid.Empty;
+      await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
+        action: () => UpdateSchedule(),
+        finallyCallback: () => scheduleView.Content = HiveAdminClient.Instance.Downtimes);
 
       if (selectedResource != null && selectedResource is SlaveGroup)
@@ -269 +235 @@
         if (!tabSlaveGroup.TabPages.Contains(tabSchedule))
           tabSlaveGroup.TabPages.Add(tabSchedule);
-
-        HiveAdminClient.Instance.DowntimeForResourceId = selectedResource.Id;
-
-        if (tabSlaveGroup.SelectedIndex == 1) {
-          await SecurityExceptionUtil.TryAsyncAndReportSecurityExceptions(
-            action: () => UpdateSchedule(),
-            finallyCallback: () => scheduleView.Content = HiveAdminClient.Instance.Downtimes);
-        }
       } else {
         if (tabSlaveGroup.TabPages.Contains(tabSchedule))
           tabSlaveGroup.TabPages.Remove(tabSchedule);
-        HiveAdminClient.Instance.DowntimeForResourceId = Guid.Empty;
       }
     }
 
     private void treeSlaveGroup_DragDrop(object sender, DragEventArgs e) {
-      if (e.Data.GetDataPresent("System.Windows.Forms.TreeNode", false)) {
-        Point pt = ((TreeView)sender).PointToClient(new Point(e.X, e.Y));
-        TreeNode destNode = ((TreeView)sender).GetNodeAt(pt);
-        TreeNode newNode = (TreeNode)e.Data.GetData("System.Windows.Forms.TreeNode");
-
-        if (destNode.TreeView == newNode.TreeView) {
-          if (destNode.Text == UngroupedGroupName || (destNode.Parent != null && destNode.Parent.Text == UngroupedGroupName)) {
-            MessageBox.Show(string.Format("You can't drag items to the group \"{0}\".{1}This group only contains slaves which haven't yet been assigned to a real group.",
-              UngroupedGroupName, Environment.NewLine), "HeuristicLab Hive Administrator", MessageBoxButtons.OK, MessageBoxIcon.Information);
-            return;
-          }
-
-          SlaveGroup sgrp = null;
-          TreeNode parentNode = null;
-          if (destNode.Tag != null && destNode.Tag is SlaveGroup) {
-            sgrp = (SlaveGroup)destNode.Tag;
-            parentNode = destNode;
-          } else if (destNode.Parent != null && destNode.Parent.Tag is SlaveGroup) {
-            sgrp = (SlaveGroup)destNode.Parent.Tag;
-            parentNode = destNode.Parent;
-          }
-
-          if (newNode.Tag is SlaveGroup && CheckParentsEqualsMovedNode(parentNode, newNode)) {
-            return;
-          }
-
-          SlaveGroup parent = (SlaveGroup)parentNode.Tag;
-
-          if (parent.OwnerUserId != null && !IsAuthorized(parent)) {
-            MessageBox.Show(string.Format("You don't have the permissions to drag items to the group \"{0}\".", ((Resource)parentNode.Tag).Name),
-              "HeuristicLab Hive Administrator", MessageBoxButtons.OK, MessageBoxIcon.Error);
-            return;
-          }
-
-          if (sgrp != null && newNode.Tag != null) {
-            //save parent group to get an id
-            if (sgrp.Id == Guid.Empty) {
-              sgrp.Store();
-            }
-
-            if (newNode.Tag is Slave) {
-              Slave slave = (Slave)newNode.Tag;
-              if (slave.ParentResourceId == null || (slave.ParentResourceId != null && slave.ParentResourceId != sgrp.Id)) {
-                slave.ParentResourceId = sgrp.Id;
-                newNode.Remove();
-                parentNode.Nodes.Clear();
-                BuildSlaveGroupTree(sgrp, parentNode);
-              }
-            } else if (newNode.Tag is SlaveGroup) {
-              SlaveGroup slaveGroup = (SlaveGroup)newNode.Tag;
-              if (slaveGroup.ParentResourceId == null || (slaveGroup.ParentResourceId != null && slaveGroup.ParentResourceId != sgrp.Id)) {
-                slaveGroup.ParentResourceId = sgrp.Id;
-                newNode.Remove();
-                parentNode.Nodes.Clear();
-                BuildSlaveGroupTree(sgrp, parentNode);
-              }
-            }
-          }
-        }
+      var sourceNode = (TreeNode)e.Data.GetData(typeof(TreeNode));
+      if (sourceNode == null) return;
+
+      var treeView = (TreeView)sender;
+      if (sourceNode.TreeView != treeView) return;
+
+      var targetPoint = treeView.PointToClient(new Point(e.X, e.Y));
+      var targetNode = treeView.GetNodeAt(targetPoint);
+
+      if (sourceNode == targetNode) return;
+
+      if (targetNode.Text == ungroupedGroupName || targetNode.Parent != null && targetNode.Parent.Text == ungroupedGroupName) {
+        MessageBox.Show(
+          string.Format(@"You cannot drag resources to group ""{0}"". This group only contains slaves which have not been assigned to a real group yet.", ungroupedGroupName),
+          "HeuristicLab Hive Administrator",
+          MessageBoxButtons.OK,
+          MessageBoxIcon.Information);
+        return;
+      }
+
+      if (sourceNode.Parent == null)
+        treeView.Nodes.Remove(sourceNode);
+      else {
+        sourceNode.Parent.Nodes.Remove(sourceNode);
+        ((Resource)sourceNode.Tag).ParentResourceId = null;
+      }
+
+      if (targetNode == null) {
+        treeView.Nodes.Add(sourceNode);
+      } else {
+        targetNode.Nodes.Add(sourceNode);
+        ((Resource)sourceNode.Tag).ParentResourceId = ((Project)targetNode.Tag).Id;
       }
     }
@@ -364 +294 @@
       e.Action = DragAction.Continue;
     }
-
-    private void tabSlaveGroup_SelectedIndexChanged(object sender, EventArgs e) {
-      if (tabSlaveGroup.SelectedIndex == 1) {
-        UpdateSchedule();
-      }
-    }
     #endregion
 
     #region Helpers
-    private void BuildSlaveGroupTree(Resource g, TreeNode tn) {
-      foreach (Resource r in Content.Where(s => s.ParentResourceId != null && s.ParentResourceId == g.Id).OrderBy(x => x.Name)) {
-        TreeNode stn = new TreeNode(r.Name);
-        if (r is Slave) {
-          stn.ImageIndex = slaveImageIndex;
-        } else if (r is SlaveGroup) {
-          stn.ImageIndex = slaveGroupImageIndex;
-        }
-        stn.SelectedImageIndex = stn.ImageIndex;
-        stn.Tag = r;
-        if (r.OwnerUserId == Access.UserInformation.Instance.User.Id) stn.BackColor = ownedResourceColor;
-        tn.Nodes.Add(stn);
-
-        BuildSlaveGroupTree(r, stn);
-      }
+    private Resource BuildResourceTree(IEnumerable<Resource> resources) {
+      treeView.Nodes.Clear();
+      if (!resources.Any()) return null;
+
+      var slaveGroups = new HashSet<SlaveGroup>(resources.OfType<SlaveGroup>());
+      var slaves = new HashSet<Slave>(resources.OfType<Slave>());
+
+      var stack = new Stack<Resource>(slaveGroups.OrderByDescending(x => x.Name));
+      var top = stack.Peek();
+
+      TreeNode currentNode = null;
+      Resource currentResource = null;
+
+      while (stack.Any()) {
+        var newResource = stack.Pop();
+        var newNode = new TreeNode(newResource.Name) { Tag = newResource };
+
+        while (currentNode != null && newResource.ParentResourceId != currentResource.Id) {
+          currentNode = currentNode.Parent;
+          currentResource = currentNode == null ? null : (Resource)currentNode.Tag;
+        }
+
+        if (currentNode == null) {
+          treeView.Nodes.Add(newNode);
+        } else {
+          currentNode.Nodes.Add(newNode);
+        }
+
+        if (newResource is Slave) {
+          newNode.ImageIndex = slaveImageIndex;
+        } else {
+          newNode.ImageIndex = slaveGroupImageIndex;
+
+          var childSlaves = slaves.Where(x => x.ParentResourceId == newResource.Id);
+          foreach (var slave in childSlaves.OrderBy(x => x.Name)) {
+            slaves.Remove(slave);
+            var slaveNode = new TreeNode(slave.Name) { Tag = slave };
+
+            slaveNode.SelectedImageIndex = slaveNode.ImageIndex = slaveImageIndex;
+            if (slave.OwnerUserId == UserInformation.Instance.User.Id)
+              slaveNode.BackColor = ownedResourceColor;
+
+            newNode.Nodes.Add(slaveNode);
+          }
+
+          var childSlaveGroups = slaveGroups.Where(x => x.ParentResourceId == newResource.Id);
+          if (childSlaveGroups.Any()) {
+            foreach (var slaveGroup in childSlaveGroups.OrderByDescending(x => x.Name)) {
+              slaveGroups.Remove(slaveGroup);
+              stack.Push(slaveGroup);
+            }
+            currentNode = newNode;
+            currentResource = newResource;
+          }
+        }
+
+        newNode.SelectedImageIndex = newNode.ImageIndex;
+        if (newResource.OwnerUserId == UserInformation.Instance.User.Id)
+          newNode.BackColor = ownedResourceColor;
+      }
+
+      var ungroupedNode = new TreeNode(ungroupedGroupName) {
+        ForeColor = SystemColors.GrayText,
+        Tag = new SlaveGroup() {
+          Name = ungroupedGroupName,
+          Description = ungroupedGroupDescription
+        }
+      };
+
+      foreach (var slave in slaves.OrderBy(x => x.Name)) {
+        var slaveNode = new TreeNode(slave.Name) { Tag = slave };
+        ungroupedNode.Nodes.Add(slaveNode);
+      }
+
+      treeView.Nodes.Add(ungroupedNode);
+      treeView.ExpandAll();
+
+      return top;
     }
 
@@ -414 +401 @@
       if (InvokeRequired) Invoke((Action)ResetView);
       else {
-        treeSlaveGroup.Nodes.Clear();
-
-        if (slaveView.Content != null && slaveView.Content is SlaveGroup) {
-          slaveView.Content.PropertyChanged -= SlaveViewContent_PropertyChanged;
-        }
-
-        slaveView.Content = null;
+        treeView.Nodes.Clear();
+
+        if (viewHost.Content != null && viewHost.Content is SlaveGroup) {
+          ((SlaveGroup)viewHost.Content).PropertyChanged -= SlaveViewContent_PropertyChanged;
+        }
+
+        viewHost.Content = null;
         if (scheduleView.Content != null) {
           scheduleView.Content.Clear();
@@ -435 +422 @@
 
     private void RemoveResource() {
-      var selectedNode = treeSlaveGroup.SelectedNode;
+      var selectedNode = treeView.SelectedNode;
       if (selectedNode == null || selectedNode.Tag == null) return;
 
@@ -474 +461 @@
     private bool IsAuthorized(Resource resource) {
       return resource != null
-          && resource.Name != UngroupedGroupName
+          && resource.Name != ungroupedGroupName
           && resource.Id != Guid.Empty
           && UserInformation.Instance.UserExists

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/SlaveView.Designer.cs

@@ -45 +45 @@
     /// </summary>
     private void InitializeComponent() {
-      this.txtSlaveState = new System.Windows.Forms.TextBox();
-      this.txtOS = new System.Windows.Forms.TextBox();
-      this.label15 = new System.Windows.Forms.Label();
-      this.label14 = new System.Windows.Forms.Label();
-      this.txtDetailsDescription = new System.Windows.Forms.TextBox();
-      this.label13 = new System.Windows.Forms.Label();
-      this.txtName = new System.Windows.Forms.TextBox();
-      this.txtCPU = new System.Windows.Forms.TextBox();
-      this.txtMemory = new System.Windows.Forms.TextBox();
-      this.txtLastHeartbeat = new System.Windows.Forms.TextBox();
-      this.label12 = new System.Windows.Forms.Label();
-      this.label11 = new System.Windows.Forms.Label();
-      this.label10 = new System.Windows.Forms.Label();
-      this.label9 = new System.Windows.Forms.Label();
-      this.label1 = new System.Windows.Forms.Label();
-      this.txtFreeMemory = new System.Windows.Forms.TextBox();
-      this.txtId = new System.Windows.Forms.TextBox();
-      this.label2 = new System.Windows.Forms.Label();
-      this.label3 = new System.Windows.Forms.Label();
-      this.txtHbIntervall = new System.Windows.Forms.TextBox();
-      this.cbxDisposable = new System.Windows.Forms.CheckBox();
-      this.label4 = new System.Windows.Forms.Label();
-      this.cbxPublic = new System.Windows.Forms.CheckBox();
-      this.label5 = new System.Windows.Forms.Label();
+      this.cpuLabel = new System.Windows.Forms.Label();
+      this.cpuTextBox = new System.Windows.Forms.TextBox();
+      this.memoryLabel = new System.Windows.Forms.Label();
+      this.memoryTextBox = new System.Windows.Forms.TextBox();
+      this.operatingSystemLabel = new System.Windows.Forms.Label();
+      this.operatingSystemTextBox = new System.Windows.Forms.TextBox();
+      this.stateLabel = new System.Windows.Forms.Label();
+      this.stateTextBox = new System.Windows.Forms.TextBox();
+      this.lastHeartbeatLabel = new System.Windows.Forms.Label();
+      this.lastHeartbeatTextBox = new System.Windows.Forms.TextBox();
+      this.disposableLabel = new System.Windows.Forms.Label();
+      this.disposableCheckBox = new System.Windows.Forms.CheckBox();
+      ((System.ComponentModel.ISupportInitialize)(this.heartbeatIntervalNumericUpDown)).BeginInit();
       this.SuspendLayout();
       // 
-      // txtSlaveState
-      // 
-      this.txtSlaveState.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtSlaveState.Enabled = false;
-      this.txtSlaveState.Location = new System.Drawing.Point(146, 242);
-      this.txtSlaveState.Name = "txtSlaveState";
-      this.txtSlaveState.Size = new System.Drawing.Size(390, 20);
-      this.txtSlaveState.TabIndex = 27;
-      // 
-      // txtOS
-      // 
-      this.txtOS.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtOS.Enabled = false;
-      this.txtOS.Location = new System.Drawing.Point(146, 216);
-      this.txtOS.Name = "txtOS";
-      this.txtOS.Size = new System.Drawing.Size(390, 20);
-      this.txtOS.TabIndex = 26;
-      // 
-      // label15
-      // 
-      this.label15.AutoSize = true;
-      this.label15.Location = new System.Drawing.Point(3, 245);
-      this.label15.Name = "label15";
-      this.label15.Size = new System.Drawing.Size(35, 13);
-      this.label15.TabIndex = 25;
-      this.label15.Text = "State:";
-      // 
-      // label14
-      // 
-      this.label14.AutoSize = true;
-      this.label14.Location = new System.Drawing.Point(3, 219);
-      this.label14.Name = "label14";
-      this.label14.Size = new System.Drawing.Size(93, 13);
-      this.label14.TabIndex = 24;
-      this.label14.Text = "Operating System:";
-      // 
-      // txtDetailsDescription
-      // 
-      this.txtDetailsDescription.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtDetailsDescription.Enabled = false;
-      this.txtDetailsDescription.Location = new System.Drawing.Point(146, 112);
-      this.txtDetailsDescription.Name = "txtDetailsDescription";
-      this.txtDetailsDescription.Size = new System.Drawing.Size(390, 20);
-      this.txtDetailsDescription.TabIndex = 23;
-      // 
-      // label13
-      // 
-      this.label13.AutoSize = true;
-      this.label13.Location = new System.Drawing.Point(3, 115);
-      this.label13.Name = "label13";
-      this.label13.Size = new System.Drawing.Size(63, 13);
-      this.label13.TabIndex = 22;
-      this.label13.Text = "Description:";
-      // 
-      // txtName
-      // 
-      this.txtName.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtName.Location = new System.Drawing.Point(146, 8);
-      this.txtName.Name = "txtName";
-      this.txtName.Size = new System.Drawing.Size(390, 20);
-      this.txtName.TabIndex = 21;
-      this.txtName.TextChanged += new System.EventHandler(this.txtName_TextChanged);
-      // 
-      // txtCPU
-      // 
-      this.txtCPU.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtCPU.Enabled = false;
-      this.txtCPU.Location = new System.Drawing.Point(146, 138);
-      this.txtCPU.Name = "txtCPU";
-      this.txtCPU.Size = new System.Drawing.Size(390, 20);
-      this.txtCPU.TabIndex = 20;
-      // 
-      // txtMemory
-      // 
-      this.txtMemory.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtMemory.Enabled = false;
-      this.txtMemory.Location = new System.Drawing.Point(146, 164);
-      this.txtMemory.Name = "txtMemory";
-      this.txtMemory.Size = new System.Drawing.Size(390, 20);
-      this.txtMemory.TabIndex = 19;
-      // 
-      // txtLastHeartbeat
-      // 
-      this.txtLastHeartbeat.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtLastHeartbeat.Enabled = false;
-      this.txtLastHeartbeat.Location = new System.Drawing.Point(146, 268);
-      this.txtLastHeartbeat.Name = "txtLastHeartbeat";
-      this.txtLastHeartbeat.Size = new System.Drawing.Size(390, 20);
-      this.txtLastHeartbeat.TabIndex = 18;
-      // 
-      // label12
-      // 
-      this.label12.AutoSize = true;
-      this.label12.Location = new System.Drawing.Point(3, 271);
-      this.label12.Name = "label12";
-      this.label12.Size = new System.Drawing.Size(80, 13);
-      this.label12.TabIndex = 17;
-      this.label12.Text = "Last Heartbeat:";
-      // 
-      // label11
-      // 
-      this.label11.AutoSize = true;
-      this.label11.Location = new System.Drawing.Point(3, 167);
-      this.label11.Name = "label11";
-      this.label11.Size = new System.Drawing.Size(47, 13);
-      this.label11.TabIndex = 16;
-      this.label11.Text = "Memory:";
-      // 
-      // label10
-      // 
-      this.label10.AutoSize = true;
-      this.label10.Location = new System.Drawing.Point(3, 141);
-      this.label10.Name = "label10";
-      this.label10.Size = new System.Drawing.Size(32, 13);
-      this.label10.TabIndex = 15;
-      this.label10.Text = "CPU:";
-      // 
-      // label9
-      // 
-      this.label9.AutoSize = true;
-      this.label9.Location = new System.Drawing.Point(3, 11);
-      this.label9.Name = "label9";
-      this.label9.Size = new System.Drawing.Size(38, 13);
-      this.label9.TabIndex = 14;
-      this.label9.Text = "Name:";
-      // 
-      // label1
-      // 
-      this.label1.AutoSize = true;
-      this.label1.Location = new System.Drawing.Point(3, 193);
-      this.label1.Name = "label1";
-      this.label1.Size = new System.Drawing.Size(70, 13);
-      this.label1.TabIndex = 28;
-      this.label1.Text = "Free memory:";
-      // 
-      // txtFreeMemory
-      // 
-      this.txtFreeMemory.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtFreeMemory.Enabled = false;
-      this.txtFreeMemory.Location = new System.Drawing.Point(146, 190);
-      this.txtFreeMemory.Name = "txtFreeMemory";
-      this.txtFreeMemory.Size = new System.Drawing.Size(390, 20);
-      this.txtFreeMemory.TabIndex = 29;
-      // 
-      // txtId
-      // 
-      this.txtId.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtId.Enabled = false;
-      this.txtId.Location = new System.Drawing.Point(146, 86);
-      this.txtId.Name = "txtId";
-      this.txtId.Size = new System.Drawing.Size(390, 20);
-      this.txtId.TabIndex = 30;
-      // 
-      // label2
-      // 
-      this.label2.AutoSize = true;
-      this.label2.Location = new System.Drawing.Point(3, 89);
-      this.label2.Name = "label2";
-      this.label2.Size = new System.Drawing.Size(19, 13);
-      this.label2.TabIndex = 31;
-      this.label2.Text = "Id:";
-      // 
-      // label3
-      // 
-      this.label3.AutoSize = true;
-      this.label3.Location = new System.Drawing.Point(3, 37);
-      this.label3.Name = "label3";
-      this.label3.Size = new System.Drawing.Size(137, 13);
-      this.label3.TabIndex = 32;
-      this.label3.Text = "Heartbeat Intervall (in sec) :";
-      // 
-      // txtHbIntervall
-      // 
-      this.txtHbIntervall.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left)
-            | System.Windows.Forms.AnchorStyles.Right)));
-      this.txtHbIntervall.Location = new System.Drawing.Point(146, 34);
-      this.txtHbIntervall.Name = "txtHbIntervall";
-      this.txtHbIntervall.Size = new System.Drawing.Size(390, 20);
-      this.txtHbIntervall.TabIndex = 33;
-      this.txtHbIntervall.TextChanged += new System.EventHandler(this.txtHbIntervall_TextChanged);
-      // 
-      // cbxDisposable
-      // 
-      this.cbxDisposable.AutoSize = true;
-      this.cbxDisposable.Enabled = false;
-      this.cbxDisposable.Location = new System.Drawing.Point(146, 297);
-      this.cbxDisposable.Name = "cbxDisposable";
-      this.cbxDisposable.Size = new System.Drawing.Size(15, 14);
-      this.cbxDisposable.TabIndex = 34;
-      this.cbxDisposable.UseVisualStyleBackColor = true;
-      this.cbxDisposable.CheckedChanged += new System.EventHandler(this.cbxDisposable_CheckedChanged);
-      // 
-      // label4
-      // 
-      this.label4.AutoSize = true;
-      this.label4.Location = new System.Drawing.Point(3, 297);
-      this.label4.Name = "label4";
-      this.label4.Size = new System.Drawing.Size(62, 13);
-      this.label4.TabIndex = 35;
-      this.label4.Text = "Disposable:";
-      // 
-      // cbxPublic
-      // 
-      this.cbxPublic.AutoSize = true;
-      this.cbxPublic.Enabled = false;
-      this.cbxPublic.Location = new System.Drawing.Point(146, 63);
-      this.cbxPublic.Name = "cbxPublic";
-      this.cbxPublic.Size = new System.Drawing.Size(15, 14);
-      this.cbxPublic.TabIndex = 36;
-      this.cbxPublic.UseVisualStyleBackColor = true;
-      this.cbxPublic.CheckedChanged += new System.EventHandler(this.cbxPublic_CheckedChanged);
-      // 
-      // label5
-      // 
-      this.label5.AutoSize = true;
-      this.label5.Location = new System.Drawing.Point(3, 63);
-      this.label5.Name = "label5";
-      this.label5.Size = new System.Drawing.Size(39, 13);
-      this.label5.TabIndex = 37;
-      this.label5.Text = "Public:";
+      // nameTextBox
+      // 
+      this.nameTextBox.ReadOnly = true;
+      // 
+      // cpuLabel
+      // 
+      this.cpuLabel.AutoSize = true;
+      this.cpuLabel.Location = new System.Drawing.Point(3, 141);
+      this.cpuLabel.Name = "cpuLabel";
+      this.cpuLabel.Size = new System.Drawing.Size(32, 13);
+      this.cpuLabel.TabIndex = 16;
+      this.cpuLabel.Text = "CPU:";
+      // 
+      // cpuTextBox
+      // 
+      this.cpuTextBox.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.cpuTextBox.Location = new System.Drawing.Point(130, 138);
+      this.cpuTextBox.Name = "cpuTextBox";
+      this.cpuTextBox.ReadOnly = true;
+      this.cpuTextBox.Size = new System.Drawing.Size(397, 20);
+      this.cpuTextBox.TabIndex = 17;
+      // 
+      // memoryLabel
+      // 
+      this.memoryLabel.AutoSize = true;
+      this.memoryLabel.Location = new System.Drawing.Point(3, 167);
+      this.memoryLabel.Name = "memoryLabel";
+      this.memoryLabel.Size = new System.Drawing.Size(47, 13);
+      this.memoryLabel.TabIndex = 18;
+      this.memoryLabel.Text = "Memory:";
+      // 
+      // memoryTextBox
+      // 
+      this.memoryTextBox.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.memoryTextBox.Location = new System.Drawing.Point(130, 164);
+      this.memoryTextBox.Name = "memoryTextBox";
+      this.memoryTextBox.ReadOnly = true;
+      this.memoryTextBox.Size = new System.Drawing.Size(397, 20);
+      this.memoryTextBox.TabIndex = 19;
+      // 
+      // operatingSystemLabel
+      // 
+      this.operatingSystemLabel.AutoSize = true;
+      this.operatingSystemLabel.Location = new System.Drawing.Point(3, 193);
+      this.operatingSystemLabel.Name = "operatingSystemLabel";
+      this.operatingSystemLabel.Size = new System.Drawing.Size(93, 13);
+      this.operatingSystemLabel.TabIndex = 20;
+      this.operatingSystemLabel.Text = "Operating System:";
+      // 
+      // operatingSystemTextBox
+      // 
+      this.operatingSystemTextBox.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.operatingSystemTextBox.Location = new System.Drawing.Point(130, 190);
+      this.operatingSystemTextBox.Name = "operatingSystemTextBox";
+      this.operatingSystemTextBox.ReadOnly = true;
+      this.operatingSystemTextBox.Size = new System.Drawing.Size(397, 20);
+      this.operatingSystemTextBox.TabIndex = 21;
+      // 
+      // stateLabel
+      // 
+      this.stateLabel.AutoSize = true;
+      this.stateLabel.Location = new System.Drawing.Point(3, 219);
+      this.stateLabel.Name = "stateLabel";
+      this.stateLabel.Size = new System.Drawing.Size(35, 13);
+      this.stateLabel.TabIndex = 22;
+      this.stateLabel.Text = "State:";
+      // 
+      // stateTextBox
+      // 
+      this.stateTextBox.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.stateTextBox.Location = new System.Drawing.Point(130, 216);
+      this.stateTextBox.Name = "stateTextBox";
+      this.stateTextBox.ReadOnly = true;
+      this.stateTextBox.Size = new System.Drawing.Size(397, 20);
+      this.stateTextBox.TabIndex = 23;
+      // 
+      // lastHeartbeatLabel
+      // 
+      this.lastHeartbeatLabel.AutoSize = true;
+      this.lastHeartbeatLabel.Location = new System.Drawing.Point(3, 245);
+      this.lastHeartbeatLabel.Name = "lastHeartbeatLabel";
+      this.lastHeartbeatLabel.Size = new System.Drawing.Size(80, 13);
+      this.lastHeartbeatLabel.TabIndex = 24;
+      this.lastHeartbeatLabel.Text = "Last Heartbeat:";
+      // 
+      // lastHeartbeatTextBox
+      // 
+      this.lastHeartbeatTextBox.Anchor = ((System.Windows.Forms.AnchorStyles)(((System.Windows.Forms.AnchorStyles.Top | System.Windows.Forms.AnchorStyles.Left) 
+            | System.Windows.Forms.AnchorStyles.Right)));
+      this.lastHeartbeatTextBox.Location = new System.Drawing.Point(130, 242);
+      this.lastHeartbeatTextBox.Name = "lastHeartbeatTextBox";
+      this.lastHeartbeatTextBox.ReadOnly = true;
+      this.lastHeartbeatTextBox.Size = new System.Drawing.Size(397, 20);
+      this.lastHeartbeatTextBox.TabIndex = 25;
+      // 
+      // disposableLabel
+      // 
+      this.disposableLabel.AutoSize = true;
+      this.disposableLabel.Location = new System.Drawing.Point(3, 271);
+      this.disposableLabel.Name = "disposableLabel";
+      this.disposableLabel.Size = new System.Drawing.Size(62, 13);
+      this.disposableLabel.TabIndex = 28;
+      this.disposableLabel.Text = "Disposable:";
+      // 
+      // disposableCheckBox
+      // 
+      this.disposableCheckBox.AutoSize = true;
+      this.disposableCheckBox.Location = new System.Drawing.Point(130, 270);
+      this.disposableCheckBox.Name = "disposableCheckBox";
+      this.disposableCheckBox.Size = new System.Drawing.Size(15, 14);
+      this.disposableCheckBox.TabIndex = 29;
+      this.disposableCheckBox.UseVisualStyleBackColor = true;
+      this.disposableCheckBox.CheckedChanged += new System.EventHandler(this.disposableCheckBox_CheckedChanged);
       // 
       // SlaveView
       // 
-      this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 13F);
       this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Inherit;
-      this.Controls.Add(this.label5);
-      this.Controls.Add(this.cbxPublic);
-      this.Controls.Add(this.label4);
-      this.Controls.Add(this.cbxDisposable);
-      this.Controls.Add(this.txtHbIntervall);
-      this.Controls.Add(this.label3);
-      this.Controls.Add(this.label2);
-      this.Controls.Add(this.txtId);
-      this.Controls.Add(this.txtFreeMemory);
-      this.Controls.Add(this.label1);
-      this.Controls.Add(this.txtSlaveState);
-      this.Controls.Add(this.txtOS);
-      this.Controls.Add(this.label15);
-      this.Controls.Add(this.label14);
-      this.Controls.Add(this.txtDetailsDescription);
-      this.Controls.Add(this.label13);
-      this.Controls.Add(this.txtName);
-      this.Controls.Add(this.txtCPU);
-      this.Controls.Add(this.txtMemory);
-      this.Controls.Add(this.txtLastHeartbeat);
-      this.Controls.Add(this.label12);
-      this.Controls.Add(this.label11);
-      this.Controls.Add(this.label10);
-      this.Controls.Add(this.label9);
+      this.Controls.Add(this.disposableCheckBox);
+      this.Controls.Add(this.disposableLabel);
+      this.Controls.Add(this.lastHeartbeatTextBox);
+      this.Controls.Add(this.lastHeartbeatLabel);
+      this.Controls.Add(this.stateTextBox);
+      this.Controls.Add(this.stateLabel);
+      this.Controls.Add(this.operatingSystemTextBox);
+      this.Controls.Add(this.operatingSystemLabel);
+      this.Controls.Add(this.memoryTextBox);
+      this.Controls.Add(this.memoryLabel);
+      this.Controls.Add(this.cpuTextBox);
+      this.Controls.Add(this.cpuLabel);
       this.Name = "SlaveView";
-      this.Size = new System.Drawing.Size(539, 407);
+      this.Controls.SetChildIndex(this.idTextBox, 0);
+      this.Controls.SetChildIndex(this.descriptionTextBox, 0);
+      this.Controls.SetChildIndex(this.heartbeatIntervalNumericUpDown, 0);
+      this.Controls.SetChildIndex(this.publicCheckBox, 0);
+      this.Controls.SetChildIndex(this.nameTextBox, 0);
+      this.Controls.SetChildIndex(this.cpuLabel, 0);
+      this.Controls.SetChildIndex(this.cpuTextBox, 0);
+      this.Controls.SetChildIndex(this.memoryLabel, 0);
+      this.Controls.SetChildIndex(this.memoryTextBox, 0);
+      this.Controls.SetChildIndex(this.operatingSystemLabel, 0);
+      this.Controls.SetChildIndex(this.operatingSystemTextBox, 0);
+      this.Controls.SetChildIndex(this.stateLabel, 0);
+      this.Controls.SetChildIndex(this.stateTextBox, 0);
+      this.Controls.SetChildIndex(this.lastHeartbeatLabel, 0);
+      this.Controls.SetChildIndex(this.lastHeartbeatTextBox, 0);
+      this.Controls.SetChildIndex(this.disposableLabel, 0);
+      this.Controls.SetChildIndex(this.disposableCheckBox, 0);
+      ((System.ComponentModel.ISupportInitialize)(this.heartbeatIntervalNumericUpDown)).EndInit();
       this.ResumeLayout(false);
       this.PerformLayout();
@@ -338 +219 @@
     #endregion
 
-    private System.Windows.Forms.TextBox txtSlaveState;
-    private System.Windows.Forms.TextBox txtOS;
-    private System.Windows.Forms.Label label15;
-    private System.Windows.Forms.Label label14;
-    private System.Windows.Forms.TextBox txtDetailsDescription;
-    private System.Windows.Forms.Label label13;
-    private System.Windows.Forms.TextBox txtName;
-    private System.Windows.Forms.TextBox txtCPU;
-    private System.Windows.Forms.TextBox txtMemory;
-    private System.Windows.Forms.TextBox txtLastHeartbeat;
-    private System.Windows.Forms.Label label12;
-    private System.Windows.Forms.Label label11;
-    private System.Windows.Forms.Label label10;
-    private System.Windows.Forms.Label label9;
-    private System.Windows.Forms.Label label1;
-    private System.Windows.Forms.TextBox txtFreeMemory;
-    private System.Windows.Forms.TextBox txtId;
-    private System.Windows.Forms.Label label2;
-    private System.Windows.Forms.Label label3;
-    private System.Windows.Forms.TextBox txtHbIntervall;
-    private System.Windows.Forms.CheckBox cbxDisposable;
-    private System.Windows.Forms.Label label4;
-    private System.Windows.Forms.CheckBox cbxPublic;
-    private System.Windows.Forms.Label label5;
+    private System.Windows.Forms.Label cpuLabel;
+    private System.Windows.Forms.TextBox cpuTextBox;
+    private System.Windows.Forms.Label memoryLabel;
+    private System.Windows.Forms.TextBox memoryTextBox;
+    private System.Windows.Forms.Label operatingSystemLabel;
+    private System.Windows.Forms.TextBox operatingSystemTextBox;
+    private System.Windows.Forms.Label stateLabel;
+    private System.Windows.Forms.TextBox stateTextBox;
+    private System.Windows.Forms.Label lastHeartbeatLabel;
+    private System.Windows.Forms.TextBox lastHeartbeatTextBox;
+    private System.Windows.Forms.Label disposableLabel;
+    private System.Windows.Forms.CheckBox disposableCheckBox;
   }
 }

branches/HiveProjectManagement/HeuristicLab.Clients.Hive.Administrator/3.3/Views/SlaveView.cs

@@ -1 +1 @@
 #region License Information
 /* HeuristicLab
- * Copyright (C) 2002-2016 Heuristic and Evolutionary Algorithms Laboratory (HEAL)
+ * Copyright (C) 2002-2017 Heuristic and Evolutionary Algorithms Laboratory (HEAL)
  *
  * This file is part of HeuristicLab.
@@ -22 +22 @@
 using System;
 using System.Windows.Forms;
-using HeuristicLab.Clients.Access;
-using HeuristicLab.Core.Views;
 using HeuristicLab.MainForm;
 using HeuristicLab.MainForm.WindowsForms;
 
 namespace HeuristicLab.Clients.Hive.Administrator.Views {
-  [View("SlaveView")]
-  [Content(typeof(Resource), IsDefaultView = true)]
-  public partial class SlaveView : ItemView {
-    public new Resource Content {
-      get { return (Resource)base.Content; }
+  [View("Slave View")]
+  [Content(typeof(Slave), IsDefaultView = true)]
+  public sealed partial class SlaveView : ResourceView {
+    public new Slave Content {
+      get { return (Slave)base.Content; }
       set { base.Content = value; }
     }
@@ -40 +38 @@
     }
 
-    #region Register Content Events
-    protected override void DeregisterContentEvents() {
-      base.DeregisterContentEvents();
-    }
-    protected override void RegisterContentEvents() {
-      base.RegisterContentEvents();
-    }
-    #endregion
-
+    #region Overrides
     protected override void OnContentChanged() {
       base.OnContentChanged();
       if (Content == null) {
-        ShowSlaveUI(true);
-        txtName.Clear();
-        txtCPU.Clear();
-        txtDetailsDescription.Clear();
-        txtMemory.Clear();
-        txtOS.Clear();
-        txtSlaveState.Clear();
-        txtLastHeartbeat.Clear();
-        txtFreeMemory.Clear();
-        txtId.Clear();
-        txtHbIntervall.Clear();
-        cbxDisposable.Checked = false;
-        cbxPublic.Checked = false;
+        cpuTextBox.Clear();
+        memoryTextBox.Clear();
+        operatingSystemTextBox.Clear();
+        stateTextBox.Clear();
+        lastHeartbeatTextBox.Clear();
+        disposableCheckBox.Checked = false;
       } else {
-        if (Content.GetType() == typeof(Slave)) {
-          ShowSlaveUI(true);
-          Slave ct = (Slave)Content;
-          bool authorized = UserInformation.Instance.UserExists && (ct.OwnerUserId == UserInformation.Instance.User.Id || HiveRoles.CheckAdminUserPermissions());
-          txtName.Text = ct.Name;
-          txtHbIntervall.Text = ct.HbInterval.ToString();
-          cbxPublic.Enabled = authorized;
-          cbxPublic.CheckedChanged -= new EventHandler(cbxPublic_CheckedChanged);
-          cbxPublic.Checked = ct.OwnerUserId == null;
-          cbxPublic.CheckedChanged += new EventHandler(cbxPublic_CheckedChanged);
-          txtCPU.Text = string.Format("{0} Cores @ {1} Mhz, Arch.: {2}", ct.Cores.ToString(), ct.CpuSpeed.ToString(), ct.CpuArchitecture.ToString());
-          txtDetailsDescription.Text = ct.Description;
-          txtMemory.Text = ct.Memory.ToString();
-          txtOS.Text = ct.OperatingSystem;
-          txtSlaveState.Text = ct.SlaveState.ToString();
-          txtLastHeartbeat.Text = ct.LastHeartbeat.ToString();
-          txtFreeMemory.Text = ct.FreeMemory.ToString();
-          txtId.Text = ct.Id.ToString();
-          cbxDisposable.Enabled = authorized;
-          cbxDisposable.Checked = ct.IsDisposable.GetValueOrDefault();
-        } else if (Content.GetType() == typeof(SlaveGroup)) {
-          SlaveGroup ct = (SlaveGroup)Content;
-          txtName.Text = ct.Name;
-          txtHbIntervall.Text = ct.HbInterval.ToString();
-          cbxPublic.Enabled = ct.Name != "UNGROUPED" && HiveRoles.CheckAdminUserPermissions();
-          cbxPublic.CheckedChanged -= new EventHandler(cbxPublic_CheckedChanged);
-          cbxPublic.Checked = ct.OwnerUserId == null;
-          cbxPublic.CheckedChanged += new EventHandler(cbxPublic_CheckedChanged);
-          ShowSlaveUI(false);
-        } else {
-          throw new Exception("Unknown Resource in SlaveView");
-        }
+        cpuTextBox.Text = string.Format("{0} Cores @ {1} MHz, Arch.: {2}", Content.Cores, Content.CpuSpeed, Content.CpuArchitecture);
+        memoryTextBox.Text = string.Format("{0} ({1} Free)", Content.Memory, Content.FreeMemory);
+        operatingSystemTextBox.Text = Content.OperatingSystem;
+        stateTextBox.Text = Content.SlaveState.ToString();
+        lastHeartbeatTextBox.Text = Content.LastHeartbeat.ToString();
+        disposableCheckBox.Checked = Content.IsDisposable.GetValueOrDefault();
       }
-    }
-
-    private void ShowSlaveUI(bool show) {
-      label1.Visible = show;
-      label2.Visible = show;
-      label4.Visible = show;
-      label10.Visible = show;
-      label11.Visible = show;
-      label12.Visible = show;
-      label13.Visible = show;
-      label14.Visible = show;
-      label15.Visible = show;
-      txtCPU.Visible = show;
-      txtDetailsDescription.Visible = show;
-      txtMemory.Visible = show;
-      txtOS.Visible = show;
-      txtSlaveState.Visible = show;
-      txtLastHeartbeat.Visible = show;
-      txtFreeMemory.Visible = show;
-      txtId.Visible = show;
-      txtName.Enabled = !show;
-      cbxDisposable.Visible = show;
     }
 
     protected override void SetEnabledStateOfControls() {
       base.SetEnabledStateOfControls();
-      bool enabled = Content != null;
-      txtName.Enabled = enabled;
-      txtHbIntervall.Enabled = enabled;
-      cbxPublic.Enabled = enabled;
+      disposableCheckBox.Enabled = Content != null;
     }
+    #endregion
 
-    private void txtName_TextChanged(object sender, EventArgs e) {
-      if (Content != null && Content is SlaveGroup) {
-        Content.Name = txtName.Text;
-      }
+    #region Event Handlers
+    private void disposableCheckBox_CheckedChanged(object sender, EventArgs e) {
+      if (Content != null)
+        Content.IsDisposable = disposableCheckBox.Checked;
     }
-
-    private void txtHbIntervall_TextChanged(object sender, EventArgs e) {
-      if (Content != null) {
-        if (txtHbIntervall.Text.Length > 0) {
-          try {
-            int interval = int.Parse(txtHbIntervall.Text);
-            Content.HbInterval = interval;
-          } catch (Exception) {
-            MessageBox.Show("Please enter a numeric value for the Heartbeat Interval.", "HeuristicLab Hive Administrator", MessageBoxButtons.OK, MessageBoxIcon.Error);
-            txtHbIntervall.Text = "10";
-          }
-        }
-      }
-    }
-
-    private void cbxDisposable_CheckedChanged(object sender, EventArgs e) {
-      if (Content != null) {
-        ((Slave)Content).IsDisposable = cbxDisposable.Checked;
-      }
-    }
-
-    private void cbxPublic_CheckedChanged(object sender, EventArgs e) {
-      if (Content != null) {
-        Content.OwnerUserId = cbxPublic.Checked ? null : new Guid?(UserInformation.Instance.User.Id);
-      }
-    }
+    #endregion
   }
 }