struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lukasz Lenart (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (WW-4331) ConventionUnknownHandler throws null
Date Thu, 24 Apr 2014 20:13:20 GMT

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

Lukasz Lenart commented on WW-4331:
-----------------------------------

I assume the solution is to return null instead throwing null?

> 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 diff:
> 423c423
> <         throw null;
> ---
> >         return null;



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

Mime
View raw message