[hibernate#1504] Clean up EntityTYpes

* Format code * Remove warnings
DavideD · Jul 5, 2023 · 505ffb6 · 505ffb6
1 parent 60193ef
commit 505ffb6
Showing 1 changed file with 23 additions and 17 deletions.
diff --git a/hibernate-reactive-core/src/main/java/org/hibernate/reactive/engine/impl/EntityTypes.java b/hibernate-reactive-core/src/main/java/org/hibernate/reactive/engine/impl/EntityTypes.java
@@ -24,7 +24,6 @@
 import org.hibernate.reactive.persister.entity.impl.ReactiveEntityPersister;
 import org.hibernate.reactive.session.impl.ReactiveQueryExecutorLookup;
 import org.hibernate.reactive.session.impl.ReactiveSessionImpl;
-import org.hibernate.reactive.util.impl.CompletionStages;
 import org.hibernate.type.EntityType;
 import org.hibernate.type.ForeignKeyDirection;
 import org.hibernate.type.OneToOneType;
@@ -79,8 +78,11 @@ static boolean isNull(EntityType entityType, Object owner, SharedSessionContract
 			OneToOneType type = (OneToOneType) entityType;
 			String propertyName = type.getPropertyName();
 			if ( propertyName != null ) {
-				EntityPersister ownerPersister = session.getFactory().getMetamodel()
-						.entityPersister( entityType.getAssociatedEntityName() );
+				final EntityPersister ownerPersister = session.getFactory()
+						.getRuntimeMetamodels()
+						.getMappingMetamodel()
+						.getEntityDescriptor( entityType.getAssociatedEntityName() );
+
 				Object id = session.getContextEntityIdentifier( owner );
 				EntityKey entityKey = session.generateEntityKey( id, ownerPersister );
 				return session.getPersistenceContextInternal().isPropertyNull( entityKey, propertyName );
@@ -183,7 +185,8 @@ public static CompletionStage<Object[]> replace(
 							 copyCache
 					 ).thenCompose( copy -> {
 						 if ( copy instanceof CompletionStage ) {
-							 return ( (CompletionStage) copy ).thenAccept( nonStageCopy -> copied[i] = nonStageCopy );
+							 return ( (CompletionStage<?>) copy )
+									 .thenAccept( nonStageCopy -> copied[i] = nonStageCopy );
 						 }
 						 else {
 							 copied[i] = copy;
@@ -202,7 +205,7 @@ public static CompletionStage<Object[]> replace(
 			final Type[] types,
 			final SessionImplementor session,
 			final Object owner,
-			final Map copyCache,
+			final Map<Object, Object> copyCache,
 			final ForeignKeyDirection foreignKeyDirection) {
 		Object[] copied = new Object[original.length];
 		for ( int i = 0; i < types.length; i++ ) {
@@ -235,7 +238,7 @@ public static CompletionStage<Object[]> replace(
 							 foreignKeyDirection
 					 ).thenCompose( copy -> {
 						 if ( copy instanceof CompletionStage ) {
-							 return ( (CompletionStage) copy ).thenAccept( nonStageCopy -> copied[i] = nonStageCopy );
+							 return ( (CompletionStage<?>) copy ).thenAccept( nonStageCopy -> copied[i] = nonStageCopy );
 						 }
 						 else {
 							 copied[i] = copy;
@@ -254,7 +257,7 @@ private static CompletionStage<Object> replace(
 			Object target,
 			SessionImplementor session,
 			Object owner,
-			Map copyCache,
+			Map<Object, Object> copyCache,
 			ForeignKeyDirection foreignKeyDirection)
 			throws HibernateException {
 		boolean include = entityType.isAssociationType()
@@ -274,7 +277,7 @@ private static CompletionStage<Object> replace(
 			Object target,
 			SessionImplementor session,
 			Object owner,
-			Map copyCache) {
+			Map<Object, Object> copyCache) {
 		if ( original == null ) {
 			return nullFuture();
 		}
@@ -320,11 +323,12 @@ private static CompletionStage<Object> resolveIdOrUniqueKey(
 			Object original,
 			SessionImplementor session,
 			Object owner,
-			Map copyCache) {
+			Map<Object, Object> copyCache) {
 		return getIdentifier( entityType, original, session )
 				.thenCompose( id -> {
 					if ( id == null ) {
-						throw new AssertionFailure( "non-transient entity has a null id: " + original.getClass().getName() );
+						throw new AssertionFailure( "non-transient entity has a null id: " + original.getClass()
+								.getName() );
 					}
 					// For the special case of a @ManyToOne joined on a (non-primary) unique key,
 					// the "id" class is actually the associated entity object itself, but treated
@@ -336,9 +340,8 @@ private static CompletionStage<Object> resolveIdOrUniqueKey(
 								Object idOrUniqueKey = entityType.getIdentifierOrUniqueKeyType( session.getFactory() )
 										.replace( fetched, null, session, owner, copyCache );
 								if ( idOrUniqueKey instanceof CompletionStage ) {
-									return ( (CompletionStage) idOrUniqueKey ).thenCompose(
-											key -> resolve( entityType, key, owner, session )
-									);
+									return ( (CompletionStage<?>) idOrUniqueKey )
+											.thenCompose( key -> resolve( entityType, key, owner, session ) );
 								}
 
 								return resolve( entityType, idOrUniqueKey, owner, session );
@@ -361,8 +364,8 @@ private static CompletionStage<Object> getIdentifier(
 			return nullFuture();
 		}
 
-		if( value instanceof HibernateProxy ) {
-			return getIdentifierFromHibernateProxy( entityType, (HibernateProxy)value, session );
+		if ( value instanceof HibernateProxy ) {
+			return getIdentifierFromHibernateProxy( entityType, (HibernateProxy) value, session );
 		}
 
 		final LazyInitializer lazyInitializer = extractLazyInitializer( value );
@@ -399,7 +402,10 @@ else if ( isPersistentAttributeInterceptable( value ) ) {
 
 	}
 
-	private static CompletionStage<Object> getIdentifierFromHibernateProxy(EntityType entityType, HibernateProxy proxy, SharedSessionContractImplementor session) {
+	private static CompletionStage<Object> getIdentifierFromHibernateProxy(
+			EntityType entityType,
+			HibernateProxy proxy,
+			SharedSessionContractImplementor session) {
 		LazyInitializer initializer = proxy.getHibernateLazyInitializer();
 		final String entityName = initializer.getEntityName();
 		final Object identifier = initializer.getIdentifier();
@@ -421,7 +427,7 @@ private static CompletionStage<Object> getIdentifierFromHibernateProxy(EntityTyp
 						}
 						return completedFuture( propertyValue );
 					}
-					return CompletionStages.nullFuture();
+					return nullFuture();
 				} );
 	}