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.

Sign up for GitHub

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

Jump to bottom

MVEL regression: failure to declare new variables in the template #2550

Closed
tsegismont opened this issue Dec 22, 2023 · 1 comment
Closed

MVEL regression: failure to declare new variables in the template #2550

tsegismont opened this issue Dec 22, 2023 · 1 comment
Assignees
@tsegismont
Labels
bug
Milestone
4.5.2

Comments

@tsegismont
Copy link
Contributor

There was a regression introduced in #468. The user-supplied variables were passed to the template engine with a MapVariableResolverFactory and we switched to a context object.

For this reason, it was no longer possible to declare variables in template files.
@tsegismont tsegismont added the bug label Dec 22, 2023
@tsegismont tsegismont added this to the 4.5.2 milestone Dec 22, 2023
@tsegismont tsegismont self-assigned this Dec 22, 2023
@tsegismont
Copy link
Contributor Author

Fixed by 4dfd2a9

@tsegismont tsegismont mentioned this issue Dec 22, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsegismont tsegismont

Labels
bug
Milestone
4.5.2
Development

No branches or pull requests
1 participant
@tsegismont