HHH-11459 - Always resolve unloaded, lazy collection in enhanced entity when cascading; added comments

This commit is contained in:
Gail Badner 2017-03-24 19:15:21 -07:00
parent 41bd06266d
commit 0b6ce73fcc
1 changed files with 18 additions and 3 deletions

View File

@ -93,15 +93,30 @@ public final class Cascade {
if ( style.doCascade( action ) ) { if ( style.doCascade( action ) ) {
Object child; Object child;
// For bytecode enhanced entities, need to fetch the attribute if ( hasUninitializedLazyProperties &&
if ( hasUninitializedLazyProperties && persister.getPropertyLaziness()[i] && action.performOnLazyProperty() ) { !persister.getInstrumentationMetadata().isAttributeLoaded( parent, propertyName ) ) {
// parent is a bytecode enhanced entity.
// cascading to an uninitialized, lazy value.
if ( types[i].isCollectionType() ) { if ( types[i].isCollectionType() ) {
// The collection does not need to be loaded from the DB.
// CollectionType#resolve will return an uninitialized PersistentCollection.
// The action will initialize the collection later, if necessary.
child = types[i].resolve( LazyPropertyInitializer.UNFETCHED_PROPERTY, eventSource, parent ); child = types[i].resolve( LazyPropertyInitializer.UNFETCHED_PROPERTY, eventSource, parent );
// TODO: it would be nice to be able to set the attribute in parent using
// persister.setPropertyValue( parent, i, child ).
// Unfortunately, that would cause the uninitialized collection to be
// loaded from the DB.
} }
else { else if ( action.performOnLazyProperty() ) {
// The (non-collection) attribute needs to be initialized so that
// the action can be performed on the initialized attribute.
LazyAttributeLoadingInterceptor interceptor = persister.getInstrumentationMetadata().extractInterceptor( parent ); LazyAttributeLoadingInterceptor interceptor = persister.getInstrumentationMetadata().extractInterceptor( parent );
child = interceptor.fetchAttribute( parent, propertyName ); child = interceptor.fetchAttribute( parent, propertyName );
} }
else {
// Nothing to do, so just skip cascading to this lazy (non-collection) attribute.
continue;
}
} }
else { else {
child = persister.getPropertyValue( parent, i ); child = persister.getPropertyValue( parent, i );