This patch worked well and I have committed it to CVS.  Thanks Hans!

-Mike

On Mon, 2005-02-07 at 04:01, Hessmann Hans-Joerg wrote:
> Hello,
> 
> I modified the method addParentToTree(TestElement newParent) of
> org.apache.jmeter.gui.action.AddParent in jmeter-2.0.2 to:
> 
>     protected void addParentToTree(TestElement newParent)
>     {
>         GuiPackage guiPackage = GuiPackage.getInstance();
>         JMeterTreeNode newNode =
>             new JMeterTreeNode(newParent, guiPackage.getTreeModel());
>         JMeterTreeNode currentNode =
>             guiPackage.getTreeListener().getCurrentNode();
>         JMeterTreeNode parentNode = (JMeterTreeNode)
> currentNode.getParent();
>         int index = parentNode.getIndex(currentNode);
>         guiPackage.getTreeModel().insertNodeInto(
>                 newNode,
>                 parentNode,
>                 index);
>         JMeterTreeNode[] nodes =
>             guiPackage.getTreeListener().getSelectedNodes();
>         for (int i = 0; i < nodes.length; i++) {
>             moveNode(guiPackage, nodes[i], newNode);
>         }
>     }
>     
>     private void moveNode(GuiPackage guiPackage, JMeterTreeNode node,
> JMeterTreeNode newParentNode) {
>         guiPackage.getTreeModel().removeNodeFromParent(node);
>         guiPackage.getTreeModel().insertNodeInto(
>                 node,
>                 newParentNode,
>                 newParentNode.getChildCount());        
>     }
>         
> With this patch applied, the parent controller contains all selected nodes,
> after it has been added. I made this modification because it saves me time,
> when I restructure recorded tests.
> 
> I don't have access to CVS. So, if you think this is useful for others, then
> submit it as a patch to the current version, please.
> 
> Best regards,
> 
> Hans J�rg Hessmann
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
-- 
Michael Stover <[EMAIL PROTECTED]>
Apache Software Foundation


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to