Github user ubikloadpack commented on the issue:
https://github.com/apache/jmeter/pull/223
Hello,
Thanks for PR.
My notes:
- defaultScriptLanguage should be uppercase being a constant.
- use StringUtils.isEmpty to avoid spaces breaking the control
- Maybe a solution to @vlsi note is to set the script , something like this
(note that I don't think it will set the lang on the final test but only avoid
repeating the log message for the same element each time it runs
` if (StringUtils.isEmpty(lang)) {
lang = DEFAULT_SCRIPT_LANGUAGE;
setScript(lang);
log.warn("Script language has not been chosen on the UI:
"+getName()+", the script will be interpreted as a groovy script");
}
`
Regards
@ubikloadpack
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
