[
https://jira.jboss.org/jira/browse/JBSEAM-2133?page=com.atlassian.jira.pl...
]
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