You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by gp...@apache.org on 2010/05/14 06:20:44 UTC

svn commit: r944105 - /myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java

Author: gpetracek
Date: Fri May 14 04:20:44 2010
New Revision: 944105

URL: http://svn.apache.org/viewvc?rev=944105&view=rev
Log:
minor change

Modified:
    myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java

Modified: myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java?rev=944105&r1=944104&r2=944105&view=diff
==============================================================================
--- myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java (original)
+++ myfaces/extensions/validator/branches/branch_for_jsf_2_0/validation-modules/bean-validation/src/main/java/org/apache/myfaces/extensions/validator/beanval/interceptor/BeanValidationTagAwareValidationInterceptor.java Fri May 14 04:20:44 2010
@@ -81,15 +81,18 @@ public class BeanValidationTagAwareValid
                                    String clientId,
                                    Validator[] validators)
     {
-        List<String> validatorsOfTagList = new ArrayList<String>();
+        List<String> groupsClassNamesOfTagList = new ArrayList<String>();
 
         for (Validator validator : validators)
         {
             //don't check with instanceof
             if (validator.getClass().getName().equals(BeanValidator.class.getName()))
             {
-                validatorsOfTagList.addAll(
-                        Arrays.asList(((BeanValidator) validator).getValidationGroups().split(",")));
+                if(((BeanValidator) validator).getValidationGroups() != null)
+                {
+                    groupsClassNamesOfTagList.addAll(
+                            Arrays.asList(((BeanValidator) validator).getValidationGroups().split(",")));
+                }
 
                 //prevent double-validation
                 editableValueHolder.removeValidator(validator);
@@ -98,7 +101,7 @@ public class BeanValidationTagAwareValid
         }
 
         Class currentClass;
-        for(String groupClassName : validatorsOfTagList)
+        for(String groupClassName : groupsClassNamesOfTagList)
         {
             currentClass = ClassUtils.tryToLoadClassForName(groupClassName);