pochopsp opened a new pull request, #6291:
URL: https://github.com/apache/jmeter/pull/6291

   ## Description
   This pull request adds a new column in the HTTP Sampler parameters table 
which allows the user to enable or disable the http parameter.
   
   To implement this feature, a new "isEnabledFromGui" property has been added 
in HTTPArgument and it has been used in several places (HTTPHC4Impl, 
HTTPSamplerBase, PutWriter etc.) to determine whether to include the argument 
in the request or not.
   To enable the property setting by the user a new column has been added in 
HTTPArgumentsPanel.
   The HTTPArgumentSchema has been adjusted to include this new field too.
   Tests file and localization files has been adjusted to be compliant with 
this new attribute.
   
   ## Motivation and Context
   I think this feature could be really useful, because sometimes I and other 
people I know wished JMeter had it while doing some tests.
   Fixes https://github.com/apache/jmeter/issues/5466
   
   ## How Has This Been Tested?
   I have successfully built the project by using `./gradlew build` and all 
tests ran without issues.
   My testing environment is Windows 10, Intel I5 6th gen, 16GB Ram DD4, Oracle 
JDK 17.
   
   ## Screenshots (if appropriate):
   
![image](https://github.com/apache/jmeter/assets/50794441/7b2c628f-cdef-48de-a707-bf72ad5e244a)
   
![image](https://github.com/apache/jmeter/assets/50794441/7c55986e-63e0-4d4b-bd85-4f4e69f9791a)
   
   ## Types of changes
   - New feature (non-breaking change which adds functionality)
   
   ## Checklist:
   - [x ] My code follows the [code style][style-guide] of this project.
   - [ ] I have updated the documentation accordingly.
   
   [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@jmeter.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to