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:05:01 UTC

svn commit: r535145 - in /jakarta/jmeter/branches/rel-2-2/src: components/org/apache/jmeter/control/gui/ModuleControllerGui.java core/org/apache/jmeter/resources/messages.properties

Author: sebb
Date: Fri May  4 03:05:00 2007
New Revision: 535145

URL: http://svn.apache.org/viewvc?view=rev&rev=535145
Log:
Replace string by resource
Add NON-NLS markers

Modified:
    jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ModuleControllerGui.java
    jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/resources/messages.properties

Modified: jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ModuleControllerGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ModuleControllerGui.java?view=diff&rev=535145&r1=535144&r2=535145
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ModuleControllerGui.java (original)
+++ jakarta/jmeter/branches/rel-2-2/src/components/org/apache/jmeter/control/gui/ModuleControllerGui.java Fri May  4 03:05:00 2007
@@ -32,6 +32,7 @@
 import org.apache.jmeter.control.Controller;
 import org.apache.jmeter.control.ModuleController;
 import org.apache.jmeter.gui.GuiPackage;
+import org.apache.jmeter.gui.action.ActionNames;
 import org.apache.jmeter.gui.tree.JMeterTreeNode;
 import org.apache.jmeter.gui.util.MenuFactory;
 import org.apache.jmeter.testelement.TestElement;
@@ -44,7 +45,6 @@
 /**
  * ModuleController Gui.
  * 
- * @version $Revision$ on $Date$
  */
 public class ModuleControllerGui extends AbstractControllerGui /*
 																 * implements
@@ -60,11 +60,6 @@
 
 	private JLabel warningLabel;
 
-	public static final String CONTROLLER = "Module To Run";
-
-	// TODO should be a resource, and probably ought to be resolved at run-time
-	// (to allow language change)
-
 	/**
 	 * Initializes the gui panel for the ModuleController instance.
 	 */
@@ -73,7 +68,7 @@
 	}
 
 	public String getLabelResource() {
-		return "module_controller_title";
+		return "module_controller_title"; // $NON-NLS-1$
 	}
 
 	/*
@@ -85,11 +80,12 @@
 		super.configure(el);
 		ModuleController controller = (ModuleController) el;
 		this.selected = controller.getSelectedNode();
-		if (selected == null && controller.getNodePath() != null)
-			warningLabel.setText(JMeterUtils.getResString("module_controller_warning")
+		if (selected == null && controller.getNodePath() != null) {
+			warningLabel.setText(JMeterUtils.getResString("module_controller_warning") // $NON-NLS-1$
 					+ renderPath(controller.getNodePath()));
-		else
-			warningLabel.setText("");
+		} else {
+			warningLabel.setText(""); // $NON-NLS-1$
+		}
 		reinitialize();
 	}
 
@@ -104,8 +100,9 @@
 				continue;
 			}
 			buf.append(iter.next());
-			if (iter.hasNext())
-				buf.append(" > ");
+			if (iter.hasNext()) {
+				buf.append(" > "); // $NON-NLS-1$
+			}
 		}
 		return buf.toString();
 	}
@@ -142,8 +139,15 @@
 
 	public JPopupMenu createPopupMenu() {
 		JPopupMenu menu = new JPopupMenu();
-		JMenu addMenu = MenuFactory.makeMenus(new String[] { MenuFactory.CONFIG_ELEMENTS, MenuFactory.ASSERTIONS,
-				MenuFactory.TIMERS, MenuFactory.LISTENERS, }, JMeterUtils.getResString("Add"), "Add");
+		JMenu addMenu = MenuFactory.makeMenus(
+				new String[] {
+						MenuFactory.CONFIG_ELEMENTS, 
+						MenuFactory.ASSERTIONS,
+						MenuFactory.TIMERS, 
+						MenuFactory.LISTENERS, 
+				}, 
+				JMeterUtils.getResString("add"),  // $NON-NLS-1$
+				ActionNames.ADD);
 		menu.add(addMenu);
 		MenuFactory.addEditMenu(menu, true);
 		MenuFactory.addFileMenu(menu);
@@ -157,12 +161,14 @@
 
 		// DROP-DOWN MENU
 		JPanel modulesPanel = new JPanel(new FlowLayout(FlowLayout.LEFT, 20, 5));
-		modulesPanel.add(new JLabel(CONTROLLER));
+		JLabel nodesLabel = new JLabel(JMeterUtils.getResString("module_controller_module_to_run")); // $NON-NLS-1$
+		modulesPanel.add(nodesLabel);
 		nodesModel = new DefaultComboBoxModel();
 		nodes = new JComboBox(nodesModel);
+        nodesLabel.setLabelFor(nodes);
 		reinitialize();
 		modulesPanel.add(nodes);
-		warningLabel = new JLabel("");
+		warningLabel = new JLabel(""); // $NON-NLS-1$
 		modulesPanel.add(warningLabel);
 		add(modulesPanel);
 	}
@@ -174,7 +180,7 @@
 		JMeterTreeNode root;
 		if (gp != null) {
 			root = (JMeterTreeNode) GuiPackage.getInstance().getTreeModel().getRoot();
-			buildNodesModel(root, "", 0);
+			buildNodesModel(root, "", 0); // $NON-NLS-1$
 		}
 		if (selected != null) {
 			for (int i = 0; i < nodesModel.getSize(); i++) {
@@ -190,9 +196,9 @@
 
 	private void buildNodesModel(JMeterTreeNode node, String parent_name, int level) {
 		if (level == 0 && (parent_name == null || parent_name.length() == 0)) {
-			nodesModel.addElement(new TreeNodeWrapper(null, ""));
+			nodesModel.addElement(new TreeNodeWrapper(null, "")); // $NON-NLS-1$
 		}
-		String seperator = " > ";
+		String seperator = " > "; // $NON-NLS-1$
 		if (node != null) {
 			for (int i = 0; i < node.getChildCount(); i++) {
 				StringBuffer name = new StringBuffer();
@@ -226,7 +232,7 @@
 		int multi = 4;
 		StringBuffer spaces = new StringBuffer(level * multi);
 		for (int i = 0; i < level * multi; i++) {
-			spaces.append(" ");
+			spaces.append(" "); // $NON-NLS-1$
 		}
 		return spaces.toString();
 	}

Modified: jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/resources/messages.properties
URL: http://svn.apache.org/viewvc/jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/resources/messages.properties?view=diff&rev=535145&r1=535144&r2=535145
==============================================================================
--- jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/resources/messages.properties (original)
+++ jakarta/jmeter/branches/rel-2-2/src/core/org/apache/jmeter/resources/messages.properties Fri May  4 03:05:00 2007
@@ -444,6 +444,7 @@
 modtest=Modification test
 module_controller_title=Module Controller
 module_controller_warning=Could not find module: 
+module_controller_module_to_run=Module To Run 
 monitor_equation_active=Active:  (busy/max) > 25%
 monitor_equation_dead=Dead:  no response
 monitor_equation_healthy=Healthy:  (busy/max) < 25%



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