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

Gerhard Petracek commented on EXTCDI-249:
-----------------------------------------

it isn't needed for PropertyExpressionInterpreter because it's the default 
implementation.

imo we don't need such an indirection right now, because it's mainly an 
extension point for concrete projects and we won't have that many alternatives 
in our code-base. currently it's just SystemPropertyExpressionInterpreter. -> 
it would be enough to move it to 
org.apache.myfaces.extensions.cdi.core.api.interpreter

(as soon as we need such an indirection (it's never needed by users for custom 
implementations), we can add it easily).

                
> move PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter to 
> core-api
> --------------------------------------------------------------------------------------
>
>                 Key: EXTCDI-249
>                 URL: https://issues.apache.org/jira/browse/EXTCDI-249
>             Project: MyFaces CODI
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 1.0.2
>            Reporter: Mark Struberg
>
> the SystemPropertyExpressionInterpreter is pretty often very useful.
> Since PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter 
> are only a slim shim over the AbstractPropertyExpressionInterpreter (which is 
> already in core-api), we should move them over to core-api as well.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to