myfaces-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jwald...@apache.org
Subject svn commit: r1368107 - /myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java
Date Wed, 01 Aug 2012 16:38:20 GMT
Author: jwaldman
Date: Wed Aug  1 16:38:20 2012
New Revision: 1368107

URL: http://svn.apache.org/viewvc?rev=1368107&view=rev
Log:
TRINIDAD-2290 Need to log the exception in each catch block of ResourceServlet._getResourceLoader()
commit version 2 patch. This removes the logging for the catch block (NoSuchMethodException)
that is the expected default path. 
thanks to Mark Yvanovich for the patch.

Modified:
    myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java

Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java?rev=1368107&r1=1368106&r2=1368107&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java
(original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/webapp/ResourceServlet.java
Wed Aug  1 16:38:20 2012
@@ -312,7 +312,9 @@ public class ResourceServlet extends Htt
               Class<?> clazz = cl.loadClass(className);
               try
               {
+                // check if this is a decorator first, and if not, fall back to the no-arg
constructor. 
                 Constructor<?> decorator = clazz.getConstructor(_DECORATOR_SIGNATURE);
+                  
                 // We are now calling a special temp directory version of DirectoryResourceLoader
to
                 // assure that the servlet context's temp directory doesn't change on us.
                 ResourceLoader delegate = new TempDirectoryResourceLoader(getServletContext());
@@ -328,7 +330,6 @@ public class ResourceServlet extends Htt
               catch (NoSuchMethodException e)
               {
                 // by default, create new instance with no-args constructor
-                _logLoaderException(e, servletPath);
                 loader = (ResourceLoader) clazz.newInstance();
               }
             }



Mime
View raw message