Changeset 140

Timestamp:

3/8/2008 5:47:58 AM (5 years ago)

Author:

lowjoel

Message:

Move the text generation for UI display into the Manager.Task class since it should be an unified identifier.

Location:

branches/eraser6

Files:

• Eraser/TaskPropertiesForm.cs (modified) (2 diffs)
• Manager/Task.cs (modified) (6 diffs)

branches/eraser6/Eraser/TaskPropertiesForm.cs

@@ -22 +22 @@
         }
 
+        /// <summary>
+        /// Sets or retrieves the task object to be edited or being edited.
+        /// </summary>
+        public Task Task
+        {
+            get { return task; }
+            set { task = value; UpdateUIFromTask(); }
+        }
+
+        private void UpdateUIFromTask()
+        {
+            throw new NotImplementedException("UpdateUIFromTask not implemented.");
+        }
+
         private void dataAdd_Click(object sender, EventArgs e)
         {
@@ -29 +43 @@
                 {
                     Task.EraseTarget entry = form.GetTaskEntry();
-                    ListViewItem item = null;
-                    if (entry is Task.FilesystemObject)
-                        item = data.Items.Add((entry as Task.FilesystemObject).Path);
-                    else if (entry is Task.FreeSpace)
-                        item = data.Items.Add("Unused space on " + (entry as Task.FreeSpace).Drive);
-                    else
-                        throw new NotImplementedException("Unimplemented data erasure type.");
+                    ListViewItem item = data.Items.Add(entry.UIText);
                     
                     item.SubItems.Add(entry.Method.Name);

branches/eraser6/Manager/Task.cs

@@ -13 +13 @@
         /// Represents a generic target of erasure
         /// </summary>
-        public class EraseTarget
+        public abstract class EraseTarget
         {
             /// <summary>
@@ -23 +23 @@
                 get { return method; }
                 set { method = value; }
+            }
+
+            /// <summary>
+            /// Retrieves the text to display representing this task.
+            /// </summary>
+            public abstract string UIText
+            {
+                get;
             }
 
@@ -48 +56 @@
             }
 
+            public override string UIText
+            {
+                get { return Path; }
+            }
+
             private string path;
         }
@@ -57 +70 @@
         {
             public string Drive;
+
+            public override string UIText
+            {
+                get { return string.Format("Unused disk space ({0})", Drive); }
+            }
         }
 
@@ -130 +148 @@
 
         /// <summary>
-        /// The name for this task. This is just an opaque value for the user to recognize
-        /// the task.
+        /// The name for this task. This is just an opaque value for the user to
+        /// recognize the task.
         /// </summary>
         public string Name
@@ -148 +166 @@
         }
 
+        /// <summary>
+        /// The schedule for running the task.
+        /// </summary>
+        public Schedule Schedule
+        {
+            get { return schedule; }
+            set { schedule = value; }
+        }
+
         private uint id;
         private string name;
+        private Schedule schedule = null;
         private List<EraseTarget> entries = new List<EraseTarget>();
     }