Author: jeromy Date: Fri Apr 25 01:40:10 2008 New Revision: 651537 URL: http://svn.apache.org/viewvc?rev=651537&view=rev Log: Added more descriptive LOG.error msg for a missing freemarker template
WW-2615 Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/template/FreemarkerTemplateEngine.java Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/template/FreemarkerTemplateEngine.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/template/FreemarkerTemplateEngine.java?rev=651537&r1=651536&r2=651537&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/template/FreemarkerTemplateEngine.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/components/template/FreemarkerTemplateEngine.java Fri Apr 25 01:40:10 2008 @@ -127,7 +127,18 @@ } if (template == null) { - LOG.error("Could not load template " + templateContext.getTemplate()); + if (LOG.isErrorEnabled()) { + LOG.error("Could not load the FreeMarker template named '" + templateContext.getTemplate().getName() +"':"); + for (Template t : templates) { + LOG.error("Attempted: " + getFinalTemplateName(t)); + } + LOG.error("The TemplateLoader provided by the FreeMarker Configuration was a: "+config.getTemplateLoader().getClass().getName()); + if (freemarkerCaching) { + LOG.error("FreeMarker Template caching is enabled ("+StrutsConstants.STRUTS_FREEMARKER_TEMPLATES_CACHE+"=true) and this template is now cached as missing (the TemplateLoader won't be asked again)"); + } else { + LOG.error("FreeMarker Template caching is disabled ("+StrutsConstants.STRUTS_FREEMARKER_TEMPLATES_CACHE+"=false)"); + } + } if (exception != null) { throw exception; } else {