You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/11/14 16:13:00 UTC

svn commit: r474811 - /myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java

Author: mmarinschek
Date: Tue Nov 14 07:13:00 2006
New Revision: 474811

URL: http://svn.apache.org/viewvc?view=rev&rev=474811
Log:
cosmetic change: added better error-handling while finding converters

Modified:
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java?view=diff&rev=474811&r1=474810&r2=474811
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/application/ApplicationImpl.java Tue Nov 14 07:13:00 2006
@@ -463,6 +463,10 @@
 
         Class converterClass = (Class) _converterIdToClassMap.get(converterId);
 
+        if(converterClass == null){
+            throw new FacesException("Could not find any registered converter-class for converterId : "+converterId);            
+        }
+
         try {
             Converter converter= (Converter) converterClass.newInstance();
 
@@ -471,8 +475,8 @@
             return converter;
         }
         catch (Exception e) {
-            log.error("Could not instantiate converter " + converterClass, e);
-            throw new FacesException("Could not instantiate converter: " + converterClass, e);
+            log.error("Could not instantiate converter " + converterClass+" for converterId : "+converterId, e);
+            throw new FacesException("Could not instantiate converter " + converterClass+" for converterId : "+converterId, e);
         }
     }