svn commit: r897266 - /myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java

[ma...@apache.org]

Author: matzew
Date: Fri Jan  8 17:12:19 2010
New Revision: 897266

URL: http://svn.apache.org/viewvc?rev=897266&view=rev
Log:
some more on the not present BeanValidation issue

Modified:
    myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java

Modified: myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java?rev=897266&r1=897265&r2=897266&view=diff
==============================================================================
--- myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java (original)
+++ myfaces/trinidad/branches/trinidad-2.0.x/trinidad-api/src/main/java-templates/org/apache/myfaces/trinidad/component/UIXEditableValueTemplate.java Fri Jan  8 17:12:19 2010
@@ -536,12 +536,11 @@
             // Trial-error approach to check for Bean Validation impl existence.
             Validation.buildDefaultValidatorFactory().getValidator();
           }
-          // should this be ValidationException ?
-          catch (Exception ve)
+          catch (Exception validationException)
           {
             // SPEC section 3.5.6.2:
             // TODO do a i18n version of the error msg
-            throw new FacesException("A ValidatorFactory can not be retrieved", ve);
+            throw new FacesException("A ValidatorFactory can not be retrieved", validationException);
           }
         }
       }
@@ -549,8 +548,8 @@
       {
         // SPEC section 3.5.6.2:
         // if a Bean Validation provider is not present, bean validation is disabled
-        // TODO need a better warning here, which has more information
-        _LOG.warning(cnfe);
+        // TODO need a better warning (i18n) here, which has more information
+        _LOG.warning("A Bean Validation provider is not present, therefore bean validation is disabled");
         couldLoadBeanValidationAPI = Boolean.FALSE;
       }