Fixed regression with fallback for non-resolvable property type

Author: jhoeller

spring-beans/src/test/java/org/springframework/beans/BeanWrapperTests.java

@@ -1577,6 +1577,24 @@ public void testPropertyTypeMismatch() {
 		assertEquals(8, bwi.getPropertyValue("object"));
 	}
 
+	@Test
+	public void testGenericArraySetter() {
+		SkipReaderStub foo = new SkipReaderStub();
+		BeanWrapperImpl bwi = new BeanWrapperImpl(foo);
+		List<String> values = new LinkedList<String>();
+		values.add("1");
+		values.add("2");
+		values.add("3");
+		values.add("4");
+		bwi.setPropertyValue("items", values);
+		Object[] result = foo.items;
+		assertEquals(4, result.length);
+		assertEquals("1", result[0]);
+		assertEquals("2", result[1]);
+		assertEquals("3", result[2]);
+		assertEquals("4", result[3]);
+	}
+
 
 	static class Spr10115Bean {
 
@@ -1991,4 +2009,21 @@ public Integer getObject() {
 		}
 	}
 
+
+	public static class SkipReaderStub<T> {
+
+		public T[] items;
+
+		public SkipReaderStub() {
+		}
+
+		public SkipReaderStub(T... items) {
+			this.items = items;
+		}
+
+		public void setItems(T... items) {
+			this.items = items;
+		}
+	}
+
 }

spring-core/src/main/java/org/springframework/core/ResolvableType.java

@@ -628,7 +628,7 @@ public Class<?> resolve() {
 	 * Resolve this type to a {@link java.lang.Class}, returning the specified
 	 * {@code fallback} if the type cannot be resolved. This method will consider bounds
 	 * of {@link TypeVariable}s and {@link WildcardType}s if direct resolution fails;
-	 * however, bounds of Object.class will be ignored.
+	 * however, bounds of {@code Object.class} will be ignored.
 	 * @param fallback the fallback class to use if resolution fails (may be {@code null})
 	 * @return the resolved {@link Class} or the {@code fallback}
 	 * @see #resolve()

spring-core/src/main/java/org/springframework/core/convert/TypeDescriptor.java

@@ -105,7 +105,7 @@ public TypeDescriptor(Field field) {
 	public TypeDescriptor(Property property) {
 		Assert.notNull(property, "Property must not be null");
 		this.resolvableType = ResolvableType.forMethodParameter(property.getMethodParameter());
-		this.type = this.resolvableType.resolve(Object.class);
+		this.type = this.resolvableType.resolve(property.getType());
 		this.annotations = nullSafeAnnotations(property.getAnnotations());
 	}