[GitHub] [struts] JCgH4164838Gh792C124B5 commented on pull request #528: Potential expression cache enhancement for 2.6 series

[GitBox <gi...@apache.org>]

JCgH4164838Gh792C124B5 commented on PR #528:
URL: https://github.com/apache/struts/pull/528#issuecomment-1086978579

   Hi @lukaszlenart .
   While attempting to put together some documentation on configuring the new caches, I found that I could not successfully register a custom implementation (only the default implementation ever got registered). 
   Going over the code for other factories, I noticed that there was processing in `ConstantConfig` and `StrutsBeanSelectionProvider` that is not present in this PR.  Adding equivalent logic for the two new cache factories locally did not seem to work, either (when using a configuration patterned after the custom `TextProviderFactory` example).  I attempted to debug the initialization, but cannot find what is missing.
   The factory initialization processing is complex, so there is probably something I am not understanding in order to make things work as intended.
   If you (or anyone reading this) can suggest where I should look next, or suggest another example of custom factory initialization that might make things clearer, it would be appreciated.  😄 


-- 
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: issues-unsubscribe@struts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org