root
/
spring-framework
mirror of https://github.com/spring-projects/spring-framework.git
1
0
Fork 0
Code Issues Actions 10 Packages Projects Releases Wiki Activity

Restore lenient matching of unresolved nested bound 

Closes gh-34541
This commit is contained in:
Juergen Hoeller 2025-03-06 15:44:12 +01:00
parent c5ecc50bfe
commit 63c8e7cb5d
2 changed files with 24 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
spring-core/src/main/java/org/springframework/core/ResolvableType.java
View File

@ -343,7 +343,7 @@ public class ResolvableType implements Serializable {
return otherBounds.isAssignableFrom(this, matchedBefore); return otherBounds.isAssignableFrom(this, matchedBefore);
} }
else if (!strict) { else if (!strict) {
return (matchedBefore != null ? otherBounds.equalsType(this) : return (matchedBefore != null ? otherBounds.equalsType(this, matchedBefore) :
otherBounds.isAssignableTo(this, matchedBefore)); otherBounds.isAssignableTo(this, matchedBefore));
} }
else { else {
@ -1794,11 +1794,13 @@ public class ResolvableType implements Serializable {
* Return {@code true} if these bounds are equal to the specified type. * Return {@code true} if these bounds are equal to the specified type.
* @param type the type to test against * @param type the type to test against
* @return {@code true} if these bounds are equal to the type * @return {@code true} if these bounds are equal to the type
* @since 6.2.3 * @since 6.2.4
*/ */
public boolean equalsType(ResolvableType type) { public boolean equalsType(ResolvableType type, @Nullable Map<Type, Type> matchedBefore) {
for (ResolvableType bound : this.bounds) { for (ResolvableType bound : this.bounds) {
if (!type.equalsType(bound)) { if (this.kind == Kind.UPPER && bound.hasUnresolvableGenerics() ?
!type.isAssignableFrom(bound, true, matchedBefore, false) :
!type.equalsType(bound)) {
return false; return false;
} }
} }

18
spring-core/src/test/java/org/springframework/core/ResolvableTypeTests.java
View File

@ -1527,6 +1527,12 @@ class ResolvableTypeTests {
assertThat(repository3.isAssignableFromResolvedPart(repository2)).isTrue(); assertThat(repository3.isAssignableFromResolvedPart(repository2)).isTrue();
} }
@Test
void gh34541() throws Exception {
ResolvableType typeWithGenerics = ResolvableType.forField(getClass().getDeclaredField("paymentCreator"));
assertThat(typeWithGenerics.isAssignableFrom(PaymentCreator.class)).isTrue();
}
private ResolvableType testSerialization(ResolvableType type) throws Exception { private ResolvableType testSerialization(ResolvableType type) throws Exception {
ByteArrayOutputStream bos = new ByteArrayOutputStream(); ByteArrayOutputStream bos = new ByteArrayOutputStream();
@ -1928,6 +1934,18 @@ class ResolvableTypeTests {
} }
PaymentCreator<? extends Payment, PaymentCreatorParameter<? extends Payment>> paymentCreator;
static class PaymentCreator<T extends Payment, P extends PaymentCreatorParameter<T>> {
}
static class PaymentCreatorParameter<T extends Payment> {
}
abstract static class Payment {
}
private static class ResolvableTypeAssert extends AbstractAssert<ResolvableTypeAssert, ResolvableType>{ private static class ResolvableTypeAssert extends AbstractAssert<ResolvableTypeAssert, ResolvableType>{
public ResolvableTypeAssert(ResolvableType actual) { public ResolvableTypeAssert(ResolvableType actual) {