Updated the way we write in the data version

Author: GearsAD

src/CloudGraphsDFG/services/CGStructure.jl

@@ -10,6 +10,7 @@ function _convertNodeToDict(abstractNode::N)::Dict{String, Any} where N <: Abstr
     data = length(cp.data) != 0 ? JSON2.write(cp.data) : "{}"
     ser = JSON2.read(JSON2.write(abstractNode), Dict{String, Any})
     ser["data"] = base64encode(data)
+    ser["_version"] = _getDFGVersion()
     return ser
 end
 

src/CloudGraphsDFG/services/CloudGraphsDFG.jl

@@ -178,8 +178,7 @@ function updateVariable!(dfg::CloudGraphsDFG, variable::DFGVariable; skipAddErro
     # NOTE: We are not merging the variable.tags into the labels anymore. We can index by that but not
     # going to pollute the graph with unnecessary (and potentially dangerous) labels.
     addProps = Dict(
-        "softtype" => "\"$(string(typeof(getSofttype(variable))))\"",
-        "_version" => "\"$(_getDFGVersion())\"")
+        "softtype" => "\"$(string(typeof(getSofttype(variable))))\"")
     query = """
     MATCH (session:$(join(_getLabelsForType(dfg, Session), ":")))
     MERGE (node:$(join(_getLabelsForInst(dfg, variable), ":")))
@@ -299,8 +298,7 @@ function updateFactor!(dfg::CloudGraphsDFG, factor::DFGFactor; skipAddError::Boo
     # going to pollute the graph with unnecessary (and potentially dangerous) labels.
     fnctype = getSolverData(factor).fnc.usrfnc!
     addProps = Dict(
-        "fnctype" => "\"$(String(_getname(fnctype)))\"",
-        "_version" => "\"$(_getDFGVersion())\"")
+        "fnctype" => "\"$(String(_getname(fnctype)))\"")
     query = """
     MATCH (session:$(join(_getLabelsForType(dfg, Session), ":")))
     MERGE (node:$(join(_getLabelsForInst(dfg, factor), ":")))

src/CloudGraphsDFG/services/CommonFunctions.jl

@@ -198,7 +198,8 @@ function _structToNeo4jProps(inst::Union{User, Robot, Session, PVND, N, APPE, AB
     for (k, v) in addProps
         write(io, "$cypherNodeName.$k= $v,")
     end
-    # The node type
+    # Write in the version and node type
+    write(io, "$cypherNodeName._version=\"$(_getDFGVersion())\",")
     write(io, "$cypherNodeName._type=\"$(typeof(inst))\"")
     # Ref String(): "When possible, the memory of v will be used without copying when the String object is created.
     # This is guaranteed to be the case for byte vectors returned by take!" # Apparent replacement for takebuf_string()