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

Gerhard Petracek commented on EXTCDI-208:
-----------------------------------------

it was done that way because i saw different instances even within a request. 
but i can't remember the setup. however, it shouldn't be called that often 
within a request and the injection process isn't expansive esp. due to the 
@Advanced check (which is the default behaviour). an alternative which would be 
safe as well is lazy injection directly in DefaultErrorViewExceptionHandler.

> CodiFacesContextWrapper.getExceptionHandler() tries to inject many times 
> during one request/response
> ----------------------------------------------------------------------------------------------------
>
>                 Key: EXTCDI-208
>                 URL: https://issues.apache.org/jira/browse/EXTCDI-208
>             Project: MyFaces CODI
>          Issue Type: Improvement
>    Affects Versions: 1.0.0
>         Environment: codi 1.0.0
>            Reporter: Martin Kočí
>            Priority: Minor
>
> org.apache.myfaces.extensions.cdi.jsf2.impl.listener.request.CodiFacesContextWrapper.getExceptionHandler()
>  tries to inject field always - in every method call. 
> ExceptionHandler instance has request scoped lifespan and it cannot change - 
> I think is suitable to inject fields only once.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to