[ 
https://issues.apache.org/jira/browse/JEXL-406?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17767869#comment-17767869
 ] 

Henri Biestro commented on JEXL-406:
------------------------------------

Regarding your failed test, I've updated the code in 
[fcf59afc|https://github.com/apache/commons-jexl/commit/fcf59afce4900c895b953867cd4fe7204aca52eb]
 to have a clearer idea of where that fails. Note that this is based on the 
latest git code, not the 3.3 (released) version.

Regarding the map literal modifications, they fit your purpose but not the 
general case. Whether to interpret identifiers used as map keys as variables or 
not can lead to much confusion (imho).

Back to the original ask, will expose Arguments momentarily.

> allow override Engine.createTemplateInterpreter ?
> -------------------------------------------------
>
>                 Key: JEXL-406
>                 URL: https://issues.apache.org/jira/browse/JEXL-406
>             Project: Commons JEXL
>          Issue Type: Improvement
>    Affects Versions: 3.3
>            Reporter: Xu Pengcheng
>            Assignee: Henri Biestro
>            Priority: Major
>         Attachments: image-2023-09-22-08-57-22-860.png
>
>
> I defined a Interpreter for JexlEngine but found it does not work while 
> evaluating template string, I debugged and found Interpreter of jexl is not 
> applied for TemplateEngine by default.
> I found there is a method createTemplateInterpreter in Engine.class
> {code:java}
> /**
> * Creates a template interpreter.
> * @param args the template interpreter arguments
> */
> protected Interpreter createTemplateInterpreter(final 
> TemplateInterpreter.Arguments args) {
> return new TemplateInterpreter(args);
> }{code}
> But the parameter TemplateInterpreter.Arguments is not visible outside of the 
> package, is it possible to change the visibility so that we can override it?
>  
> Thanks!



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to