[ http://issues.apache.org/struts/browse/SHALE-191?page=all ]
     
Craig McClanahan resolved SHALE-191:
------------------------------------

    Fix Version: 1.0.3
     Resolution: Fixed

Fixed in nightly build 20060627.  @Value is deprecated for backwards 
compatibility, but @Property is the path forward.


> Consider renaming @Value to @Property
> -------------------------------------
>
>          Key: SHALE-191
>          URL: http://issues.apache.org/struts/browse/SHALE-191
>      Project: Shale
>         Type: New Feature

>   Components: Tiger
>     Versions: 1.0.3
>     Reporter: Adam Brod
>      Fix For: 1.0.3

>
> I find it very confusing to use the annotation @Value for a managed property. 
>  I think in terms of the faces-config.xml file, "Managed Beans" and "Managed 
> Properties".  If I want to configure a managed bean with Shale-Tiger, I 
> annotate my class with the @Bean.  If I want to configure my managed 
> property, I would expect to annotate my property with @Property.  Since 
> annotations have a value attribute, it makes sense to me that 
> @Property("#{param.id}") would read as "managed property with value 
> #{param.id}".
> In Shale-Tiger 1.0.3, I have to annotate my property with @Value.  When I 
> look at my code (@Value("#{param.id}")) , it doesn't read very well.  
> "Managed value with value #{param.id}".
> I think it would make more sense to use @Property instead of @Value.  I 
> noticed that Craig accidentally used @Property in an email as well, so 
> clearly I'm not the only one who thinks in terms of @Property and not @Value 
> - 
> http://www.nabble.com/Re%3A--Shale--tiger---abstract-base-class--p4838056.html
>   :)

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

Reply via email to