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

   ## Description
   
   Harmonize the spelling of the SampleResult variables used in scripting 
components.
   
   ## Motivation and Context 
   
   In scripting listeners, assertions functions and samplers the SampleResult 
object is often given in the the scripting context via a variable. That 
variable should have the same name in all those components. This PR uses the 
spelling `sampleResult` in favour of `SampleResult`, as the variable points to 
an instance and in Java instances are named starting with a lowercase character.
   
   This issue has been described in #6368.
   
   One could argue to use the old name `SampleResult` as it might be found in 
more existing test cases.
   
   As the next version of JMeter will be a major version, we could use this to 
deprecate the old writing and remove those in a later major version.
   
   ## How Has This Been Tested?
   
   Test cases have been adapted to use the new and old spelling.
   
   ## Screenshots (if appropriate):
   
   ## Types of changes
   <!--- What types of changes does your code introduce? Delete as appropriate 
-->
   - Bug fix (non-breaking change which fixes an issue)
   
   ## Checklist:
   <!--- Go over all the following points, and put an `x` in all the boxes that 
apply. -->
   <!--- If you're unsure about any of these, don't hesitate to ask. We're here 
to help! -->
   - [x] My code follows the [code style][style-guide] of this project.
   - [x] 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