Commits

Anonymous committed a2a665a
  • Participants
  • Parent commits 76dc5d7

Comments (0)

Files changed (2)

File src/java/com/opensymphony/xwork/DefaultActionInvocation.java

 
         if (aResult != null) {
             aResult.execute(this);
+        } else {
+            LOG.warn("No result defined for action " + getAction().getClass().getName() + " and result " + getResultCode());
         }
     }
 

File src/java/com/opensymphony/xwork/config/providers/XmlConfigurationProvider.java

 
         HashMap actionParams = XmlHelper.getParams(actionElement);
 
-        Map results = buildResults(actionElement, packageContext);
+        Map results;
+
+        try {
+            results = buildResults(actionElement, packageContext);
+        } catch (ConfigurationException e) {
+            throw new ConfigurationException("Error building results for action " + name + " in namespace " + packageContext.getNamespace(), e);
+        }
 
         List interceptorList = buildInterceptorList(actionElement, packageContext);
 
                 }
 
                 ResultTypeConfig config = (ResultTypeConfig) packageContext.getAllResultTypeConfigs().get(resultType);
+
                 if (config == null) {
-                    throw new ConfigurationException("There is no result type defined for " + resultType);
+                    throw new ConfigurationException("There is no result type defined for type '" + resultType + "' mapped with name '" + resultName + "'");
                 }
+
                 Class resultClass = config.getClazz();
 
                 // invalid result type specified in result definition