fix: removing a ChoiceNode option now correctly disconnects the attached edge, closes #1122

AlmasB · AlmasB · commit 47748e0ec4c6 · 2022-03-07T20:10:25.000Z
diff --git a/fxgl-tools/src/main/kotlin/com/almasb/fxgl/tools/dialogues/DialoguePane.kt b/fxgl-tools/src/main/kotlin/com/almasb/fxgl/tools/dialogues/DialoguePane.kt
@@ -529,6 +529,10 @@ class DialoguePane(graph: DialogueGraph = DialogueGraph()) : Pane() {
                 c.addedSubList.forEach { outPoint ->
                     attachMouseHandler(outPoint)
                 }
+
+                c.removed.forEach { outPoint ->
+                    disconnectOutLink(outPoint)
+                }
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -529,6 +529,10 @@ class DialoguePane(graph: DialogueGraph = DialogueGraph()) : Pane() {`
`529`	`529`	`c.addedSubList.forEach { outPoint ->`
`530`	`530`	`attachMouseHandler(outPoint)`
`531`	`531`	`}`
	`532`	`+`
	`533`	`+ c.removed.forEach { outPoint ->`
	`534`	`+ disconnectOutLink(outPoint)`
	`535`	`+ }`
`532`	`536`	`}`
`533`	`537`	`}`
`534`	`538`