Fixed dependency cycle.

Closes #2514
This commit is contained in:
Jens Schauder
2021-12-14 14:55:27 +01:00
parent ecec2e5666
commit 6cf2634a64
3 changed files with 7 additions and 7 deletions

View File

@@ -1,353 +0,0 @@
/*
* Copyright 2021 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.data.mapping.context;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.function.Consumer;
import org.springframework.data.mapping.PropertyPath;
import org.springframework.data.projection.ProjectionInformation;
import org.springframework.data.util.ClassTypeInformation;
import org.springframework.data.util.Streamable;
import org.springframework.data.util.TypeInformation;
import org.springframework.lang.Nullable;
/**
* Descriptor for a top-level mapped type representing a view onto a domain type structure. The view may exactly match
* the domain type or be a DTO/interface {@link #isProjection() projection}.
*
* @param <M> the mapped type acting as view onto the domain type.
* @param <D> the domain type.
* @author Mark Paluch
* @author Christoph Strobl
* @since 2.7
*/
public class EntityProjection<M, D> implements Streamable<EntityProjection.PropertyProjection<?, ?>> {
private final TypeInformation<M> mappedType;
private final TypeInformation<D> domainType;
private final List<PropertyProjection<?, ?>> properties;
private final boolean projection;
private final ProjectionType projectionType;
EntityProjection(TypeInformation<M> mappedType, TypeInformation<D> domainType,
List<PropertyProjection<?, ?>> properties, boolean projection, ProjectionType projectionType) {
this.mappedType = mappedType;
this.domainType = domainType;
this.properties = new ArrayList<>(properties);
this.projection = projection;
this.projectionType = projectionType;
}
/**
* Create a projecting variant of a mapped type.
*
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @param properties properties to include.
* @param projectionType must not be {@literal null}.
* @return new instance of {@link EntityProjection}.
*/
public static <M, D> EntityProjection<M, D> projecting(TypeInformation<M> mappedType, TypeInformation<D> domainType,
List<PropertyProjection<?, ?>> properties, ProjectionType projectionType) {
return new EntityProjection<>(mappedType, domainType, properties, true, projectionType);
}
/**
* Create a non-projecting variant of a mapped type.
*
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @param properties properties to include.
* @return new instance of {@link EntityProjection}.
*/
public static <M, D> EntityProjection<M, D> nonProjecting(TypeInformation<M> mappedType,
TypeInformation<D> domainType, List<PropertyProjection<?, ?>> properties) {
return new EntityProjection<>(mappedType, domainType, properties, false, ProjectionType.CLOSED);
}
/**
* Create a non-projecting variant of a {@code type}.
*
* @param type must not be {@literal null}.
* @return new instance of {@link EntityProjection}.
*/
public static <T> EntityProjection<T, T> nonProjecting(Class<T> type) {
ClassTypeInformation<T> typeInformation = ClassTypeInformation.from(type);
return new EntityProjection<>(typeInformation, typeInformation, Collections.emptyList(), false,
ProjectionType.CLOSED);
}
/**
* Performs the given action for each element of the {@link Streamable} recursively until all elements of the graph
* have been processed or the action throws an {@link Exception}. Unless otherwise specified by the implementing
* class, actions are performed in the order of iteration (if an iteration order is specified). Exceptions thrown by
* the action are relayed to the caller.
*
* @param action must not be {@literal null}.
*/
public void forEachRecursive(Consumer<? super PropertyProjection<?, ?>> action) {
for (PropertyProjection<?, ?> descriptor : properties) {
if (descriptor instanceof ContainerPropertyProjection) {
action.accept(descriptor);
descriptor.forEachRecursive(action);
} else if (descriptor.getProperties().isEmpty()) {
action.accept(descriptor);
} else {
descriptor.forEachRecursive(action);
}
}
}
@Override
public Iterator<PropertyProjection<?, ?>> iterator() {
return properties.iterator();
}
/**
* @return the mapped type used by this type view.
*/
public TypeInformation<M> getMappedType() {
return mappedType;
}
/**
* @return the actual mapped type used by this type view. Should be used for collection-like and map-like properties
* to determine the actual view type.
* @throws IllegalStateException if the actual type cannot be resolved.
*/
public TypeInformation<?> getActualMappedType() {
return mappedType.getRequiredActualType();
}
/**
* @return the domain type represented by this type view.
*/
public TypeInformation<D> getDomainType() {
return domainType;
}
/**
* @return the actual domain type represented by this type view. Should be used for collection-like and map-like
* properties to determine the actual domain type.
* @throws IllegalStateException if the actual type cannot be resolved.
*/
public TypeInformation<?> getActualDomainType() {
return domainType.getRequiredActualType();
}
/**
* @return {@code true} if the {@link #getMappedType()} is a projection.
*/
public boolean isProjection() {
return projection;
}
/**
* @return {@code true} if the {@link #getMappedType()} is a closed projection.
*/
public boolean isClosedProjection() {
return isProjection()
&& (ProjectionType.CLOSED.equals(projectionType) || ProjectionType.DTO.equals(projectionType));
}
List<PropertyProjection<?, ?>> getProperties() {
return properties;
}
/**
* Return a {@link EntityProjection} for a property identified by {@code name}.
*
* @param name the property name.
* @return the type view, if the property is known; {@code null} otherwise.
*/
@Nullable
public EntityProjection<?, ?> findProperty(String name) {
for (PropertyProjection<?, ?> descriptor : properties) {
if (descriptor.propertyPath.getLeafProperty().getSegment().equals(name)) {
return descriptor;
}
}
return null;
}
@Override
public String toString() {
if (isProjection()) {
return String.format("Projection(%s AS %s): %s", getActualDomainType().getType().getName(),
getActualMappedType().getType().getName(), properties);
}
return String.format("Domain(%s): %s", getActualDomainType().getType().getName(), properties);
}
/**
* Descriptor for a property-level type along its potential projection.
*
* @param <M> the mapped type acting as view onto the domain type.
* @param <D> the domain type.
*/
public static class PropertyProjection<M, D> extends EntityProjection<M, D> {
private final PropertyPath propertyPath;
PropertyProjection(PropertyPath propertyPath, TypeInformation<M> mappedType, TypeInformation<D> domainType,
List<PropertyProjection<?, ?>> properties, boolean projecting, ProjectionType projectionType) {
super(mappedType, domainType, properties, projecting, projectionType);
this.propertyPath = propertyPath;
}
/**
* Create a projecting variant of a mapped type.
*
* @param propertyPath the {@link PropertyPath path} to the actual property.
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @param properties properties to include.
* @param projectionType must not be {@literal null}.
* @return new instance of {@link PropertyProjection}.
*/
public static <M, D> PropertyProjection<M, D> projecting(PropertyPath propertyPath, TypeInformation<M> mappedType,
TypeInformation<D> domainType, List<PropertyProjection<?, ?>> properties, ProjectionType projectionType) {
return new PropertyProjection<>(propertyPath, mappedType, domainType, properties, true, projectionType);
}
/**
* Create a non-projecting variant of a mapped type.
*
* @param propertyPath the {@link PropertyPath path} to the actual property.
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @return
*/
public static <M, D> PropertyProjection<M, D> nonProjecting(PropertyPath propertyPath,
TypeInformation<M> mappedType, TypeInformation<D> domainType) {
return new PropertyProjection<>(propertyPath, mappedType, domainType, Collections.emptyList(), false,
ProjectionType.OPEN);
}
/**
* @return the property path representing this property within the root domain type.
*/
public PropertyPath getPropertyPath() {
return propertyPath;
}
@Override
public String toString() {
return String.format("%s AS %s", propertyPath.toDotPath(), getActualMappedType().getType().getName());
}
}
/**
* Descriptor for a property-level type along its potential projection that is held within a {@link Collection}-like
* or {@link Map}-like container. Property paths within containers use the deeply unwrapped actual type of the
* container as root type and as they cannot be tied immediately to the root entity.
*
* @param <M> the mapped type acting as view onto the domain type.
* @param <D> the domain type.
*/
public static class ContainerPropertyProjection<M, D> extends PropertyProjection<M, D> {
ContainerPropertyProjection(PropertyPath propertyPath, TypeInformation<M> mappedType, TypeInformation<D> domainType,
List<PropertyProjection<?, ?>> properties, boolean projecting, ProjectionType projectionType) {
super(propertyPath, mappedType, domainType, properties, projecting, projectionType);
}
/**
* Create a projecting variant of a mapped type.
*
* @param propertyPath the {@link PropertyPath path} to the actual property.
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @param properties properties to include.
* @param projectionType must not be {@literal null}.
* @return new instance of {@link ContainerPropertyProjection}.
*/
public static <M, D> ContainerPropertyProjection<M, D> projecting(PropertyPath propertyPath,
TypeInformation<M> mappedType, TypeInformation<D> domainType, List<PropertyProjection<?, ?>> properties,
ProjectionType projectionType) {
return new ContainerPropertyProjection<>(propertyPath, mappedType, domainType, properties, true, projectionType);
}
/**
* Create a non-projecting variant of a mapped type.
*
* @param propertyPath the {@link PropertyPath path} to the actual property.
* @param mappedType the target projection type. Must not be {@literal null}.
* @param domainType the source domain type. Must not be {@literal null}.
* @return new instance of {@link ContainerPropertyProjection}.
*/
public static <M, D> ContainerPropertyProjection<M, D> nonProjecting(PropertyPath propertyPath,
TypeInformation<M> mappedType, TypeInformation<D> domainType) {
return new ContainerPropertyProjection<>(propertyPath, mappedType, domainType, Collections.emptyList(), false,
ProjectionType.OPEN);
}
}
/**
* Projection type.
*
* @since 2.7
*/
public enum ProjectionType {
/**
* A DTO projection defines a value type that hold properties for the fields that are supposed to be retrieved.
*/
DTO,
/**
* An open projection has accessor methods in the interface that can be used to compute new values by using the
* {@link org.springframework.beans.factory.annotation.Value} annotation.
*/
OPEN,
/**
* A closed projection only contains accessor methods that all match properties of the target aggregate.
*/
CLOSED;
/**
* Obtain the {@link ProjectionType} from a given {@link ProjectionInformation}.
*
* @param information must not be {@literal null}.
* @return the {@link ProjectionType} according to {@link ProjectionInformation#getType() type} and
* {@link ProjectionInformation#isClosed()}.
*/
public static ProjectionType from(ProjectionInformation information) {
if (!information.getType().isInterface()) {
return DTO;
}
return information.isClosed() ? CLOSED : OPEN;
}
}
}

View File

@@ -1,272 +0,0 @@
/*
* Copyright 2021 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.data.mapping.context;
import java.beans.PropertyDescriptor;
import java.util.ArrayList;
import java.util.Collections;
import java.util.LinkedHashSet;
import java.util.List;
import java.util.Set;
import org.springframework.data.mapping.PersistentEntity;
import org.springframework.data.mapping.PersistentProperty;
import org.springframework.data.mapping.PropertyPath;
import org.springframework.data.mapping.context.EntityProjection.ProjectionType;
import org.springframework.data.projection.ProjectionFactory;
import org.springframework.data.projection.ProjectionInformation;
import org.springframework.data.util.ClassTypeInformation;
import org.springframework.data.util.TypeInformation;
import org.springframework.lang.Nullable;
import org.springframework.util.Assert;
/**
* This class is introspects the returned type in the context of a domain type for all reachable properties (w/o cycles)
* to determine which property paths are subject to projection.
*
* @author Gerrit Meier
* @author Mark Paluch
* @author Christoph Strobl
* @since 2.7
*/
public class EntityProjectionIntrospector {
private final ProjectionFactory projectionFactory;
private final ProjectionPredicate projectionPredicate;
private final MappingContext<?, ?> mappingContext;
private EntityProjectionIntrospector(ProjectionFactory projectionFactory, ProjectionPredicate projectionPredicate,
MappingContext<?, ?> mappingContext) {
this.projectionFactory = projectionFactory;
this.projectionPredicate = projectionPredicate;
this.mappingContext = mappingContext;
}
/**
* Create a new {@link EntityProjectionIntrospector} given {@link ProjectionFactory}, {@link ProjectionPredicate} and
* {@link MappingContext}.
*
* @param projectionFactory must not be {@literal null}.
* @param projectionPredicate must not be {@literal null}.
* @param mappingContext must not be {@literal null}.
* @return a new {@link EntityProjectionIntrospector} instance.
*/
public static EntityProjectionIntrospector create(ProjectionFactory projectionFactory,
ProjectionPredicate projectionPredicate, MappingContext<?, ?> mappingContext) {
Assert.notNull(projectionFactory, "ProjectionFactory must not be null");
Assert.notNull(projectionPredicate, "ProjectionPredicate must not be null");
Assert.notNull(mappingContext, "MappingContext must not be null");
return new EntityProjectionIntrospector(projectionFactory, projectionPredicate, mappingContext);
}
/**
* Introspect a {@link Class mapped type} in the context of a {@link Class domain type} whether the returned type is a
* projection and what property paths are participating in the projection.
* <p>
* Nested properties (direct types, within maps, collections) are introspected for nested projections and contain
* property paths for closed projections.
* <p>
* Deeply nested types (e.g. {@code Map&lt;?, List&lt;Person&gt;&gt;}) are represented with a property path that uses
* the unwrapped type and no longer the root domain type {@code D}.
*
* @param mappedType must not be {@literal null}.
* @param domainType must not be {@literal null}.
* @return the introspection result.
* @see org.springframework.data.mapping.context.EntityProjection.ContainerPropertyProjection
*/
public <M, D> EntityProjection<M, D> introspect(Class<M> mappedType, Class<D> domainType) {
ClassTypeInformation<M> returnedTypeInformation = ClassTypeInformation.from(mappedType);
ClassTypeInformation<D> domainTypeInformation = ClassTypeInformation.from(domainType);
boolean isProjection = projectionPredicate.test(mappedType, domainType);
if (!isProjection) {
return EntityProjection.nonProjecting(returnedTypeInformation, domainTypeInformation, Collections.emptyList());
}
ProjectionInformation projectionInformation = projectionFactory.getProjectionInformation(mappedType);
if (!projectionInformation.isClosed()) {
return EntityProjection.projecting(returnedTypeInformation, domainTypeInformation, Collections.emptyList(),
ProjectionType.OPEN);
}
PersistentEntity<?, ?> persistentEntity = mappingContext.getRequiredPersistentEntity(domainType);
List<EntityProjection.PropertyProjection<?, ?>> propertyDescriptors = getProperties(null, projectionInformation,
returnedTypeInformation, persistentEntity, null);
return EntityProjection.projecting(returnedTypeInformation, domainTypeInformation, propertyDescriptors, ProjectionType.CLOSED);
}
private List<EntityProjection.PropertyProjection<?, ?>> getProperties(@Nullable PropertyPath propertyPath,
ProjectionInformation projectionInformation, TypeInformation<?> projectionTypeInformation,
PersistentEntity<?, ?> persistentEntity, @Nullable CycleGuard cycleGuard) {
List<EntityProjection.PropertyProjection<?, ?>> propertyDescriptors = new ArrayList<>();
// TODO: PropertyDescriptor only created for DTO's with getters/setters
for (PropertyDescriptor inputProperty : projectionInformation.getInputProperties()) {
PersistentProperty<?> persistentProperty = persistentEntity.getPersistentProperty(inputProperty.getName());
if (persistentProperty == null) {
continue;
}
CycleGuard cycleGuardToUse = cycleGuard != null ? cycleGuard : new CycleGuard();
TypeInformation<?> property = projectionTypeInformation.getRequiredProperty(inputProperty.getName());
TypeInformation<?> actualType = property.getRequiredActualType();
boolean container = isContainer(actualType);
PropertyPath nestedPropertyPath = propertyPath == null
? PropertyPath.from(persistentProperty.getName(), persistentEntity.getTypeInformation())
: propertyPath.nested(persistentProperty.getName());
TypeInformation<?> unwrappedReturnedType = unwrapContainerType(actualType);
TypeInformation<?> unwrappedDomainType = unwrapContainerType(
persistentProperty.getTypeInformation().getRequiredActualType());
if (isProjection(unwrappedReturnedType, unwrappedDomainType)) {
List<EntityProjection.PropertyProjection<?, ?>> nestedPropertyDescriptors;
if (cycleGuardToUse.isCycleFree(persistentProperty)) {
nestedPropertyDescriptors = getProjectedProperties(container ? null : nestedPropertyPath,
unwrappedReturnedType, unwrappedDomainType, cycleGuardToUse);
} else {
nestedPropertyDescriptors = Collections.emptyList();
}
if (container) {
propertyDescriptors.add(EntityProjection.ContainerPropertyProjection.projecting(nestedPropertyPath, property,
persistentProperty.getTypeInformation(), nestedPropertyDescriptors, ProjectionType.from(projectionInformation)));
} else {
propertyDescriptors.add(EntityProjection.PropertyProjection.projecting(nestedPropertyPath, property,
persistentProperty.getTypeInformation(), nestedPropertyDescriptors, ProjectionType.from(projectionInformation)));
}
} else {
if (container) {
propertyDescriptors.add(EntityProjection.ContainerPropertyProjection.nonProjecting(nestedPropertyPath,
property, persistentProperty.getTypeInformation()));
} else {
propertyDescriptors.add(EntityProjection.PropertyProjection.nonProjecting(nestedPropertyPath, property,
persistentProperty.getTypeInformation()));
}
}
}
return propertyDescriptors;
}
private static TypeInformation<?> unwrapContainerType(TypeInformation<?> type) {
TypeInformation<?> unwrapped = type;
while (isContainer(unwrapped)) {
unwrapped = unwrapped.getRequiredActualType();
}
return unwrapped;
}
private static boolean isContainer(TypeInformation<?> actualType) {
return actualType.isCollectionLike() || actualType.isMap();
}
private boolean isProjection(TypeInformation<?> returnedType, TypeInformation<?> domainType) {
return projectionPredicate.test(returnedType.getRequiredActualType().getType(),
domainType.getRequiredActualType().getType());
}
private List<EntityProjection.PropertyProjection<?, ?>> getProjectedProperties(@Nullable PropertyPath propertyPath,
TypeInformation<?> returnedType, TypeInformation<?> domainType, CycleGuard cycleGuard) {
ProjectionInformation projectionInformation = projectionFactory.getProjectionInformation(returnedType.getType());
PersistentEntity<?, ?> persistentEntity = mappingContext.getRequiredPersistentEntity(domainType);
// Closed projection should get handled as above (recursion)
return projectionInformation.isClosed()
? getProperties(propertyPath, projectionInformation, returnedType, persistentEntity, cycleGuard)
: Collections.emptyList();
}
/**
* Represents a predicate (boolean-valued function) of a {@link Class target type} and its {@link Class underlying
* type}.
*/
public interface ProjectionPredicate {
/**
* Evaluates this predicate on the given arguments.
*
* @param target the target type.
* @param underlyingType the underlying type.
* @return {@code true} if the input argument matches the predicate, otherwise {@code false}.
*/
boolean test(Class<?> target, Class<?> underlyingType);
/**
* Return a composed predicate that represents a short-circuiting logical AND of this predicate and another. When
* evaluating the composed predicate, if this predicate is {@code false}, then the {@code other} predicate is not
* evaluated.
* <p>
* Any exceptions thrown during evaluation of either predicate are relayed to the caller; if evaluation of this
* predicate throws an exception, the {@code other} predicate will not be evaluated.
*
* @param other a predicate that will be logically-ANDed with this predicate
* @return a composed predicate that represents the short-circuiting logical AND of this predicate and the
* {@code other} predicate
*/
default ProjectionPredicate and(ProjectionPredicate other) {
return (target, underlyingType) -> test(target, underlyingType) && other.test(target, underlyingType);
}
/**
* Return a predicate that represents the logical negation of this predicate.
*
* @return a predicate that represents the logical negation of this predicate
*/
default ProjectionPredicate negate() {
return (target, underlyingType) -> !test(target, underlyingType);
}
/**
* Return a predicate that considers whether the {@code target type} is participating in the type hierarchy.
*/
static ProjectionPredicate typeHierarchy() {
ProjectionPredicate predicate = (target, underlyingType) -> target.isAssignableFrom(underlyingType) || // hierarchy
underlyingType.isAssignableFrom(target);
return predicate.negate();
}
}
static class CycleGuard {
Set<PersistentProperty<?>> seen = new LinkedHashSet<>();
public boolean isCycleFree(PersistentProperty<?> property) {
return seen.add(property);
}
}
}