Skip to content
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

Prevent path expander termination filter from filtering below minLevel #379

Merged

Conversation

InverseFalcon
Copy link
Collaborator

This fixes bug #364.

@InverseFalcon InverseFalcon force-pushed the SkipTerminationFilterBelowMinLevel branch from f7f2f3c to 5cf89d3 Compare April 8, 2017 08:18
…vel.

Ensured filterStartNode = false does not filter the start node below minLevel.
@InverseFalcon InverseFalcon force-pushed the SkipTerminationFilterBelowMinLevel branch from 5cf89d3 to 6f82cda Compare April 8, 2017 08:33
@InverseFalcon
Copy link
Collaborator Author

To clarify the first commit message, the terminology changes were only in the ExpandPathTest method names, no terminology changes in the PathExplorer code or the documentation.

@jexp jexp merged commit d3d2944 into neo4j-contrib:3.1 Apr 8, 2017
@InverseFalcon InverseFalcon deleted the SkipTerminationFilterBelowMinLevel branch April 10, 2017 02:09
jexp pushed a commit that referenced this pull request Apr 12, 2017
#379)

* Adjust path expander filter terminology
* Prevented path expander termination filter from filtering below minLevel.
Ensured filterStartNode = false does not filter the start node below minLevel.
albertodelazzari pushed a commit to albertodelazzari/neo4j-apoc-procedures that referenced this pull request Jun 28, 2017
neo4j-contrib#379)

* Adjust path expander filter terminology
* Prevented path expander termination filter from filtering below minLevel.
Ensured filterStartNode = false does not filter the start node below minLevel.
ncordon pushed a commit to ncordon/neo4j-apoc-procedures that referenced this pull request Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants