GH-2828 - Use correct node name related node in find by example.

Closes #2828

Author: meistermeier

src/main/java/org/springframework/data/neo4j/core/mapping/IdDescription.java

@@ -52,6 +52,7 @@ public final class IdDescription {
 	 * The property that stores the id if applicable.
 	 */
 	private @Nullable final String graphPropertyName;
+	private final boolean isDeprecated;
 
 	private final Lazy<Expression> idExpression;
 
@@ -93,6 +94,7 @@ private IdDescription(SymbolicName symbolicName, @Nullable Class<? extends IdGen
 		this.idGeneratorClass = idGeneratorClass;
 		this.idGeneratorRef = idGeneratorRef != null && idGeneratorRef.isEmpty() ? null : idGeneratorRef;
 		this.graphPropertyName = graphPropertyName;
+		this.isDeprecated = isDeprecated;
 
 		this.idExpression = Lazy.of(() -> {
 			final Node rootNode = Cypher.anyNode(symbolicName);
@@ -109,6 +111,23 @@ public Expression asIdExpression() {
 		return this.idExpression.get();
 	}
 
+	/**
+	 * Creates the right identifier expression for this node entity.
+	 * Note: This enforces a recalculation of the name on invoke.
+	 *
+	 * @param nodeName use this name as the symbolic name of the node in the query
+	 * @return An expression that represents the right identifier type.
+	 */
+	public Expression asIdExpression(String nodeName) {
+		final Node rootNode = Cypher.anyNode(nodeName);
+		if (this.isInternallyGeneratedId()) {
+			return isDeprecated ? Functions.id(rootNode) : Functions.elementId(rootNode);
+		} else {
+			return this.getOptionalGraphPropertyName()
+					.map(propertyName -> Cypher.property(nodeName, propertyName)).get();
+		}
+	}
+
 	public Optional<Class<? extends IdGenerator<?>>> getIdGeneratorClass() {
 		return Optional.ofNullable(idGeneratorClass);
 	}

src/main/java/org/springframework/data/neo4j/core/mapping/NodeDescription.java

@@ -136,6 +136,9 @@ default boolean isUsingInternalIds() {
 	NodeDescription<?> getParentNodeDescription();
 
 	/**
+	 * Creates the right identifier expression for this node entity.
+	 * Note: The expression gets cached and won't get recalculated at every invocation.
+	 *
 	 * @return An expression that represents the right identifier type.
 	 */
 	default Expression getIdExpression() {

src/main/java/org/springframework/data/neo4j/repository/query/Predicate.java

@@ -167,7 +167,7 @@ private static void addConditionAndParameters(Neo4jMappingContext mappingContext
 			if (isRootNode) {
 				condition = predicate.neo4jPersistentEntity.getIdExpression().isEqualTo(literalOf(theValue));
 			} else {
-				condition = nodeDescription.getIdExpression().isEqualTo(literalOf(theValue));
+				condition = nodeDescription.getIdDescription().asIdExpression(wrapper.getNodeName()).isEqualTo(literalOf(theValue));
 			}
 		} else {
 			Expression property =  !isRootNode ? property(wrapper.getNodeName(), propertyName) : property(Constants.NAME_OF_TYPED_ROOT_NODE.apply(nodeDescription), propertyName);

src/test/java/org/springframework/data/neo4j/integration/imperative/RepositoryIT.java

@@ -3118,7 +3118,10 @@ void findEntityWithRelationshipByFindOneByExample(@Autowired RelationshipReposit
 			long petNode2Id = TestIdentitySupport.getInternalId(petNode2);
 
 			PersonWithRelationship probe = new PersonWithRelationship();
-			probe.setName("Freddie");
+			Hobby hobbies = new Hobby();
+			hobbies.setId(hobbyNodeId);
+			hobbies.setName("Music");
+			probe.setHobbies(hobbies);
 			PersonWithRelationship loadedPerson = repository.findOne(Example.of(probe)).get();
 			assertThat(loadedPerson.getName()).isEqualTo("Freddie");
 			assertThat(loadedPerson.getId()).isEqualTo(personId);