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

[hibernate-commits at lists.jboss.org]

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) {