HV-1623 Clean up property metadata

- changed Constrainable to Property to better represent the intended objects
to be processed by the builder
- removed unused parameters, fields and methods

Author: marko-bekhta

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

@@ -6,10 +6,7 @@
  */
 package org.hibernate.validator.internal.metadata.aggregated;
 
-import java.lang.reflect.Method;
 import java.lang.reflect.Type;
-import java.security.AccessController;
-import java.security.PrivilegedAction;
 import java.util.ArrayDeque;
 import java.util.Collections;
 import java.util.Deque;
@@ -24,7 +21,6 @@
 
 import javax.validation.ElementKind;
 
-import org.hibernate.validator.HibernateValidatorPermission;
 import org.hibernate.validator.internal.engine.valueextraction.ValueExtractorManager;
 import org.hibernate.validator.internal.metadata.core.ConstraintHelper;
 import org.hibernate.validator.internal.metadata.core.MetaConstraint;
@@ -44,7 +40,6 @@
 import org.hibernate.validator.internal.properties.javabean.JavaBeanGetter;
 import org.hibernate.validator.internal.util.CollectionHelper;
 import org.hibernate.validator.internal.util.TypeResolutionHelper;
-import org.hibernate.validator.internal.util.privilegedactions.GetDeclaredMethod;
 import org.hibernate.validator.internal.util.stereotypes.Immutable;
 
 /**
@@ -72,8 +67,7 @@ private PropertyMetaData(String propertyName,
 							 Type type,
 							 Set<MetaConstraint<?>> constraints,
 							 Set<MetaConstraint<?>> containerElementsConstraints,
-							 Set<Cascadable> cascadables,
-							 boolean cascadingProperty) {
+							 Set<Cascadable> cascadables) {
 		super(
 				propertyName,
 				type,
@@ -158,9 +152,8 @@ public static class Builder extends MetaDataBuilder {
 		);
 
 		private final String propertyName;
-		private final Map<Constrainable, Cascadable.Builder> cascadableBuilders = new HashMap<>();
+		private final Map<Property, Cascadable.Builder> cascadableBuilders = new HashMap<>();
 		private final Type propertyType;
-		private boolean cascadingProperty = false;
 
 		public Builder(Class<?> beanClass, ConstrainedField constrainedProperty, ConstraintHelper constraintHelper, TypeResolutionHelper typeResolutionHelper,
 				ValueExtractorManager valueExtractorManager) {
@@ -207,8 +200,6 @@ public boolean accepts(ConstrainedElement constrainedElement) {
 		public final void add(ConstrainedElement constrainedElement) {
 			super.add( constrainedElement );
 
-			cascadingProperty = cascadingProperty || constrainedElement.getCascadingMetaDataBuilder().isCascading();
-
 			if ( constrainedElement.getCascadingMetaDataBuilder().isMarkedForCascadingOnAnnotatedObjectOrContainerElements() ||
 					constrainedElement.getCascadingMetaDataBuilder().hasGroupConversionsOnAnnotatedObjectOrContainerElements() ) {
 
@@ -316,24 +307,6 @@ else if ( constrainedElement.getKind() == ConstrainedElementKind.METHOD ) {
 			return null;
 		}
 
-		/**
-		 * Returns an accessible copy of the given member.
-		 */
-		private Method getAccessible(Method original) {
-			SecurityManager sm = System.getSecurityManager();
-			if ( sm != null ) {
-				sm.checkPermission( HibernateValidatorPermission.ACCESS_PRIVATE_MEMBERS );
-			}
-
-			Class<?> clazz = original.getDeclaringClass();
-
-			return run( GetDeclaredMethod.andMakeAccessible( clazz, original.getName() ) );
-		}
-
-		private <T> T run(PrivilegedAction<T> action) {
-			return System.getSecurityManager() != null ? AccessController.doPrivileged( action ) : action.run();
-		}
-
 		@Override
 		public PropertyMetaData build() {
 			Set<Cascadable> cascadables = cascadableBuilders.values()
@@ -346,8 +319,7 @@ public PropertyMetaData build() {
 					propertyType,
 					adaptOriginsAndImplicitGroups( getDirectConstraints() ),
 					adaptOriginsAndImplicitGroups( getContainerElementConstraints() ),
-					cascadables,
-					cascadingProperty
+					cascadables
 			);
 		}
 	}