Bug 59116

Summary: Add the possibility to setup a prefix to sampler name recorded by proxy
Product: JMeter - Now in Github Reporter: Antonio Gomes Rodrigues <ra0077>
Component: MainAssignee: JMeter issues mailing list <issues>
Severity: enhancement CC: p.mouawad, ra0077
Priority: P2 Keywords: PatchAvailable
Version: 2.13   
Target Milestone: ---   
Hardware: All   
OS: All   

Description Antonio Gomes Rodrigues 2016-03-04 14:38:18 UTC

I have add a new option in proxy recorder to add a prefix in sampler name recorded by the proxy.

It allow to be more productive (less work to rename samplers after a recording) and complete the regex in backend listener I have add previously

Comment 1 Antonio Gomes Rodrigues 2016-03-04 14:43:03 UTC
PR 156 commited
Comment 2 Sebb 2016-03-04 15:39:36 UTC
(In reply to Antonio Gomes Rodrigues from comment #1)
> PR 156 commited

>     https://github.com/apache/jmeter/pull/156.patch

Sorry, but the patch is unusable as it stands.

There are huge numbers of irrelevant changes which make it all but impossible to review. Here is the summary:

 .../apache/jmeter/resources/messages.properties    | 2689 ++++++++---------
 .../apache/jmeter/resources/messages_fr.properties | 2659 ++++++++---------
 .../jmeter/protocol/http/proxy/ProxyControl.java   | 3039 ++++++++++----------
 .../protocol/http/proxy/gui/ProxyControlGui.java   |   25 +-
 4 files changed, 4223 insertions(+), 4189 deletions(-)

I would expect a patch to fix this to be 100 changes or so, not 1000s.

Patches should contain the minimum number of changes.
This allows developers to quickly review the proposed change.
Also once the patch is applied, it makes the history easier to read.

If you want the patch to be applied, please provide a minimal one.
Comment 3 Antonio Gomes Rodrigues 2016-03-04 16:42:18 UTC
PR 158 without tab commited
Comment 4 Philippe Mouawad 2016-03-04 22:17:28 UTC
Thanks for PR.
I made few modification , see comments.

Author: pmouawad
Date: Fri Mar  4 22:16:39 2016
New Revision: 1733665

URL: http://svn.apache.org/viewvc?rev=1733665&view=rev
Bug 59116 - Add the possibility to setup a prefix to sampler name recorded by proxy
#resolve #158
Fixed the following in PR:
- SampleResult did not match the modified Sample label
- Added code in keyListener to ensure the model is updated, otherwise the ProxyControl would still use old prefix until user leaves the component
- French label
Bugzilla Id: 59116

Comment 5 Antonio Gomes Rodrigues 2016-03-05 07:16:06 UTC
Thanks to the modifications
Comment 6 The ASF infrastructure team 2022-09-24 20:38:03 UTC
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/3880