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

justin miller commented on WW-4331:
-----------------------------------

yes sorry.  returning null allows the HandlerManager to move on to the next 
Handler

> ConventionUnknownHandler throws null
> ------------------------------------
>
>                 Key: WW-4331
>                 URL: https://issues.apache.org/jira/browse/WW-4331
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Plugin - Convention
>    Affects Versions: 2.3.15.2
>         Environment: Tomcat 7.0.50
> Java 1.7.0_40
>            Reporter: justin miller
>            Priority: Minor
>             Fix For: 2.3.18
>
>
> ConventionUnknownHandler's handleUnknownActionMethod() always throws null 
> which causes a NullPointerException when using the Rest-Plugin. This method 
> is called due to RestActionMapper setting the default action method to 
> "index" while the ConventionUnknownHandler creates an ActionSupport proxy 
> expecting the "execute" method.  Throwing null prevents the HandlerManager 
> from handing off to the next UnknownHandler in the list, if one exists.  This 
> is related to issue WW-3368.  Here's the abbreviated stack trace:
> {noformat}
> java.lang.NullPointerException
>       
> org.apache.struts2.convention.ConventionUnknownHandler.handleUnknownActionMethod(ConventionUnknownHandler.java:423)
>       
> com.opensymphony.xwork2.DefaultUnknownHandlerManager.handleUnknownMethod(DefaultUnknownHandlerManager.java:96)
>       
> com.opensymphony.xwork2.DefaultActionInvocation.invokeAction(DefaultActionInvocation.java:437)
>       
> com.opensymphony.xwork2.DefaultActionInvocation.invokeActionOnly(DefaultActionInvocation.java:289)
>       
> com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:252)
>       
> org.apache.struts2.rest.RestActionInvocation.invoke(RestActionInvocation.java:138)
> {noformat}
> Here's the patch:
> 423c423
> <         throw null;
> ---
> >         return null;



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to