Polishing.

Refine type naming.

Author: mp911de

src/main/java/org/springframework/data/core/SerializableLambdaReader.java

@@ -151,7 +151,7 @@ public MemberDescriptor read(Object lambdaObject) {
 			}
 
 			if (captured != null //
-					&& captured instanceof KPropertyReferenceImpl<?, ?> propRef) {
+					&& captured instanceof KPropertyPath<?, ?> propRef) {
 				return KPropertyPathDescriptor.create(propRef);
 			}
 		}

src/main/java/org/springframework/data/core/TypedPropertyPaths.java

@@ -129,7 +129,7 @@ public static <P, T> TypedPropertyPath<T, P> of(PropertyReference<T, P> lambda)
 
 		if (KotlinDetector.isKotlinReflectPresent()) {
 			if (metadata instanceof KPropertyPathMetadata kMetadata
-				&& kMetadata.getProperty() instanceof KPropertyReferenceImpl<?, ?> ref) {
+					&& kMetadata.getProperty() instanceof KPropertyPath<?, ?> ref) {
 				return KotlinDelegate.of(ref);
 			}
 		}
@@ -145,7 +145,7 @@ static class KotlinDelegate {
 		@SuppressWarnings({ "rawtypes", "unchecked" })
 		public static <T, P> TypedPropertyPath<T, P> of(Object property) {
 
-			if (property instanceof KPropertyReferenceImpl paths) {
+			if (property instanceof KPropertyPath paths) {
 
 				TypedPropertyPath parent = of(paths.getProperty());
 				TypedPropertyPath child = of(paths.getLeaf());

src/main/kotlin/org/springframework/data/core/KPropertyExtensions.kt

@@ -39,7 +39,7 @@ fun <T : Any, P> KProperty1<T, P>.toPath(): TypedPropertyPath<T, P> =
 	KTypedPropertyPath.of(this)
 
 /**
- * Builds [KPropertyReferenceImpl] from Property References.
+ * Builds [KPropertyPath] from Property References.
  * Refer to a nested property in an embeddable or association.
  *
  * For example, referring to the field "author.name":
@@ -56,7 +56,7 @@ operator fun <T, M, P> KProperty1<T, M?>.div(other: KProperty1<M, P?>): KPropert
 	KSinglePropertyReference(this, other) as KProperty1<T, P>
 
 /**
- * Builds [KPropertyReferenceImpl] from Property References.
+ * Builds [KPropertyPath] from Property References.
  * Refer to a nested property in an embeddable or association.
  *
  * Note, that this function is different from [div] above in the

src/main/kotlin/org/springframework/data/core/KPropertyReferenceImpl.kt renamed to src/main/kotlin/org/springframework/data/core/KPropertyPath.kt

@@ -28,15 +28,15 @@ import kotlin.reflect.KProperty1
  * @author Yoann de Martino
  * @since 4.1
  */
-internal interface KPropertyReferenceImpl<T, out P> : KProperty1<T, P> {
+internal interface KPropertyPath<T, out P> : KProperty1<T, P> {
 	val property: KProperty1<T, *>
 	val leaf: KProperty1<*, P>
 }
 
 internal class KSinglePropertyReference<T, M, out P>(
 	val parent: KProperty1<T, M?>,
 	val child: KProperty1<M, P>
-) : KProperty1<T, P> by child as KProperty1<T, P>, KPropertyReferenceImpl<T, P> {
+) : KProperty1<T, P> by child as KProperty1<T, P>, KPropertyPath<T, P> {
 
 	override fun get(receiver: T): P {
 
@@ -71,7 +71,7 @@ internal class KSinglePropertyReference<T, M, out P>(
 internal class KIterablePropertyReference<T, M, out P>(
 	val parent: KProperty1<T, Iterable<M?>?>,
 	val child: KProperty1<M, P>
-) : KProperty1<T, P> by child as KProperty1<T, P>, KPropertyReferenceImpl<T, P> {
+) : KProperty1<T, P> by child as KProperty1<T, P>, KPropertyPath<T, P> {
 
 	override fun get(receiver: T): P {
 		throw UnsupportedOperationException("Collection retrieval not supported")
@@ -95,7 +95,7 @@ internal class KIterablePropertyReference<T, M, out P>(
  */
 internal fun asString(property: KProperty<*>): String {
 	return when (property) {
-		is KPropertyReferenceImpl<*, *> ->
+		is KPropertyPath<*, *> ->
 			"${asString(property.property)}.${property.leaf.name}"
 
 		else -> property.name

src/main/kotlin/org/springframework/data/core/PropertyReferenceExtensions.kt

@@ -79,8 +79,8 @@ class KPropertyReference {
 		 */
 		fun <T, P> of(property: KProperty<P?>): PropertyReference<T, P> {
 
-			if (property is KPropertyReferenceImpl<*, *>) {
-				throw IllegalArgumentException("Property reference ${property.name} must not be a property path")
+			if (property is KPropertyPath<*, *>) {
+				throw IllegalArgumentException("Property reference '${property.toDotPath()}' must be a single property reference, not a property path")
 			}
 
 			if (property is KProperty1<*, *>) {
@@ -97,7 +97,7 @@ class KPropertyReference {
 				return PropertyReferences.ResolvedKPropertyReference(metadata)
 			}
 
-			throw IllegalArgumentException("Property ${property.name} is not a KProperty")
+			throw IllegalArgumentException("Property '${property.name}' is not a KProperty")
 		}
 
 	}

src/main/kotlin/org/springframework/data/core/TypedPropertyPathExtensions.kt

@@ -77,9 +77,9 @@ class KTypedPropertyPath {
 		 */
 		fun <T, P> of(property: KProperty<P?>): TypedPropertyPath<T, P> {
 
-			if (property is KPropertyReferenceImpl<*, *>) {
+			if (property is KPropertyPath<*, *>) {
 
-				val paths = property as KPropertyReferenceImpl<*, *>
+				val paths = property as KPropertyPath<*, *>
 
 				val parent = of<Any, Any>(paths.property)
 				val child = of<Any, Any>(paths.leaf)
@@ -104,7 +104,7 @@ class KTypedPropertyPath {
 				return TypedPropertyPaths.ResolvedKPropertyPath(metadata)
 			}
 
-			throw IllegalArgumentException("Property ${property.name} is not a KProperty")
+			throw IllegalArgumentException("Property '${property.name}' is not a KProperty")
 		}
 
 	}

src/test/kotlin/org/springframework/data/core/KTypedPropertyPathUnitTests.kt

@@ -9,7 +9,7 @@ import org.junit.jupiter.params.provider.MethodSource
 import java.util.stream.Stream
 
 /**
- * Unit tests for [KPropertyReferenceImpl] and related functionality.
+ * Unit tests for [KPropertyPath] and related functionality.
  *
  * @author Mark Paluch
  */