Hibernate ORM - Reimplement isProxiable() with Jandex

Author: gsmet

extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/HibernateOrmProcessor.java

@@ -1400,7 +1400,7 @@ private PreGeneratedProxies generatedProxies(Set<String> managedClassAndPackageN
                 } else {
                     Set<String> proxyInterfaceNames = new TreeSet<>();
                     proxyInterfaceNames.add(ClassNames.HIBERNATE_PROXY.toString()); //always added
-                    if (!proxyHelper.isProxiable(managedClassOrPackageName)) {
+                    if (!proxyHelper.isProxiable(combinedIndex.getClassByName(managedClassOrPackageName))) {
                         // we need to make sure the actual class is proxiable
                         continue;
                     }
@@ -1461,7 +1461,7 @@ private boolean isModified(String entity, Set<String> changedClasses, IndexView
             }
         }
         DotName superName = clazz.superName();
-        if (superName != null) {
+        if (superName != null && !DotName.OBJECT_NAME.equals(superName)) {
             return isModified(superName.toString(), changedClasses, index);
         }
         return false;

extensions/hibernate-orm/deployment/src/main/java/io/quarkus/hibernate/orm/deployment/ProxyBuildingHelper.java

@@ -6,11 +6,11 @@
 
 import org.hibernate.bytecode.internal.bytebuddy.BytecodeProviderImpl;
 import org.hibernate.proxy.pojo.bytebuddy.ByteBuddyProxyHelper;
+import org.jboss.jandex.ClassInfo;
 
 import net.bytebuddy.ClassFileVersion;
 import net.bytebuddy.description.method.MethodDescription;
 import net.bytebuddy.description.type.TypeDefinition;
-import net.bytebuddy.description.type.TypeDescription;
 import net.bytebuddy.dynamic.DynamicType;
 import net.bytebuddy.matcher.ElementMatcher;
 import net.bytebuddy.matcher.ElementMatchers;
@@ -52,17 +52,12 @@ private ByteBuddyProxyHelper getByteBuddyProxyHelper() {
         return this.byteBuddyProxyHelper;
     }
 
-    public boolean isProxiable(String managedClassOrPackageName) {
-        TypePool.Resolution mappedClassResolution = typePool.describe(managedClassOrPackageName);
-        if (!mappedClassResolution.isResolved()) {
-            // Probably a package name - consider it's not proxiable.
-            return false;
-        }
-
-        TypeDescription mappedClass = mappedClassResolution.resolve();
-
-        return !mappedClass.isFinal()
-                && !mappedClass.getDeclaredMethods().filter(NO_ARG_CONSTRUCTOR).isEmpty();
+    public boolean isProxiable(ClassInfo classInfo) {
+        return classInfo != null
+                && !classInfo.isInterface()
+                && !classInfo.isAbstract()
+                && !classInfo.isFinal()
+                && classInfo.hasNoArgsConstructor();
     }
 
     @Override