HHH-11294 - NPE in org.hibernate.jpa.internal.util.PersistenceUtilHelper.isInitialized(PersistentAttributeInterceptable)

hibernate-core/src/main/java/org/hibernate/jpa/internal/util/PersistenceUtilHelper.java

@@ -94,7 +94,7 @@ else if ( reference instanceof PersistentCollection ) {
 	@SuppressWarnings("SimplifiableIfStatement")
 	private static boolean isInitialized(PersistentAttributeInterceptable interceptable) {
 		final LazyAttributeLoadingInterceptor interceptor = extractInterceptor( interceptable );
-		return interceptable == null || !interceptor.hasAnyUninitializedAttributes();
+		return interceptable == null || interceptor == null || !interceptor.hasAnyUninitializedAttributes();
 	}
 
 	private static LazyAttributeLoadingInterceptor extractInterceptor(PersistentAttributeInterceptable interceptable) {

hibernate-core/src/test/java/org/hibernate/jpa/userguide/util/PersistenceUtilHelperTest.java

@@ -8,11 +8,14 @@
 
 import javax.persistence.spi.LoadState;
 
-import org.junit.Test;
+import org.hibernate.engine.spi.PersistentAttributeInterceptable;
-
+import org.hibernate.engine.spi.PersistentAttributeInterceptor;
 import org.hibernate.jpa.internal.util.PersistenceUtilHelper;
 
+import org.junit.Test;
+
 import static org.junit.Assert.assertEquals;
+
 /**
  * Tests for HHH-5094 and HHH-5334
  *
@@ -50,51 +53,96 @@ public String getPublicAccessPropertyValue() {
 			return publicAccessProperty;
 		}
 	}
+
 	@Test
 	public void testIsLoadedWithReferencePublicField() {
 		assertEquals(
 				LoadState.UNKNOWN,
-				PersistenceUtilHelper.isLoadedWithReference( new FieldAccessBean(), "publicAccessProperty", cache )
+				PersistenceUtilHelper.isLoadedWithReference(
+						new FieldAccessBean(),
+						"publicAccessProperty",
+						cache
+				)
 		);
 	}
+
 	@Test
 	public void testIsLoadedWithReferencePublicMethod() {
 		assertEquals(
 				LoadState.UNKNOWN,
 				PersistenceUtilHelper.isLoadedWithReference(
-						new MethodAccessBean(), "publicAccessPropertyValue", cache
+						new MethodAccessBean(),
+						"publicAccessPropertyValue",
+						cache
 				)
 		);
 	}
+
 	@Test
 	public void testIsLoadedWithReferenceProtectedField() {
 		assertEquals(
 				LoadState.UNKNOWN,
-				PersistenceUtilHelper.isLoadedWithReference( new FieldAccessBean(), "protectedAccessProperty", cache )
+				PersistenceUtilHelper.isLoadedWithReference(
+						new FieldAccessBean(),
+						"protectedAccessProperty",
+						cache
+				)
 		);
 	}
+
 	@Test
 	public void testIsLoadedWithReferenceProtectedMethod() {
 		assertEquals(
 				LoadState.UNKNOWN,
 				PersistenceUtilHelper.isLoadedWithReference(
-						new MethodAccessBean(), "protectedAccessPropertyValue", cache
+						new MethodAccessBean(),
+						"protectedAccessPropertyValue",
+						cache
 				)
 		);
 	}
+
 	@Test
 	public void testIsLoadedWithReferencePrivateField() {
 		assertEquals(
 				LoadState.UNKNOWN,
-				PersistenceUtilHelper.isLoadedWithReference( new FieldAccessBean(), "privateAccessProperty", cache )
+				PersistenceUtilHelper.isLoadedWithReference(
+						new FieldAccessBean(),
+						"privateAccessProperty",
+						cache
+				)
 		);
 	}
+
 	@Test
 	public void testIsLoadedWithReferencePrivateMethod() {
 		assertEquals(
 				LoadState.UNKNOWN,
 				PersistenceUtilHelper.isLoadedWithReference(
-						new MethodAccessBean(), "privateAccessPropertyValue", cache
+						new MethodAccessBean(),
+						"privateAccessPropertyValue",
+						cache
+				)
+		);
+	}
+
+	@Test
+	public void testIsLoadedWithNullInterceptor() {
+		assertEquals(
+				LoadState.LOADED,
+				PersistenceUtilHelper.isLoaded(
+						new PersistentAttributeInterceptable() {
+
+							@Override
+							public PersistentAttributeInterceptor $$_hibernate_getInterceptor() {
+								return null;
+							}
+
+							@Override
+							public void $$_hibernate_setInterceptor(PersistentAttributeInterceptor interceptor) {
+
+							}
+						}
 				)
 		);
 	}