You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Jose L Martinez-Avial (JIRA)" <ji...@apache.org> on 2014/01/10 00:47:51 UTC
[jira] [Created] (WW-4274) DefaultActionInvocation shows invalid
log
Jose L Martinez-Avial created WW-4274:
-----------------------------------------
Summary: DefaultActionInvocation shows invalid log
Key: WW-4274
URL: https://issues.apache.org/jira/browse/WW-4274
Project: Struts 2
Issue Type: Improvement
Components: Core Actions
Reporter: Jose L Martinez-Avial
Priority: Trivial
The method invokeAction of com.opensymphony.xwork2.DefaultActionInvocation logs the following line for each action it executes:
DEBUG com.opensymphony.xwork2.DefaultActionInvocation - Executing action method = null
I was curious about why the method was null, and find the following:
{code:title=DefaultActionInvocation.java|borderStyle=solid}
protected String invokeAction(Object action, ActionConfig actionConfig) throws Exception {
String methodName = proxy.getMethod();
if (LOG.isDebugEnabled()) {
LOG.debug("Executing action method = " + actionConfig.getMethodName());
}
[...]
{code}
but it does not use actionConfig.getMethodName() at all. It uses methodName, from proxy.getMethod(). And actionConfig.getMethodName() is null if no method was declared on the action definition (then it uses the standard execute). I believe that the log should be modified to show methodName instead if actionConfig.getMethodName(). I'm sending attached a patch for this issue.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)