Skip to content

Commit 3de4e93

Browse files
committed
Javadoc notes on isSimpleValueType in BeanUtils vs ClassUtils
See gh-30664
1 parent 93345de commit 3de4e93

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

spring-beans/src/main/java/org/springframework/beans/BeanUtils.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -665,6 +665,8 @@ public static boolean isSimpleProperty(Class<?> type) {
665665
* a {@code Temporal}, a {@code UUID}, a {@code URI}, a {@code URL},
666666
* a {@code Locale}, or a {@code Class}.
667667
* <p>{@code Void} and {@code void} are not considered simple value types.
668+
* <p>As of 6.1, this method delegates to {@link ClassUtils#isSimpleValueType}
669+
* as-is but could potentially add further rules for bean property purposes.
668670
* @param type the type to check
669671
* @return whether the given type represents a "simple" value type
670672
* @see #isSimpleProperty(Class)

spring-core/src/main/java/org/springframework/util/ClassUtils.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -544,7 +544,8 @@ public static Class<?> resolvePrimitiveIfNecessary(Class<?> clazz) {
544544
* a {@code Locale}, or a {@code Class}.
545545
* <p>{@code Void} and {@code void} are not considered simple value types.
546546
* @param type the type to check
547-
* @return whether the given type represents a "simple" value type
547+
* @return whether the given type represents a "simple" value type,
548+
* suggesting value-based data binding and {@code toString} output
548549
* @since 6.1
549550
*/
550551
public static boolean isSimpleValueType(Class<?> type) {
@@ -555,7 +556,7 @@ public static boolean isSimpleValueType(Class<?> type) {
555556
Number.class.isAssignableFrom(type) ||
556557
Date.class.isAssignableFrom(type) ||
557558
Temporal.class.isAssignableFrom(type) ||
558-
UUID.class.isAssignableFrom(type) ||
559+
UUID.class == type ||
559560
URI.class == type ||
560561
URL.class == type ||
561562
Locale.class == type ||

0 commit comments

Comments
 (0)