[MYFACES-3049] Bean Validation doesn't work with Glassfish el-impl-2.2

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.4
• Fix Version/s: 2.0.5
• Component/s: JSR-314
• Environment: Tomcat 2.0.29
• Assignee: JCC Van Andel
• Reporter: JCC Van Andel
• Created: 21/Feb/11 10:06
• Updated: 04/Apr/11 21:45
• Resolved: 21/Feb/11 18:13

Description

I have this expression in my Facelet: #

{newPaymentBean.payment.toAccount}

"payment" resolves to the following:

@Entity
public class Payment implements Serializable {

// More stuff...

@NotNull @AccountNumber private String toAccount;

// More stuff...
}

When debugging in javax.faces.validator._BeanValidatorUELUtils, I noticed the following on line 47 "ValueReference valueReference = valueExpression.getValueReference(elCtx);":

• With Glassfish EL, "valueReference.property" is null. This causes the BeanValidator to return at line 161, and to skip validation. "valueReference.base" points to the Payment object btw.
• With JUEL 2.2.3, "valueReference.property" is "toAccount", which is correct AFAIK.

I'm not sure whether this is a MyFaces or EL issue. I remember that when I wrote the BeanValidator, that the spec literally said what to do. See: http://javaserverfaces.java.net/nonav/docs/2.0/javadocs/javax/faces/validator/BeanValidator.html#validate(javax.faces.context.FacesContext, javax.faces.component.UIComponent, java.lang.Object)

So I guess this is an EL implementation issue, but I filed it nevertheless, at least for archiving purposes...

WDYT?