Skip to content

Commit bc6d831

Browse files
committed
HHH-19372 HHH-19369 Issues with access optimizer and inheritance
1 parent 0df8911 commit bc6d831

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

hibernate-core/src/main/java/org/hibernate/bytecode/internal/bytebuddy/BytecodeProviderImpl.java

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -305,17 +305,11 @@ private Class<?> determineAccessOptimizerSuperClass(Class<?> clazz, String[] pro
305305
for ( int i = 0; i < getters.length; i++ ) {
306306
final Member getter = getters[i];
307307
final Member setter = setters[i];
308-
boolean found = false;
309308
if ( getter.getDeclaringClass() == foreignPackageClassInfo.clazz && !Modifier.isPublic( getter.getModifiers() ) ) {
310309
foreignPackageClassInfo.getters.add( getter );
311-
found = true;
312310
}
313311
if ( setter.getDeclaringClass() == foreignPackageClassInfo.clazz && !Modifier.isPublic( setter.getModifiers() ) ) {
314312
foreignPackageClassInfo.setters.add( setter );
315-
found = true;
316-
}
317-
if ( found ) {
318-
foreignPackageClassInfo.propertyNames.add( propertyNames[i] );
319313
}
320314
}
321315
if ( foreignPackageClassInfo.getters.isEmpty() && foreignPackageClassInfo.setters.isEmpty() ) {
@@ -351,7 +345,7 @@ else if ( getter instanceof Method method ) {
351345
TypeDescription.Generic.OfNonGenericType.ForLoadedType.of(
352346
getterType
353347
),
354-
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC
348+
Opcodes.ACC_PUBLIC | Opcodes.ACC_STATIC
355349
)
356350
.withParameter( foreignPackageClassInfo.clazz )
357351
.intercept(
@@ -377,7 +371,7 @@ else if ( setter instanceof Method method ) {
377371
builder = builder.defineMethod(
378372
"set_" + setter.getName(),
379373
TypeDescription.Generic.VOID,
380-
Opcodes.ACC_PROTECTED | Opcodes.ACC_STATIC
374+
Opcodes.ACC_PUBLIC | Opcodes.ACC_STATIC
381375
)
382376
.withParameter( foreignPackageClassInfo.clazz )
383377
.withParameter( setterType )

0 commit comments

Comments
 (0)