valueExpressionAnalyzer can return a null value. The condition

prevents a NPE
seam · Jun 1, 2011 · 4d9a6ae · 4d9a6ae
1 parent 505f08c
commit 4d9a6ae
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/api/src/main/java/org/jboss/seam/faces/component/UIInputContainer.java b/api/src/main/java/org/jboss/seam/faces/component/UIInputContainer.java
@@ -412,9 +412,11 @@ private boolean isRequiredByConstraint(final EditableValueHolder input, final Va
             if (valueExpression != null) {
                 ValueExpressionAnalyzer valueExpressionAnalyzer = new ValueExpressionAnalyzer(valueExpression);
                 ValueReference vref = valueExpressionAnalyzer.getValueReference(context.getELContext());
-                BeanDescriptor constraintsForClass = validator.getConstraintsForClass(vref.getBase().getClass());
-                PropertyDescriptor d = constraintsForClass.getConstraintsForProperty((String) vref.getProperty());
-                return (d != null) && d.hasConstraints();
+                if (vref != null) { // valueExpressionAnalyzer can return a null value. The condition prevents a NPE
+                    BeanDescriptor constraintsForClass = validator.getConstraintsForClass(vref.getBase().getClass());
+                    PropertyDescriptor d = constraintsForClass.getConstraintsForProperty((String) vref.getProperty());
+                    return (d != null) && d.hasConstraints();
+                }
             }
             return false;
         }