You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bval.apache.org by rm...@apache.org on 2014/12/08 08:57:35 UTC

svn commit: r1643759 - /bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java

Author: rmannibucau
Date: Mon Dec  8 07:57:35 2014
New Revision: 1643759

URL: http://svn.apache.org/r1643759
Log:
avoid NPE

Modified:
    bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java

Modified: bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java
URL: http://svn.apache.org/viewvc/bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java?rev=1643759&r1=1643758&r2=1643759&view=diff
==============================================================================
--- bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java (original)
+++ bval/branches/bval-11/bval-jsr/src/main/java/org/apache/bval/jsr/xml/ValidationParser.java Mon Dec  8 07:57:35 2014
@@ -271,10 +271,14 @@ public class ValidationParser {
         final String parameterNameProvider = xmlConfig.getParameterNameProvider();
         if (targetConfig.getParameterNameProvider() == targetConfig.getDefaultParameterNameProvider()) { // ref ==
             if (parameterNameProvider != null) {
-                final Class<? extends ParameterNameProvider> clazz =
-                    loadClass(parameterNameProvider).asSubclass(ParameterNameProvider.class);
-                targetConfig.parameterNameProviderClass(clazz);
-                log.log(Level.INFO, String.format("Using %s as validation provider.", parameterNameProvider));
+                final Class<?> loaded = loadClass(parameterNameProvider);
+                if (loaded != null) {
+                    final Class<? extends ParameterNameProvider> clazz = loaded.asSubclass(ParameterNameProvider.class);
+                    targetConfig.parameterNameProviderClass(clazz);
+                    log.log(Level.INFO, String.format("Using %s as validation provider.", parameterNameProvider));
+                } else {
+                    log.log(Level.SEVERE, "Can't load " + parameterNameProvider);
+                }
             }
         }
     }