search

jakartaee / expression-language

Jakarta Expression Language
https://eclipse.org/ee4j/el
Other
70 stars 50 forks source link

Optional resolver updates #208

Closed markt-asf closed 7 months ago

markt-asf commented 7 months ago

Adds method invocation support to the resolver.

Makes property support more consistent. Now never returns an Optional instance when resolving a property.

Coercion is also more consistent so empty Optional instances will be treated as null before being coerced to the required type (rather than simply returning null).

Finally, since this was copied from Tomcat the license header needed to be updated. As the original author of the Tomcat code, fixing the license header is not an issue.

markt-asf commented 7 months ago

We are on a fairly tight deadline but I also want to give folks a chance to review this. Unless there are objections, I intend to merge this once it has been open for 48 hours (in ~23 hours from this comment).

pnicolucci commented 7 months ago

Thanks for working on this @markt-asf !