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 2006/04/13 01:11:38 UTC

svn commit: r393650 - /jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java

Author: sebb
Date: Wed Apr 12 16:11:36 2006
New Revision: 393650

URL: http://svn.apache.org/viewcvs?rev=393650&view=rev
Log:
Make protocol field a bit larger (for use with variables)

Modified:
    jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java

Modified: jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java
URL: http://svn.apache.org/viewcvs/jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java?rev=393650&r1=393649&r2=393650&view=diff
==============================================================================
--- jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java (original)
+++ jakarta/jmeter/branches/rel-2-1/src/protocol/http/org/apache/jmeter/protocol/http/config/gui/UrlConfigGui.java Wed Apr 12 16:11:36 2006
@@ -122,23 +122,6 @@
 		return element;
 	}
 
-	// Does not appear to be used
-	// public void configureSampler(HTTPSampler sampler)
-	// {
-	// sampler.setArguments((Arguments) argsPanel.createTestElement());
-	// sampler.setDomain(domain.getText());
-	// sampler.setProtocol(protocol.getText());
-	// sampler.setPath(path.getText());
-	// sampler.setFollowRedirects(followRedirects.isSelected());
-	// sampler.setDelegateRedirects(autoRedirects.isSelected());
-	// sampler.setUseKeepAlive(useKeepAlive.isSelected());
-	// if (port.getText().length() > 0)
-	// {
-	// sampler.setPort(Integer.parseInt(port.getText()));
-	// }
-	// sampler.setMethod((post.isSelected() ? "POST" : "GET"));
-	// }
-
 	/**
 	 * Set the text, etc. in the UI.
 	 * 
@@ -153,7 +136,7 @@
 		String portString = el.getPropertyAsString(HTTPSamplerBase.PORT);
 
 		// Only display the port number if it is meaningfully specified
-		if (portString.equals("" + HTTPSamplerBase.UNSPECIFIED_PORT)) {
+		if (portString.equals(HTTPSamplerBase.UNSPECIFIED_PORT_AS_STRING)) {
 			port.setText("");
 		} else {
 			port.setText(portString);
@@ -274,7 +257,7 @@
 
 	protected JPanel getProtocolAndMethodPanel() {
 		// PROTOCOL
-		protocol = new JTextField(4);
+		protocol = new JTextField(20);
 		protocol.setName(PROTOCOL);
 
 		JLabel protocolLabel = new JLabel(JMeterUtils.getResString("protocol"));



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