On 21 February 2012 07:41,  <[email protected]> wrote:
> Author: pmouawad
> Date: Tue Feb 21 07:41:05 2012
> New Revision: 1291632
>
> URL: http://svn.apache.org/viewvc?rev=1291632&view=rev
> Log:
> Bug 52672 - Change Controller action deletes all but one child samplers
> Added doc
>
> Modified:
>    jmeter/trunk/src/core/org/apache/jmeter/gui/action/ChangeParent.java
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/action/ChangeParent.java
> URL: 
> http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/action/ChangeParent.java?rev=1291632&r1=1291631&r2=1291632&view=diff
> ==============================================================================
> --- jmeter/trunk/src/core/org/apache/jmeter/gui/action/ChangeParent.java 
> (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/gui/action/ChangeParent.java Tue 
> Feb 21 07:41:05 2012
> @@ -79,6 +79,7 @@ public class ChangeParent implements Com
>         treeModel.removeNodeFromParent(currentNode);
>         int childCount = currentNode.getChildCount();
>         for (int i = 0; i < childCount; i++) {
> +            // Using index 0 is voluntary as child is removed in next step 
> and added to new parent

s/voluntary/required/ ?

>             JMeterTreeNode node = (JMeterTreeNode) currentNode.getChildAt(0);
>             treeModel.removeNodeFromParent(node);
>             treeModel.insertNodeInto(node, newNode, newNode.getChildCount());
>
>

Reply via email to