Graph <--> CropObject inlinks/outlinks sync on Plus/Eraser Lasso

hajicj / MUSCIMarker

Tool for annotating objects in musical scores.

http://muscimarker.readthedocs.io/en/latest/

Apache License 2.0

7 stars 2 forks source link

Graph <--> CropObject inlinks/outlinks sync on Plus/Eraser Lasso #151

Closed hajicj closed 7 years ago

hajicj commented 7 years ago

To replicate:

delete some edges of an object
then use Plus Lasso to add something to the object

hajicj commented 7 years ago

When removing and adding edges, the ObjectGraph gets modified, but the CropObjects in question do not get their inlinks and outlinks updated accordingly. This then shows up when the Plus/Eraser Lasso re-add the object: the new object is added with all its old inlinks/outlinks.

hajicj commented 7 years ago

To fix: reimplement ensure_add_edge, ensure_remove_edge, etc. in model, so that it syncs the graph automatically. Basically, this will hide graph operations behind the model.