You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mc...@apache.org on 2010/01/20 21:45:38 UTC

svn commit: r901372 - in /myfaces/core/trunk/impl: pom.xml src/main/java/org/apache/myfaces/util/ExternalSpecifications.java

Author: mconcini
Date: Wed Jan 20 20:45:37 2010
New Revision: 901372

URL: http://svn.apache.org/viewvc?rev=901372&view=rev
Log:
MYFACES-2498 - add bean validation api dependency to tolerate runtime with api but not impl.

Modified:
    myfaces/core/trunk/impl/pom.xml
    myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/util/ExternalSpecifications.java

Modified: myfaces/core/trunk/impl/pom.xml
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/pom.xml?rev=901372&r1=901371&r2=901372&view=diff
==============================================================================
--- myfaces/core/trunk/impl/pom.xml (original)
+++ myfaces/core/trunk/impl/pom.xml Wed Jan 20 20:45:37 2010
@@ -822,7 +822,12 @@
       <version>1.0.4</version>
       <scope>provided</scope>
     </dependency>
-
+        <dependency>
+            <groupId>javax.validation</groupId>
+            <artifactId>validation-api</artifactId>
+            <version>1.0.CR3</version>
+            <scope>provided</scope>
+        </dependency>
     <dependency>
       <groupId>org.testng</groupId>
       <artifactId>testng</artifactId>
@@ -861,6 +866,7 @@
       <version>2.3</version>
       <scope>test</scope>
     </dependency>
+    
     <dependency>
       <groupId>com.google.code.guice</groupId>
       <artifactId>guice</artifactId>

Modified: myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/util/ExternalSpecifications.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/util/ExternalSpecifications.java?rev=901372&r1=901371&r2=901372&view=diff
==============================================================================
--- myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/util/ExternalSpecifications.java (original)
+++ myfaces/core/trunk/impl/src/main/java/org/apache/myfaces/util/ExternalSpecifications.java Wed Jan 20 20:45:37 2010
@@ -20,6 +20,7 @@
 
 import java.util.logging.Level;
 import java.util.logging.Logger;
+import javax.validation.Validation;
 
 import org.apache.myfaces.shared_impl.util.ClassUtils;
 
@@ -55,7 +56,7 @@
 
             // cannot perform this check, because validation-api.jar is not
             // in the classpath of myfaces-impl
-            /*if (tmp)
+            if (tmp)
             {
                 try
                 {
@@ -67,7 +68,7 @@
                     log.log(Level.FINE, "Error initializing Bean Validation (could be normal)", t);
                     tmp = false;
                 }
-            }*/
+            }
         }
         catch (Throwable t)
         {