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

[Pete Muir (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBSEAM-2133?page=all ]

Pete Muir updated JBSEAM-2133:
------------------------------

     Original Estimate:     (was: 5 minutes)
    Remaining Estimate:     (was: 5 minutes)
            Complexity:   (was: Low)
              Priority: Minor  (was: Critical)
            Workaround:   (was: [Workaround Exists])

I think this is because the support was written to allow injection of components into other components, not construct strings (which would normally be done in messages, which does support this).

As long as this doesn't break the testsuite, +1.

> inline EL in component property value not properly detected
> -----------------------------------------------------------
>
>                 Key: JBSEAM-2133
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-2133
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 2.0.0.CR2
>            Reporter: Dan Allen
>         Assigned To: Dan Allen
>            Priority: Minor
>             Fix For: 2.0.1.GA
>
>         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: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira