You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by hr...@apache.org on 2006/07/26 01:14:19 UTC

svn commit: r425552 - /struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java

Author: hrabago
Date: Tue Jul 25 16:14:19 2006
New Revision: 425552

URL: http://svn.apache.org/viewvc?rev=425552&view=rev
Log:
Let the user know if an action mapping isn't going anywhere.

Modified:
    struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java

Modified: struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java
URL: http://svn.apache.org/viewvc/struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java?rev=425552&r1=425551&r2=425552&view=diff
==============================================================================
--- struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java (original)
+++ struts/struts1/trunk/core/src/main/java/org/apache/struts/chain/commands/AbstractCreateAction.java Tue Jul 25 16:14:19 2006
@@ -73,7 +73,12 @@
         String type = actionConfig.getType();
 
         if (type == null) {
-            LOG.trace("no type for " + actionConfig.getPath());
+            if ((actionConfig.getForward() == null)
+                && (actionConfig.getInclude() == null)) {
+                LOG.error("no type for " + actionConfig.getPath());
+            } else {
+                LOG.trace("no type for " + actionConfig.getPath());
+            }
 
             return (false);
         }