commit 310d9f997a605fc006ce5a03595c9d3ec71a282e
author Thiago H. de Paula Figueiredo <thiago@arsmachina.com.br> Wed Jan 09 00:22:53 2019 -0200
committer Thiago H. de Paula Figueiredo <thiago@arsmachina.com.br> Wed Jan 09 00:22:53 2019 -0200
tree 67b1e2fe8852618b7d6b2bcd58cfdbbe6fba3574
parent 49c45a3abc42465a0a8d61abb456b580bbf2f883

TAP5-2560: Error in GenericsUtils affecting property access

plastic/src/main/java/org/apache/tapestry5/ioc/internal/util/GenericsUtils.java

diff --git a/plastic/src/main/java/org/apache/tapestry5/ioc/internal/util/GenericsUtils.java b/plastic/src/main/java/org/apache/tapestry5/ioc/internal/util/GenericsUtils.java
index 9bf4d00..abbb767 100644
--- a/plastic/src/main/java/org/apache/tapestry5/ioc/internal/util/GenericsUtils.java
+++ b/plastic/src/main/java/org/apache/tapestry5/ioc/internal/util/GenericsUtils.java
@@ -12,8 +12,18 @@
 
 package org.apache.tapestry5.ioc.internal.util;
 
-import java.lang.reflect.*;
+import java.lang.reflect.Array;
+import java.lang.reflect.Field;
+import java.lang.reflect.GenericArrayType;
+import java.lang.reflect.GenericDeclaration;
+import java.lang.reflect.Method;
+import java.lang.reflect.ParameterizedType;
+import java.lang.reflect.Type;
+import java.lang.reflect.TypeVariable;
+import java.lang.reflect.WildcardType;
+import java.util.ArrayList;
 import java.util.LinkedList;
+import java.util.List;
 
 /**
  * Static methods related to the use of JDK 1.5 generics.
@@ -367,15 +377,7 @@
         }
 
         Class theClass = asClass(containingType);
-        Type genericSuperclass = theClass.getGenericSuperclass();
-        while (genericSuperclass != null && // true for interfaces with no superclass
-                !theClass.equals(Object.class) &&
-                !theClass.equals(typeVariableOwner))
-        {
-            stack.addFirst(genericSuperclass);
-            theClass = asClass(genericSuperclass);
-            genericSuperclass = theClass.getGenericSuperclass();
-        }
+        addGenericSuperclasses(theClass, typeVariableOwner, stack);
 
         int i = getTypeVariableIndex(typeVariable);
         Type resolved = typeVariable;
@@ -400,6 +402,34 @@
         return ((TypeVariable) resolved).getBounds()[0];
     }
 
+
+    private static void addGenericSuperclasses(Class theClass, final Class typeVariableOwner, final LinkedList<Type> stack) {
+        Type genericSuperclass = theClass.getGenericSuperclass();
+        while (genericSuperclass != null && // true for interfaces with no superclass
+                !theClass.equals(Object.class) &&
+                !theClass.equals(typeVariableOwner))
+        {
+            stack.addFirst(genericSuperclass);
+            theClass = asClass(genericSuperclass);
+            genericSuperclass = theClass.getGenericSuperclass();
+        }
+        for (Type type : theClass.getGenericInterfaces()) {
+            stack.add(type);
+        }
+        for (Class implementedInterface : getAllImplementedInterfaces(theClass)) {
+            addGenericSuperclasses(implementedInterface, typeVariableOwner, stack);
+        }
+    }
+    
+    private static List<Class> getAllImplementedInterfaces(Class theClass) {
+        List<Class> list = new ArrayList<>();
+        for (Class implementedInterface : theClass.getInterfaces()) {
+            list.add(implementedInterface);
+            list.addAll(getAllImplementedInterfaces(implementedInterface));
+        }
+        return list;
+    }
+
     /**
      * @param type           - something like List&lt;T>[] or List&lt;? extends T>[] or T[]
      * @param containingType - the shallowest type in the hierarchy where type is defined.