HV-1393 Removed Property Kind enum

Author: marko-bekhta

engine/src/main/java/org/hibernate/validator/internal/cfg/context/PropertyConstraintMappingContextImpl.java

@@ -23,6 +23,7 @@
 import org.hibernate.validator.internal.metadata.raw.ConstrainedProperty;
 import org.hibernate.validator.internal.properties.Callable;
 import org.hibernate.validator.internal.properties.Property;
+import org.hibernate.validator.internal.properties.javabean.JavaBeanField;
 import org.hibernate.validator.internal.util.TypeResolutionHelper;
 
 /**
@@ -56,7 +57,7 @@ protected PropertyConstraintMappingContextImpl getThis() {
 
 	@Override
 	public PropertyConstraintMappingContext constraint(ConstraintDef<?, ?> definition) {
-		if ( property.getKind() == Property.Kind.FIELD ) {
+		if ( property instanceof JavaBeanField ) {
 			super.addConstraint(
 					ConfiguredConstraint.forProperty(
 							definition, property
@@ -110,7 +111,7 @@ public ContainerElementConstraintMappingContext containerElementType(int index,
 	}
 
 	ConstrainedElement build(ConstraintHelper constraintHelper, TypeResolutionHelper typeResolutionHelper, ValueExtractorManager valueExtractorManager) {
-		if ( property.getKind() == Property.Kind.FIELD ) {
+		if ( property instanceof JavaBeanField ) {
 			return ConstrainedProperty.forField(
 					ConfigurationSource.API,
 					property,

engine/src/main/java/org/hibernate/validator/internal/metadata/aggregated/PropertyCascadable.java

@@ -13,6 +13,7 @@
 import org.hibernate.validator.internal.engine.valueextraction.ValueExtractorManager;
 import org.hibernate.validator.internal.metadata.facets.Cascadable;
 import org.hibernate.validator.internal.properties.Property;
+import org.hibernate.validator.internal.properties.javabean.JavaBeanField;
 
 /**
  * A {@link Cascadable} backed by a field of a Java bean.
@@ -31,7 +32,7 @@ public class PropertyCascadable implements Cascadable {
 		this.property = property;
 		this.cascadableType = property.getType();
 		this.cascadingMetaData = cascadingMetaData;
-		this.elementType = property.getKind() == Property.Kind.FIELD ? ElementType.FIELD : ElementType.METHOD;
+		this.elementType = property instanceof JavaBeanField ? ElementType.FIELD : ElementType.METHOD;
 	}
 
 	@Override

engine/src/main/java/org/hibernate/validator/internal/properties/Property.java

@@ -11,14 +11,7 @@
  */
 public interface Property extends Constrainable {
 
-	enum Kind {
-		FIELD,
-		GETTER,
-	}
-
 	Object getValueFrom(Object bean);
 
-	Kind getKind();
-
 	String getPropertyName();
 }

engine/src/main/java/org/hibernate/validator/internal/properties/javabean/JavaBeanField.java

@@ -58,11 +58,6 @@ public Object getValueFrom(Object bean) {
 		return ReflectionHelper.getValue( field, bean );
 	}
 
-	@Override
-	public Kind getKind() {
-		return Kind.FIELD;
-	}
-
 	@Override
 	public String getPropertyName() {
 		return getName();
@@ -121,7 +116,7 @@ private static Field getAccessible(Field original) {
 
 	/**
 	 * Runs the given privileged action, using a privileged block if required.
-	 * <p>
+	 *
 	 * <b>NOTE:</b> This must never be changed into a publicly available method to avoid execution of arbitrary
 	 * privileged actions within HV's protection domain.
 	 */

engine/src/main/java/org/hibernate/validator/internal/properties/javabean/JavaBeanGetter.java

@@ -37,11 +37,6 @@ public Object getValueFrom(Object bean) {
 		return ReflectionHelper.getValue( (Method) executable, bean );
 	}
 
-	@Override
-	public Kind getKind() {
-		return Kind.GETTER;
-	}
-
 	@Override
 	public String getPropertyName() {
 		return name;
@@ -120,7 +115,7 @@ private static Method getAccessible(Method original) {
 
 	/**
 	 * Runs the given privileged action, using a privileged block if required.
-	 * <p>
+	 *
 	 * <b>NOTE:</b> This must never be changed into a publicly available method to avoid execution of arbitrary
 	 * privileged actions within HV's protection domain.
 	 */