You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2007/05/04 12:10:33 UTC

svn commit: r535147 - in /jakarta/jmeter/branches/rel-2-2: src/components/org/apache/jmeter/control/gui/ xdocs/

Author: sebb
Date: Fri May  4 03:10:32 2007
New Revision: 535147

URL: http://svn.apache.org/viewvc?view=rev&rev=535147
Log:
Bug 42325 - Implement the "clear" method for the LogicControllers

Modified:
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ForeachControlPanel.java
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/InterleaveControlGui.java
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/RandomControlGui.java
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/SwitchControllerGui.java
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ThroughputControllerGui.java
    jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ForeachControlPanel.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ForeachControlPanel.java?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ForeachControlPanel.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ForeachControlPanel.java Fri May  4 03:10:32 2007
@@ -34,9 +34,6 @@
  * The user interface for a foreach controller which specifies that its
  * subcomponents should be executed some number of times in a loop. This
  * component can be used standalone or embedded into some other component.
- * Copyright: 2000
- * 
- * @version $Revision$ on $Date
  */
 
 public class ForeachControlPanel extends AbstractControllerGui {
@@ -64,10 +61,10 @@
 	private boolean displayName = true;
 
 	/** The name of the infinite checkbox component. */
-	private static final String INPUTVAL = "Input Field";
+	private static final String INPUTVAL = "Input Field"; // $NON-NLS-1$
 
 	/** The name of the loops field component. */
-	private static final String RETURNVAL = "Return Field";
+	private static final String RETURNVAL = "Return Field"; // $NON-NLS-1$
 
 	/**
 	 * Create a new LoopControlPanel as a standalone component.
@@ -121,19 +118,31 @@
 			if (inputVal.getText().length() > 0) {
 				((ForeachController) lc).setInputVal(inputVal.getText());
 			} else {
-				((ForeachController) lc).setInputVal("");
+				((ForeachController) lc).setInputVal(""); // $NON-NLS-1$
 			}
 			if (returnVal.getText().length() > 0) {
 				((ForeachController) lc).setReturnVal(returnVal.getText());
 			} else {
-				((ForeachController) lc).setReturnVal("");
+				((ForeachController) lc).setReturnVal(""); // $NON-NLS-1$
 			}
 			((ForeachController) lc).setUseSeparator(useSeparator.isSelected());
 		}
 	}
+    
+    /**
+     * Implements JMeterGUIComponent.clear
+     */
+    public void clear() {
+        super.clear();
+        
+        inputVal.setText(""); // $NON-NLS-1$
+        returnVal.setText(""); // $NON-NLS-1$
+        useSeparator.setSelected(true);
+    }
+    
 
 	public String getLabelResource() {
-		return "foreach_controller_title";
+		return "foreach_controller_title"; // $NON-NLS-1$
 	}
 
 	/**
@@ -173,12 +182,12 @@
 		VerticalPanel loopPanel = new VerticalPanel();
 
 		// LOOP LABEL
-		JLabel inputValLabel = new JLabel(JMeterUtils.getResString("foreach_input"));
-		JLabel returnValLabel = new JLabel(JMeterUtils.getResString("foreach_output"));
+		JLabel inputValLabel = new JLabel(JMeterUtils.getResString("foreach_input")); // $NON-NLS-1$
+		JLabel returnValLabel = new JLabel(JMeterUtils.getResString("foreach_output")); // $NON-NLS-1$
 
 		// TEXT FIELD
 		JPanel inputValSubPanel = new JPanel(new BorderLayout(5, 0));
-		inputVal = new JTextField("", 5);
+		inputVal = new JTextField("", 5); // $NON-NLS-1$
 		inputVal.setName(INPUTVAL);
 		inputValLabel.setLabelFor(inputVal);
 		inputValSubPanel.add(inputValLabel, BorderLayout.WEST);
@@ -186,14 +195,14 @@
 
 		// TEXT FIELD
 		JPanel returnValSubPanel = new JPanel(new BorderLayout(5, 0));
-		returnVal = new JTextField("", 5);
+		returnVal = new JTextField("", 5); // $NON-NLS-1$
 		returnVal.setName(RETURNVAL);
 		returnValLabel.setLabelFor(returnVal);
 		returnValSubPanel.add(returnValLabel, BorderLayout.WEST);
 		returnValSubPanel.add(returnVal, BorderLayout.CENTER);
 
 		// Checkbox
-		useSeparator = new JCheckBox(JMeterUtils.getResString("foreach_use_separator"), true);
+		useSeparator = new JCheckBox(JMeterUtils.getResString("foreach_use_separator"), true); // $NON-NLS-1$
 
 		loopPanel.add(inputValSubPanel);
 		loopPanel.add(returnValSubPanel);
@@ -201,5 +210,4 @@
 
 		return loopPanel;
 	}
-
 }

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/InterleaveControlGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/InterleaveControlGui.java?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/InterleaveControlGui.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/InterleaveControlGui.java Fri May  4 03:10:32 2007
@@ -25,9 +25,6 @@
 import org.apache.jmeter.util.JMeterUtils;
 import org.apache.jorphan.gui.layout.VerticalLayout;
 
-/**
- * @version $Revision$ on $Date$
- */
 public class InterleaveControlGui extends AbstractControllerGui {
 	private JCheckBox style;
 
@@ -64,8 +61,16 @@
 		}
 	}
 
+    /**
+     * Implements JMeterGUIComponent.clear
+     */
+    public void clear() {
+        super.clear();    
+        style.setSelected(false);
+    }
+    
 	public String getLabelResource() {
-		return "interleave_control_title";
+		return "interleave_control_title"; // $NON-NLS-1$
 	}
 
 	private void init() {
@@ -74,7 +79,7 @@
 
 		add(makeTitlePanel());
 
-		style = new JCheckBox(JMeterUtils.getResString("ignore_subcontrollers"));
+		style = new JCheckBox(JMeterUtils.getResString("ignore_subcontrollers")); // $NON-NLS-1$
 		add(style);
 	}
 }

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/RandomControlGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/RandomControlGui.java?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/RandomControlGui.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/RandomControlGui.java Fri May  4 03:10:32 2007
@@ -26,9 +26,6 @@
 import org.apache.jmeter.util.JMeterUtils;
 import org.apache.jorphan.gui.layout.VerticalLayout;
 
-/**
- * @version $Revision$ on $Date$
- */
 public class RandomControlGui extends AbstractControllerGui {
 	private JCheckBox style;
 
@@ -56,6 +53,14 @@
 		}
 	}
 
+    /**
+     * Implements JMeterGUIComponent.clear
+     */
+    public void clear() {
+        super.clear(); 
+        style.setSelected(false);
+    }
+
 	public void configure(TestElement el) {
 		super.configure(el);
 		if (((RandomController) el).getStyle() == InterleaveControl.IGNORE_SUB_CONTROLLERS) {
@@ -66,7 +71,7 @@
 	}
 
 	public String getLabelResource() {
-		return "random_control_title";
+		return "random_control_title"; // $NON-NLS-1$
 	}
 
 	private void init() {
@@ -74,7 +79,7 @@
 		setBorder(makeBorder());
 		add(makeTitlePanel());
 
-		style = new JCheckBox(JMeterUtils.getResString("ignore_subcontrollers"));
+		style = new JCheckBox(JMeterUtils.getResString("ignore_subcontrollers")); // $NON-NLS-1$
 		add(style);
 	}
 }

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/SwitchControllerGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/SwitchControllerGui.java?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/SwitchControllerGui.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/SwitchControllerGui.java Fri May  4 03:10:32 2007
@@ -28,11 +28,8 @@
 import org.apache.jmeter.testelement.TestElement;
 import org.apache.jmeter.util.JMeterUtils;
 
-/**
- * @version $Revision$ on $Date$
- */
 public class SwitchControllerGui extends AbstractControllerGui {
-	private static final String SWITCH_LABEL = "switch_controller_label";
+	private static final String SWITCH_LABEL = "switch_controller_label"; // $NON-NLS-1$
 
 	private JTextField switchValue;
 
@@ -56,13 +53,21 @@
 		((SwitchController) ic).setSelection(switchValue.getText());
 	}
 
+    /**
+     * Implements JMeterGUIComponent.clear
+     */
+    public void clear() {
+        super.clear();
+        switchValue.setText(""); // $NON-NLS-1$
+    }
+
 	public void configure(TestElement el) {
 		super.configure(el);
 		switchValue.setText(((SwitchController) el).getSelection());
 	}
 
 	public String getLabelResource() {
-		return "switch_controller_title";
+		return "switch_controller_title"; // $NON-NLS-1$
 	}
 
 	private void init() {
@@ -78,11 +83,10 @@
 	private JPanel createSwitchPanel() {
 		JPanel switchPanel = new JPanel(new BorderLayout(5, 0));
 		JLabel selectionLabel = new JLabel(JMeterUtils.getResString(SWITCH_LABEL));
-		switchValue = new JTextField("");
+		switchValue = new JTextField(""); // $NON-NLS-1$
 		selectionLabel.setLabelFor(switchValue);
 		switchPanel.add(selectionLabel, BorderLayout.WEST);
 		switchPanel.add(switchValue, BorderLayout.CENTER);
 		return switchPanel;
 	}
-
 }

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ThroughputControllerGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ThroughputControllerGui.java?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ThroughputControllerGui.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ThroughputControllerGui.java Fri May  4 03:10:32 2007
@@ -35,9 +35,6 @@
 import org.apache.jmeter.util.JMeterUtils;
 import org.apache.jorphan.gui.layout.VerticalLayout;
 
-/**
- * @version $Revision$ on $Date$
- */
 public class ThroughputControllerGui extends AbstractControllerGui {
 	private JComboBox styleBox;
 
@@ -49,15 +46,15 @@
 
 	private boolean isPerThread = true;
 
-	private String BYNUMBER_LABEL = JMeterUtils.getResString("throughput_control_bynumber_label");
+	private String BYNUMBER_LABEL = JMeterUtils.getResString("throughput_control_bynumber_label"); // $NON-NLS-1$
 
-	private String BYPERCENT_LABEL = JMeterUtils.getResString("throughput_control_bypercent_label");
+	private String BYPERCENT_LABEL = JMeterUtils.getResString("throughput_control_bypercent_label"); // $NON-NLS-1$
 
-	private String THROUGHPUT_LABEL = JMeterUtils.getResString("throughput_control_tplabel");
+	private String THROUGHPUT_LABEL = JMeterUtils.getResString("throughput_control_tplabel"); // $NON-NLS-1$
 
-	private String THROUGHPUT = "Througput Field";
+	private String THROUGHPUT = "Througput Field"; // $NON-NLS-1$
 
-	private String PERTHREAD_LABEL = JMeterUtils.getResString("throughput_control_perthread_label");
+	private String PERTHREAD_LABEL = JMeterUtils.getResString("throughput_control_perthread_label"); // $NON-NLS-1$
 
 	public ThroughputControllerGui() {
 		init();
@@ -93,6 +90,16 @@
 		}
 	}
 
+    /**
+     * Implements JMeterGUIComponent.clear
+     */
+    public void clear() {
+        super.clear();
+        styleBox.setSelectedIndex(0);
+        throughput.setText("1"); // $NON-NLS-1$
+        perthread.setSelected(true);
+    }
+
 	public void configure(TestElement el) {
 		super.configure(el);
 		if (((ThroughputController) el).getStyle() == ThroughputController.BYNUMBER) {
@@ -138,7 +145,7 @@
 		throughput = new JTextField(5);
 		tpPanel.add(throughput);
 		throughput.setName(THROUGHPUT);
-		throughput.setText("1");
+		throughput.setText("1"); // $NON-NLS-1$
 		// throughput.addActionListener(this);
 		tpPanel.add(throughput);
 		add(tpPanel);

Modified: jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml?view=diff&rev=535147&r1=535146&r2=535147
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml (original)
+++ jakarta/jmeter/branches/rel-2-2/xdocs/changes.xml Fri May  4 03:10:32 2007
@@ -223,6 +223,7 @@
 <li>Bug 30120 - Name of simple controller is resetted if a new simple controller is added as child</li>
 <li>Bug 41078 - merge results in name change of test plan</li>
 <li>Bug 40077 - Creating new Elements copies values from Existing elements</li>
+<li>Bug 42325 - Implement the "clear" method for the LogicControllers</li>
 </ul>
 
 <h3>Version 2.2</h3>



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org