Changeset 2045 for trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs

Timestamp:

5/4/2010 3:04:18 AM (3 years ago)

Author:

lowjoel

Message:

Fixed the null Progress property upon the completion of the FileSystemObjectErasureTarget?.Execute function. The Progress property is now managed by the derived class itself.

File:

• trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs (modified) (2 diffs)

trunk/eraser/Eraser.DefaultPlugins/ErasureTargets/FileSystemObjectErasureTarget.cs

@@ -179 +179 @@
         }
 
-        public override void Execute()
+        /// <remarks>The <see cref="Progress"/> property must be defined prior
+        /// to the execution of this function.</remarks>
+        public virtual void Execute()
         {
             //Retrieve the list of files to erase.
@@ -190 +192 @@
 
             //Set the event's current target status.
-            SteppedProgressManager progress = new SteppedProgressManager();
-            Progress = progress;
-            Task.Progress.Steps.Add(new SteppedProgressManagerStep(progress, 1.0f / Task.Targets.Count));
+            if (Progress == null)
+                throw new InvalidOperationException("The Progress property must not be null.");
+            Task.Progress.Steps.Add(new SteppedProgressManagerStep(Progress, 1.0f / Task.Targets.Count));
 
             //Iterate over every path, and erase the path.