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.

Sign up for GitHub

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

Jump to bottom

apoc.diff.nodes() returns list-type properties as different when they are the same #2992

Closed
michelekorosec opened this issue Jun 15, 2022 · 0 comments · Fixed by #3083 or neo4j/apoc#206
Closed

apoc.diff.nodes() returns list-type properties as different when they are the same #2992

michelekorosec opened this issue Jun 15, 2022 · 0 comments · Fixed by #3083 or neo4j/apoc#206
Assignees
@vga91

Comments

@michelekorosec
Copy link

Expected Behavior

Return list-type properties as inCommon when all elements in the list are the same and as different when the elements are different

Actual Behavior

Returns all list-type properties as different, no matter if the list elements are the same or not

How to Reproduce the Problem

Simple Dataset

MERGE (charlie:Person {name:"Charlie",grocery_list:["tomatoes","bread","cookies"]})
MERGE (hannah:Person {name:"Hannah",grocery_list:["tomatoes","bread","cookies"]})
RETURN apoc.diff.nodes(charlie,hannah)

Steps

  1. Create nodes charlie and hannah (see above) in a blank database
  2. Run RETURN apoc.diff.nodes(charlie,hannah)

Screenshots

Screenshot (2)

Specifications

Versions

  • OS: Windows
  • Neo4j: 4.4.6
  • Neo4j-Apoc: 4.4.0.3
@vga91 vga91 self-assigned this Jun 16, 2022
vga91 added a commit to vga91/neo4j-apoc-procedures that referenced this issue Aug 1, 2022
@vga91
Fixes neo4j-contrib#2992: apoc.diff.nodes() returns list-type propert…
51fe8a8 
…ies as different when they are the same
vga91 added a commit to vga91/neo4j-apoc-procedures that referenced this issue Aug 1, 2022
@vga91
Fixes neo4j-contrib#2992: apoc.diff.nodes() returns list-type propert…
c44e1f7 
…ies as different when they are the same
vga91 added a commit to vga91/neo4j-apoc-procedures that referenced this issue Aug 1, 2022
@vga91
Fixes neo4j-contrib#2992: apoc.diff.nodes() returns list-type propert…
ea2a2c7 
…ies as different when they are the same
@vga91 vga91 mentioned this issue Aug 1, 2022
Merged
vga91 added a commit that referenced this issue Aug 2, 2022
@vga91
Fixes #2992: apoc.diff.nodes() returns list-type properties as differ…
b069e2b 
…ent when they are the same (#3083)

* apoc.diff.nodes() returns list-type properties as different when they are the same
* code clean
neo4j-oss-build pushed a commit that referenced this issue Aug 2, 2022
@vga91
Fixes #2992: apoc.diff.nodes() returns list-type properties as differ…
a28bd5c 
…ent when they are the same (#3083)

* apoc.diff.nodes() returns list-type properties as different when they are the same
* code clean
neo4j-oss-build pushed a commit that referenced this issue Aug 2, 2022
@vga91
Fixes #2992: apoc.diff.nodes() returns list-type properties as differ…
4835954 
…ent when they are the same (#3083)

* apoc.diff.nodes() returns list-type properties as different when they are the same
* code clean
vga91 pushed a commit that referenced this issue Aug 26, 2022
@neo4j-oss-build
Fixes #2992: apoc.diff.nodes() returns list-type properties as differ…
dfe3d1d 
…ent when they are the same (#3083) (#3089)
vga91 added a commit to neo4j/apoc that referenced this issue Oct 6, 2022
@vga91
Fixes neo4j-contrib/neo4j-apoc-procedures#2992: apoc.diff.nodes() ret…
07e7db9 
…urns list-type properties as different when they are the same
@vga91 vga91 mentioned this issue Oct 6, 2022
Merged
vga91 added a commit to neo4j/apoc that referenced this issue Jan 17, 2023
@vga91
[NOID] Fixes neo4j-contrib/neo4j-apoc-procedures#2992: apoc.diff.node…
e52488b 
…s() returns list-type properties as different when they are the same
vga91 added a commit to neo4j/apoc that referenced this issue Jan 30, 2023
@vga91
[NOID] Fixes neo4j-contrib/neo4j-apoc-procedures#2992: apoc.diff.node…
e6ef455 
…s() returns list-type properties as different when they are the same (#206)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vga91 vga91

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@vga91 @michelekorosec