Changeset 9057

Timestamp:

12/14/12 13:03:29 (12 years ago)

Author:

bburlacu

Message:

#1763: Fixed bug in subtree cut/paste introduced with previous commit, simplified code.

File:

• trunk/sources/HeuristicLab.Problems.DataAnalysis.Symbolic.Views/3.4/InteractiveSymbolicExpressionTreeChart.cs (modified) (1 diff)

trunk/sources/HeuristicLab.Problems.DataAnalysis.Symbolic.Views/3.4/InteractiveSymbolicExpressionTreeChart.cs

@@ -217 +217 @@
         case EditOp.CutSubtree: {
             if (tempNode.IterateNodesBreadth().Contains(node))
-              throw new ArgumentException();
+              throw new ArgumentException();// cannot cut/paste a node into itself
+            ModifyTree(Tree, tempNode.Parent, tempNode, null); //remove node from its original parent      
+            ModifyTree(Tree, node, null, tempNode); //insert it as a child to the new parent 
             break;
           }
         case EditOp.CopySubtree: {
             var clone = (SymbolicExpressionTreeNode)tempNode.Clone();
-            clone.Parent = tempNode.Parent;
-            tempNode = clone;
-            ModifyTree(Tree, node, null, tempNode);
+            ModifyTree(Tree, node, null, clone);
             break;
           }