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);
}
}