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