Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added JEXL3 as a syntax alias for JSyntaxTextArea #585

Merged
merged 1 commit into from
May 2, 2020

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