You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Philip Luppens (JIRA)" <ji...@apache.org> on 2006/05/22 11:01:16 UTC

[jira] Created: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
---------------------------------------------------------------------------------

         Key: WW-1322
         URL: http://issues.apache.org/struts/browse/WW-1322
     Project: Struts Action 2
        Type: Improvement

  Components: Interceptors  
    Versions: 2.0    
    Reporter: Philip Luppens


The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Updated: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "Philip Luppens (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]

Philip Luppens updated WW-1322:
-------------------------------

    Attachment: prepareInterceptor-patch.txt

I should really think twice before submitting - when a method wasn't found, an exception was thrown - not a good idea.
Added new patch, swallows the exception. If you think logging it is better, add a logging statement in the catch block.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt, prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Closed: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "James Mitchell (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]
     
James Mitchell closed WW-1322:
------------------------------

    Resolution: Not A Problem

Per request of Phil (via WW chat), closing this because it is a patch for xwork, not SAF.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt, prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Closed: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "James Mitchell (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]
     
James Mitchell closed WW-1322:
------------------------------

    Resolution: Not A Problem

Per request of Phil (via WW chat), closing this because it is a patch for xwork, not SAF.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt, prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Updated: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "Philip Luppens (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]

Philip Luppens updated WW-1322:
-------------------------------

    Attachment: prepareInterceptor-patch.txt

Patch added - executes prepare{methodName} and if not found, prepareDo{methodName} to avoid confusion.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Updated: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "Philip Luppens (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]

Philip Luppens updated WW-1322:
-------------------------------

    Attachment: prepareInterceptor-patch.txt

I should really think twice before submitting - when a method wasn't found, an exception was thrown - not a good idea.
Added new patch, swallows the exception. If you think logging it is better, add a logging statement in the catch block.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt, prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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


[jira] Updated: (WW-1322) Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName

Posted by "Philip Luppens (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/struts/browse/WW-1322?page=all ]

Philip Luppens updated WW-1322:
-------------------------------

    Attachment: prepareInterceptor-patch.txt

Patch added - executes prepare{methodName} and if not found, prepareDo{methodName} to avoid confusion.

> Alter prepareInterceptor to call prepare{MethodName} when doing action!methodName
> ---------------------------------------------------------------------------------
>
>          Key: WW-1322
>          URL: http://issues.apache.org/struts/browse/WW-1322
>      Project: Struts Action 2
>         Type: Improvement

>   Components: Interceptors
>     Versions: 2.0
>     Reporter: Philip Luppens
>  Attachments: prepareInterceptor-patch.txt
>
> The prepareInterceptor should call prepare{MethodName} in addition to a general prepare() method to allow method-specific preparing of Actions.

-- 
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