Changeset 1493 for branches/eraser6/6.0/Eraser.DefaultPlugins/CustomMethodEditorForm.cs

Timestamp:

1/10/2010 8:36:05 AM (3 years ago)

Author:

lowjoel

Message:

addresses #285: Various Custom Erasure Method editor bugs
-When adding a pass with a pass currently selected, add it after the selected pass.
-When duplicating passes, save the currently selected one before duplication (so both get the new data.)
-When unselecting passes, unset the currentPass member variable.

File:

• branches/eraser6/6.0/Eraser.DefaultPlugins/CustomMethodEditorForm.cs (modified) (3 diffs)

branches/eraser6/6.0/Eraser.DefaultPlugins/CustomMethodEditorForm.cs

@@ -170 +170 @@
         private void passesAddBtn_Click(object sender, EventArgs e)
         {
+            //Save the current pass being edited
+            if (currentPass != null)
+                SavePass(currentPass);
+
+            //Then create a new, random pass, adding it to the list
             ErasureMethodPass pass = new ErasureMethodPass(ErasureMethod.WriteRandom, null);
             ListViewItem item = AddPass(pass);
 
+            //If a pass is currently selected, insert the pass after the currently selected one.
             if (passesLv.SelectedIndices.Count > 0)
             {
                 item.Remove();
-                passesLv.Items.Insert(passesLv.SelectedIndices[passesLv.SelectedIndices.Count - 1],
+                passesLv.Items.Insert(passesLv.SelectedIndices[passesLv.SelectedIndices.Count - 1] + 1,
                     item);
                 RenumberPasses();
@@ -192 +198 @@
         private void passesDuplicateBtn_Click(object sender, EventArgs e)
         {
+            //Save the current pass to prevent data loss
+            SavePass(currentPass);
+
             foreach (ListViewItem item in passesLv.SelectedItems)
             {
@@ -253 +262 @@
             {
                 if (e.Item == currentPass)
+                {
                     SavePass(e.Item);
+                    currentPass = null;
+                }
             }
             else if (passesLv.SelectedIndices.Count == 1)