Author: agomes Date: Sat Nov 11 22:21:09 2017 New Revision: 1814992 URL: http://svn.apache.org/viewvc?rev=1814992&view=rev Log: Bug 61672 - add tht choice between prefix and transaction name to name sampler during recording : Philippe catch
Modified: jmeter/trunk/docs/images/screenshots/proxy_control.png jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/DefaultSamplerCreator.java jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/ProxyControl.java jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java Modified: jmeter/trunk/docs/images/screenshots/proxy_control.png URL: http://svn.apache.org/viewvc/jmeter/trunk/docs/images/screenshots/proxy_control.png?rev=1814992&r1=1814991&r2=1814992&view=diff ============================================================================== Binary files - no diff available. Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/DefaultSamplerCreator.java URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/DefaultSamplerCreator.java?rev=1814992&r1=1814991&r2=1814992&view=diff ============================================================================== --- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/DefaultSamplerCreator.java (original) +++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/DefaultSamplerCreator.java Sat Nov 11 22:21:09 2017 @@ -281,12 +281,12 @@ public class DefaultSamplerCreator exten protected void computeSamplerName(HTTPSamplerBase sampler, HttpRequestHdr request) { String prefix = request.getPrefix(); - int HttpSampleNameMode = request.getHttpSampleNameMode(); + int httpSampleNameMode = request.getHttpSampleNameMode(); if (!HTTPConstants.CONNECT.equals(request.getMethod()) && isNumberRequests()) { if(!StringUtils.isEmpty(prefix)) { - if (HttpSampleNameMode==SAMPLER_NAME_NAMING_MODE_PREFIX) { + if (httpSampleNameMode==SAMPLER_NAME_NAMING_MODE_PREFIX) { sampler.setName(prefix + incrementRequestNumberAndGet() + " " + sampler.getPath()); - } else if (HttpSampleNameMode==SAMPLER_NAME_NAMING_MODE_COMPLETE) { + } else if (httpSampleNameMode==SAMPLER_NAME_NAMING_MODE_COMPLETE) { sampler.setName(incrementRequestNumberAndGet() + " " + prefix); } else { log.debug("Sampler name naming mode not recognized"); @@ -296,9 +296,9 @@ public class DefaultSamplerCreator exten } } else { if(!StringUtils.isEmpty(prefix)) { - if (HttpSampleNameMode==SAMPLER_NAME_NAMING_MODE_PREFIX) { + if (httpSampleNameMode==SAMPLER_NAME_NAMING_MODE_PREFIX) { sampler.setName(prefix+sampler.getPath()); - } else if (HttpSampleNameMode==SAMPLER_NAME_NAMING_MODE_COMPLETE) { + } else if (httpSampleNameMode==SAMPLER_NAME_NAMING_MODE_COMPLETE) { sampler.setName(prefix); } else { log.debug("Sampler name naming mode not recognized"); Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/ProxyControl.java URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/ProxyControl.java?rev=1814992&r1=1814991&r2=1814992&view=diff ============================================================================== --- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/ProxyControl.java (original) +++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/ProxyControl.java Sat Nov 11 22:21:09 2017 @@ -392,8 +392,8 @@ public class ProxyControl extends Generi setProperty(new BooleanProperty(SAMPLER_DOWNLOAD_IMAGES, b)); } - public void setHTTPSampleNamingMode(int HTTPNamingMode) { - setProperty(new IntegerProperty(HTTP_SAMPLER_NAMING_MODE, HTTPNamingMode)); + public void setHTTPSampleNamingMode(int httpNamingMode) { + setProperty(new IntegerProperty(HTTP_SAMPLER_NAMING_MODE, httpNamingMode)); } public void setPrefixHTTPSampleName(String prefixHTTPSampleName) { Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java?rev=1814992&r1=1814991&r2=1814992&view=diff ============================================================================== --- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java (original) +++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java Sat Nov 11 22:21:09 2017 @@ -159,7 +159,7 @@ public class ProxyControlGui extends Log /** * To choose between a prefix or a transaction name */ - private JComboBox<String> HTTPSampleNamingMode; + private JComboBox<String> httpSampleNamingMode; /** * Add a prefix/transaction name to HTTP sample name recorded @@ -291,7 +291,7 @@ public class ProxyControlGui extends Log model.setSamplerFollowRedirects(samplerFollowRedirects.isSelected()); model.setUseKeepAlive(useKeepAlive.isSelected()); model.setSamplerDownloadImages(samplerDownloadImages.isSelected()); - model.setHTTPSampleNamingMode(HTTPSampleNamingMode.getSelectedIndex()); + model.setHTTPSampleNamingMode(httpSampleNamingMode.getSelectedIndex()); model.setPrefixHTTPSampleName(prefixHTTPSampleName.getText()); model.setProxyPauseHTTPSample(proxyPauseHTTPSample.getText()); model.setNotifyChildSamplerListenerOfFilteredSamplers(notifyChildSamplerListenerOfFilteredSamplersCB.isSelected()); @@ -354,7 +354,7 @@ public class ProxyControlGui extends Log samplerFollowRedirects.setSelected(model.getSamplerFollowRedirects()); useKeepAlive.setSelected(model.getUseKeepalive()); samplerDownloadImages.setSelected(model.getSamplerDownloadImages()); - HTTPSampleNamingMode.setSelectedIndex(model.getHTTPSampleNamingMode()); + httpSampleNamingMode.setSelectedIndex(model.getHTTPSampleNamingMode()); prefixHTTPSampleName.setText(model.getPrefixHTTPSampleName()); proxyPauseHTTPSample.setText(model.getProxyPauseHTTPSample()); notifyChildSamplerListenerOfFilteredSamplersCB.setSelected(model.getNotifyChildSamplerListenerOfFilteredSamplers()); @@ -389,7 +389,7 @@ public class ProxyControlGui extends Log if (e.getSource() instanceof JComboBox) { JComboBox combo = (JComboBox) e.getSource(); if(HTTP_SAMPLER_NAMING_MODE.equals(combo.getName())){ - model.setHTTPSampleNamingMode(HTTPSampleNamingMode.getSelectedIndex()); + model.setHTTPSampleNamingMode(httpSampleNamingMode.getSelectedIndex()); } } else { @@ -854,9 +854,9 @@ public class ProxyControlGui extends Log DefaultComboBoxModel<String> choice = new DefaultComboBoxModel<>(); choice.addElement(JMeterUtils.getResString("sample_name_prefix")); // $NON-NLS-1$ choice.addElement(JMeterUtils.getResString("sample_name_transaction")); // $NON-NLS-1$ - HTTPSampleNamingMode = new JComboBox<>(choice); - HTTPSampleNamingMode.setName(HTTP_SAMPLER_NAMING_MODE); - HTTPSampleNamingMode.addItemListener(this); + httpSampleNamingMode = new JComboBox<>(choice); + httpSampleNamingMode.setName(HTTP_SAMPLER_NAMING_MODE); + httpSampleNamingMode.addItemListener(this); prefixHTTPSampleName = new JTextField(4); prefixHTTPSampleName.addKeyListener(this); @@ -885,7 +885,7 @@ public class ProxyControlGui extends Log JPanel panel = new JPanel(gridBagLayout); panel.setBorder(BorderFactory.createTitledBorder(BorderFactory.createEtchedBorder(), JMeterUtils.getResString("proxy_sampler_settings"))); // $NON-NLS-1$ - panel.add(HTTPSampleNamingMode, gbc.clone()); + panel.add(httpSampleNamingMode, gbc.clone()); gbc.gridx++; gbc.weightx = 3; gbc.fill=GridBagConstraints.HORIZONTAL;