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


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