[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-2133) inline EL in component property value not properly detected

[Dan Allen (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBSEAM-2133?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12430825#action_12430825 ] 

Dan Allen commented on JBSEAM-2133:
-----------------------------------

I have made the decision to implement the following policy. If an initial value (a <property> in components.xml or a key in a seam.properties file) contains EL, it will be evaluated before being assigned to the bean property on the component, with on exception. If the property's type is a ValueExpression or MethodExpression (from the Seam library), then the evaluation will be deferred. You can either get the expression out using getExpressionString() or you can evaluate it when it is right for your application using getValue() (ValueExpression) or invoke() (MethodExpression).

> inline EL in component property value not properly detected
> -----------------------------------------------------------
>
>                 Key: JBSEAM-2133
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-2133
>             Project: Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 2.0.0.CR2
>            Reporter: Dan Allen
>            Assignee: Dan Allen
>             Fix For: 2.1.0.CR1
>
>         Attachments: JBSEAM-2133-v1.txt
>
>
> Currently, the EL notation in a property is only being evaluated if the property starts with "#{". There are two problems with this assumption.
> 1. Inline EL can only be used if the property starts with an expression (i.e. <property name="message">#{applicationName} is a cool application</property>)
> 2. There is no way to escape EL so that is is evaluated by the property accepting the string (rather than when the value is assigned).
> The correct behavior would be to treat the property value as a value expression if it CONTAINS "#{". The benefits are two fold:
> 1. A property value can use inline EL that is evaluated when the property value is set (see above)
> 2. The resolved value of the property can be EL (delayed EL) that is to be interpreted by the component as needed. This is done by escaping the EL per the unified EL spec (i.e. <property name="created-message">The user \#{user.name} has been created!</property)
> Surprisingly, everything is already in place. All that needs to happen is the check for EL needs to change from "startsWith" to "contains".

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira