-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow chaining statements to yielding calls. #964
Conversation
Please test and add the new interface for those kinda calls / chains you are missing. |
|
See spring-projects/spring-data-neo4j#2895 To get a build, use
|
@@ -79,7 +71,7 @@ public static FunctionInvocation id(@NotNull Node node) { | |||
@NotNull @Contract(pure = true) | |||
@Deprecated(since = "2023.3.0") | |||
@SuppressWarnings({ "squid:S1133" }) // Yes, I promise, this will be removed at some point, but not yet. | |||
public static FunctionInvocation id(@NotNull Relationship relationship) { | |||
static FunctionInvocation id(@NotNull Relationship relationship) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could get already removed because there is no access to this method anymore, in contrast to the id(Node node)
version that is used by the AbstractNode
.
neo4j-cypher-dsl/src/main/java/org/neo4j/cypherdsl/core/DefaultStatementBuilder.java
Outdated
Show resolved
Hide resolved
neo4j-cypher-dsl/src/main/java/org/neo4j/cypherdsl/core/OngoingListBasedPredicateFunction.java
Outdated
Show resolved
Hide resolved
Make the yielding call return statement, but that would require a major change. So use this opportunity to remove all teh deprecations. Build will fail now as we did not make SDN aware of this in time (bummer)
…gListBasedPredicateFunction.java Co-authored-by: Gerrit Meier <meistermeier@gmail.com>
…tStatementBuilder.java Co-authored-by: Gerrit Meier <meistermeier@gmail.com>
290eb0d
to
5e05e69
Compare
|
No description provided.