[hibernate-commits] Hibernate SVN: r18141 - validator/trunk/hibernate-validator/src/main/java/org/hibernate/validator/engine/resolver.

Author: hardy.ferentschik Date: 2009-12-05 07:07:27 -0500 (Sat, 05 Dec 2009) New Revision: 18141 Modified: validator/trunk/hibernate-validator/src/main/java/org/hibernate/validator/engine/resolver/JPATraversableResolver.java Log: HV-266 Added a null check for traversableProperty.getName() Modified: validator/trunk/hibernate-validator/src/main/java/org/hibernate/validator/engine/resolver/JPATraversableResolver.java =================================================================== --- validator/trunk/hibernate-validator/src/main/java/org/hibernate/validator/engine/resolver/JPATraversableResolver.java 2009-12-05 12:01:41 UTC (rev 18140) +++ validator/trunk/hibernate-validator/src/main/java/org/hibernate/validator/engine/resolver/JPATraversableResolver.java 2009-12-05 12:07:27 UTC (rev 18141) @@ -28,10 +28,10 @@ */ public class JPATraversableResolver implements TraversableResolver { - // TODO Check the call to PersistenceUtil. traversableProperty.getName() is this correct? + // we have to check traversableProperty.getName() against null to check the root gets validated (see HV-266) public boolean isReachable(Object traversableObject, Path.Node traversableProperty, Class<?> rootBeanType, Path pathToTraversableObject, ElementType elementType) { - return traversableObject == null || - Persistence.getPersistenceUtil().isLoaded( traversableObject, traversableProperty.getName() ); + return traversableObject == null || traversableProperty.getName() == null || Persistence.getPersistenceUtil() + .isLoaded( traversableObject, traversableProperty.getName() ); } public boolean isCascadable(Object traversableObject, Path.Node traversableProperty, Class<?> rootBeanType, Path pathToTraversableObject, ElementType elementType) {