[
https://issues.apache.org/jira/browse/WW-4331?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14076637#comment-14076637
]
ASF subversion and git services commented on WW-4331:
-----------------------------------------------------
Commit ff100ce2d9c939e9cf4867bc46966a7dfecb6770 in struts's branch
refs/heads/develop from [~lukaszlenart]
[ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=ff100ce ]
WW-4331 Returns null instead of throwing exception
This allows other UnknownHandler to be called
> 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
> Assignee: Lukasz Lenart
> 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)
