Skip to content

Conversation

@drivera-armedia
Copy link
Contributor

Description

Added jexl3 as a language alias for text/java, for use with JSyntaxTextArea:
Add: jexl3=text/java to textarea.properties

Motivation and Context

The JSR223 and other similar components that use JSyntaxTextArea may set the language to JEXL3, which is a perfecly valid scripting language name, but the syntax highlighter won't recognize and and won't enable the highlighting. By adding this alias, the issue is resolved. Considering that an alias already existed for jexl2, it makes sense that jexl3 be added.

How Has This Been Tested?

The change was tested in a custom build. Because of the nature of the change (adding a property mapping) there is no risk of breakage. Furthermore, the value of the property mapping is verified to be good (same as is used in other mappings, i.e. for jexl2).

Screenshots (if appropriate):

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the code style of this project.
  • I have updated the documentation accordingly.

@pmouawad pmouawad merged commit 0c666f5 into apache:master May 2, 2020
@pmouawad
Copy link
Contributor

pmouawad commented May 2, 2020

Thanks for contributing

kkalinin pushed a commit to kkalinin/jmeter that referenced this pull request Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants