[ 
https://issues.apache.org/struts/browse/WW-2770?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44576#action_44576
 ] 

newton_dave edited comment on WW-2770 at 8/19/08 9:07 AM:
----------------------------------------------------------

This cannot possibly be a "critical" issue; "critical" should be reserved for 
things that actually affect normal operation rather than personal preference.

Using a name rather than a class allows for trivial implementation changes, 
just like the naming of spring beans. Feel free to append a patch, however; it 
will be reviewed and discussed.

      was (Author: newton_dave):
    This can't possibly be a "critical" issue; "critical" should be reserved 
for things that actually affect normal operation rather than personal 
preference.

Using a name rather than a class allows for trivial implementation changes, 
just like the naming of spring beans. Feel free to append a patch, however; it 
will be reviewed and discussed.
  
> @Result type should be a class
> ------------------------------
>
>                 Key: WW-2770
>                 URL: https://issues.apache.org/struts/browse/WW-2770
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Plugin - Convention
>            Reporter: Richard Wallace
>            Priority: Critical
>
> It feels like a huge step backwards from the Codebehind plugin to use a 
> string as the type parameter to the @Result annotation. I much prefer the 
> type safety and IDE support provided by using the actual result class.  The 
> same can be said of the interceptor annotations.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to