You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Mark Struberg (Created) (JIRA)" <de...@myfaces.apache.org> on 2011/12/05 14:53:40 UTC

[jira] [Created] (EXTCDI-249) move PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter to core-api

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

        

[jira] [Resolved] (EXTCDI-249) move PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter to core-api

Posted by "Gerhard Petracek (Resolved) (JIRA)" <de...@myfaces.apache.org>.
     [ https://issues.apache.org/jira/browse/EXTCDI-249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gerhard Petracek resolved EXTCDI-249.
-------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.0.4
    
> 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
>            Assignee: Gerhard Petracek
>             Fix For: 1.0.4
>
>
> 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

        

[jira] [Commented] (EXTCDI-249) move PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter to core-api

Posted by "Mark Struberg (Commented) (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/EXTCDI-249?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13162784#comment-13162784 ] 

Mark Struberg commented on EXTCDI-249:
--------------------------------------

we should consider introducing interfaces for them in core-api and have the implementations in core-impl.
That might be a working solution.
                
> 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

        

[jira] [Commented] (EXTCDI-249) move PropertyExpressionInterpreter and SystemPropertyExpressionInterpreter to core-api

Posted by "Gerhard Petracek (Commented) (JIRA)" <de...@myfaces.apache.org>.
    [ 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