[GitHub] [struts] kusalk opened a new pull request, #7 46: WW-5340 Preliminary refactor of OgnlUtil

kusalk opened a new pull request, #746:
URL: https://github.com/apache/struts/pull/746

   WW-5340
   --
   I've refactored this class to improve both readability and maintainability.
   
   My primary goal was to consolidate calls to ognl.Ognl, specifically the following functions:
   * `#getValue`: 5 calls to 1
   * `#setValue`: 2 calls to 1
   * `#parseExpression`: 4 calls to 1
   
   Functionally, there are only 2 changes:
   * Removed redundant tree validation from `OgnlUtil#checkValue`
   * Cache expressions prior to tree validation (i.e. even if validation fails)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]