albertoventurini / graphdb-intellij-plugin

Graph database plugin for the IntelliJ Platform
https://plugins.jetbrains.com/plugin/20417-graph-database
Apache License 2.0
53 stars 15 forks source link

Migrate to using elementId #84

Closed robsdedude closed 1 year ago

robsdedude commented 1 year ago

The cypher function id has been deprecated since Neo4j 5.0. This PR adds the capability of sniffing the DBMS version so that the DiffService can write an update query that matches the server version.

Likewise has entity.id() in the driver been deprecated. The driver will set elementId to id.toString() if the server does not provide it (4.4 and before). Therefore, it's safe to always use entity.elementId() regardless of the server version.

albertoventurini commented 1 year ago

Thank you for this @robsdedude, great work!