Hibernate/hibernate-orm
1
0
Fork 0
mirror of https://github.com/hibernate/hibernate-orm synced 2025-02-21 10:34:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

HHH-15635 Avoid type pollution in EnhancedSetterImpl

Browse Source
This commit is contained in:
Christian Beikov 2022-10-26 14:07:36 +02:00
parent ec72bab0a9
commit b3b62ad685
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
hibernate-core/src/main/java/org/hibernate/property/access/spi/EnhancedSetterImpl.java
View File

@ -26,11 +26,20 @@
* @author Luis Barreiro * @author Luis Barreiro
*/ */
public class EnhancedSetterImpl extends SetterFieldImpl { public class EnhancedSetterImpl extends SetterFieldImpl {
private static final int COMPOSITE_TRACKER_MASK = 1;
private static final int COMPOSITE_OWNER = 2;
private static final int PERSISTENT_ATTRIBUTE_INTERCEPTABLE_MASK = 4;
private final String propertyName; private final String propertyName;
private final int enhancementState;
public EnhancedSetterImpl(Class<?> containerClass, String propertyName, Field field) { public EnhancedSetterImpl(Class<?> containerClass, String propertyName, Field field) {
super( containerClass, propertyName, field ); super( containerClass, propertyName, field );
this.propertyName = propertyName; this.propertyName = propertyName;
this.enhancementState = ( CompositeOwner.class.isAssignableFrom( containerClass ) ? COMPOSITE_OWNER : 0 )
| ( CompositeTracker.class.isAssignableFrom( field.getType() ) ? COMPOSITE_TRACKER_MASK : 0 )
| ( PersistentAttributeInterceptable.class.isAssignableFrom( containerClass ) ? PERSISTENT_ATTRIBUTE_INTERCEPTABLE_MASK : 0 );
} }
@Override @Override
@ -38,12 +47,12 @@ public void set(Object target, Object value) {
super.set( target, value ); super.set( target, value );
// This sets the component relation for dirty tracking purposes // This sets the component relation for dirty tracking purposes
if ( target instanceof CompositeOwner && value instanceof CompositeTracker ) { if ( ( enhancementState & COMPOSITE_OWNER ) != 0 && ( ( enhancementState & COMPOSITE_TRACKER_MASK ) != 0 || value instanceof CompositeTracker ) ) {
( (CompositeTracker) value ).$$_hibernate_setOwner( propertyName, (CompositeOwner) target ); ( (CompositeTracker) value ).$$_hibernate_setOwner( propertyName, (CompositeOwner) target );
} }
// This marks the attribute as initialized, so it doesn't get lazily loaded afterwards // This marks the attribute as initialized, so it doesn't get lazily loaded afterwards
if ( target instanceof PersistentAttributeInterceptable ) { if ( ( enhancementState & PERSISTENT_ATTRIBUTE_INTERCEPTABLE_MASK ) != 0 ) {
PersistentAttributeInterceptor interceptor = ( (PersistentAttributeInterceptable) target ).$$_hibernate_getInterceptor(); PersistentAttributeInterceptor interceptor = ( (PersistentAttributeInterceptable) target ).$$_hibernate_getInterceptor();
if ( interceptor instanceof BytecodeLazyAttributeInterceptor ) { if ( interceptor instanceof BytecodeLazyAttributeInterceptor ) {
( (BytecodeLazyAttributeInterceptor) interceptor ).attributeInitialized( propertyName ); ( (BytecodeLazyAttributeInterceptor) interceptor ).attributeInitialized( propertyName );