fix renameiddelta
This commit is contained in:
Alexander Wang
parent
e40a65ae79
commit
e6750a99b6
3 changed files with 217 additions and 9 deletions
|
|
@ -2735,7 +2735,7 @@ func RenameIDDeltas(g *d2graph.Graph, key, newName string) (deltas map[string]st
|
||||||
}
|
}
|
||||||
|
|
||||||
mk.Key.Path[len(mk.Key.Path)-1].Unbox().SetString(newName)
|
mk.Key.Path[len(mk.Key.Path)-1].Unbox().SetString(newName)
|
||||||
uniqueKeyStr, _, err := generateUniqueKey(g, strings.Join(d2graph.Key(mk.Key), "."), nil, nil)
|
uniqueKeyStr, _, err := generateUniqueKey(g, strings.Join(d2graph.Key(mk.Key), "."), obj, nil)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
@ -2749,15 +2749,18 @@ func RenameIDDeltas(g *d2graph.Graph, key, newName string) (deltas map[string]st
|
||||||
beforeObjID := obj.ID
|
beforeObjID := obj.ID
|
||||||
|
|
||||||
appendNodeDelta := func(ch *d2graph.Object) {
|
appendNodeDelta := func(ch *d2graph.Object) {
|
||||||
|
if obj.ID != newNameKey {
|
||||||
beforeID := ch.AbsID()
|
beforeID := ch.AbsID()
|
||||||
obj.ID = newNameKey
|
obj.ID = newNameKey
|
||||||
deltas[beforeID] = ch.AbsID()
|
deltas[beforeID] = ch.AbsID()
|
||||||
obj.ID = beforeObjID
|
obj.ID = beforeObjID
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
appendEdgeDelta := func(ch *d2graph.Object) {
|
appendEdgeDelta := func(ch *d2graph.Object) {
|
||||||
for _, e := range obj.Graph.Edges {
|
for _, e := range obj.Graph.Edges {
|
||||||
if e.Src == ch || e.Dst == ch {
|
if e.Src == ch || e.Dst == ch {
|
||||||
|
if obj.ID != newNameKey {
|
||||||
beforeID := e.AbsID()
|
beforeID := e.AbsID()
|
||||||
obj.ID = newNameKey
|
obj.ID = newNameKey
|
||||||
deltas[beforeID] = e.AbsID()
|
deltas[beforeID] = e.AbsID()
|
||||||
|
|
@ -2765,6 +2768,7 @@ func RenameIDDeltas(g *d2graph.Graph, key, newName string) (deltas map[string]st
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
var recurse func(ch *d2graph.Object)
|
var recurse func(ch *d2graph.Object)
|
||||||
recurse = func(ch *d2graph.Object) {
|
recurse = func(ch *d2graph.Object) {
|
||||||
|
|
|
||||||
|
|
@ -1840,6 +1840,19 @@ func TestRename(t *testing.T) {
|
||||||
newName: `Square`,
|
newName: `Square`,
|
||||||
|
|
||||||
exp: `Square
|
exp: `Square
|
||||||
|
`,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "generated-conflict",
|
||||||
|
|
||||||
|
text: `Square
|
||||||
|
Square 2
|
||||||
|
`,
|
||||||
|
key: `Square 2`,
|
||||||
|
newName: `Square`,
|
||||||
|
|
||||||
|
exp: `Square
|
||||||
|
Square 2
|
||||||
`,
|
`,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
|
@ -7261,6 +7274,17 @@ y
|
||||||
"x": "y 2"
|
"x": "y 2"
|
||||||
}`,
|
}`,
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
name: "generated-conflict",
|
||||||
|
|
||||||
|
text: `Square
|
||||||
|
Square 2
|
||||||
|
`,
|
||||||
|
key: `Square 2`,
|
||||||
|
newName: `Square`,
|
||||||
|
|
||||||
|
exp: `{}`,
|
||||||
|
},
|
||||||
{
|
{
|
||||||
name: "rename_conflict_with_dots",
|
name: "rename_conflict_with_dots",
|
||||||
|
|
||||||
|
|
|
||||||
180
testdata/d2oracle/TestRename/generated-conflict.exp.json
generated
vendored
Normal file
180
testdata/d2oracle/TestRename/generated-conflict.exp.json
generated
vendored
Normal file
|
|
@ -0,0 +1,180 @@
|
||||||
|
{
|
||||||
|
"graph": {
|
||||||
|
"name": "",
|
||||||
|
"isFolderOnly": false,
|
||||||
|
"ast": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-2:0:16",
|
||||||
|
"nodes": [
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-0:6:6",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-0:6:6",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-0:6:6",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"map_key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,1:0:7-1:8:15",
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,1:0:7-1:8:15",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,1:0:7-1:8:15",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"primary": {},
|
||||||
|
"value": {}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"root": {
|
||||||
|
"id": "",
|
||||||
|
"id_val": "",
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
"edges": null,
|
||||||
|
"objects": [
|
||||||
|
{
|
||||||
|
"id": "Square",
|
||||||
|
"id_val": "Square",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-0:6:6",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,0:0:0-0:6:6",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square",
|
||||||
|
"raw_string": "Square"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"id": "Square 2",
|
||||||
|
"id_val": "Square 2",
|
||||||
|
"references": [
|
||||||
|
{
|
||||||
|
"key": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,1:0:7-1:8:15",
|
||||||
|
"path": [
|
||||||
|
{
|
||||||
|
"unquoted_string": {
|
||||||
|
"range": "d2/testdata/d2oracle/TestRename/generated-conflict.d2,1:0:7-1:8:15",
|
||||||
|
"value": [
|
||||||
|
{
|
||||||
|
"string": "Square 2",
|
||||||
|
"raw_string": "Square 2"
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"key_path_index": 0,
|
||||||
|
"map_key_edge_index": -1
|
||||||
|
}
|
||||||
|
],
|
||||||
|
"attributes": {
|
||||||
|
"label": {
|
||||||
|
"value": "Square 2"
|
||||||
|
},
|
||||||
|
"labelDimensions": {
|
||||||
|
"width": 0,
|
||||||
|
"height": 0
|
||||||
|
},
|
||||||
|
"style": {},
|
||||||
|
"near_key": null,
|
||||||
|
"shape": {
|
||||||
|
"value": "rectangle"
|
||||||
|
},
|
||||||
|
"direction": {
|
||||||
|
"value": ""
|
||||||
|
},
|
||||||
|
"constraint": {
|
||||||
|
"value": ""
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"zIndex": 0
|
||||||
|
}
|
||||||
|
]
|
||||||
|
},
|
||||||
|
"err": "<nil>"
|
||||||
|
}
|
||||||
Loading…
Reference in a new issue