Ignore bridge methods on method handling

finesoft · Jan 27, 2024 · 2c3f319 · 2c3f319
1 parent 215e127
commit 2c3f319
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/corant-config/src/main/java/org/corant/config/PropertyInjector.java b/corant-config/src/main/java/org/corant/config/PropertyInjector.java
@@ -43,6 +43,10 @@ public PropertyInjector(boolean nameUpperCase, Object instance) {
     this.nameUpperCase = nameUpperCase;
     Map<String, Method> setters = new HashMap<>();
     for (Method method : instanceClass.getMethods()) {
+      if (method.isBridge()) {
+        // TODO consider the bridge method "change" visibility of base class's methods ??
+        continue;
+      }
       String name = method.getName();
       if (method.getParameterCount() == 1 && name.startsWith("set")) {
         setters.put(name.substring(3), method);

diff --git a/corant-context/src/main/java/org/corant/context/proxy/ProxyInvocationHandler.java b/corant-context/src/main/java/org/corant/context/proxy/ProxyInvocationHandler.java
@@ -42,7 +42,8 @@ public ProxyInvocationHandler(final Class<?> clazz,
     this.clazz = shouldNotNull(clazz);
     Map<Method, MethodInvoker> methodInvokers = new HashMap<>();
     for (Method method : clazz.getMethods()) {
-      if (method.isDefault() || Modifier.isStatic(method.getModifiers())) {
+      if (method.isDefault() || method.isBridge() || Modifier.isStatic(method.getModifiers())) {
+        // TODO consider the bridge method "change" visibility of base class's methods ??
         continue;
       }
       methodInvokers.put(method, invokerBuilder.apply(method));

diff --git a/corant-context/src/main/java/org/corant/context/proxy/ProxyUtils.java b/corant-context/src/main/java/org/corant/context/proxy/ProxyUtils.java
@@ -95,7 +95,9 @@ public static Map<Method, List<InterceptorInvocation>> getInterceptorChains(
         getInterceptorBindings(beanType.getAnnotations(), beanManager);
 
     for (Method method : beanType.getMethods()) {
-      if ( /* method.isDefault() || */ Modifier.isStatic(method.getModifiers())) {
+      if ( /* method.isDefault() || */method.isBridge()
+          || Modifier.isStatic(method.getModifiers())) {
+        // TODO consider the bridge method "change" visibility of base class's methods ??
         continue;
       }
       List<Annotation> methodLevelBindings =