Undo will lose all group view !

[bbsuuo]

when undo , it's will reload graph view, and it's will remove all elements

remove element will call GraphViewChangedCallback function

and when remove element, group view will check this node on group , and remove from base graph data

it's will lose all group view ，this kind of incorrect call can lead to many potential problems later

[bbsuuo]

I just changed the order of the clear group view ()

the order of calls should clear the group first

and i remove graphViewChanged before reload and add later

some callback chain messed up , i try to code refactoring , and the code and ideas are valuable

[DeJhon-Huang]

I also want to know how to solve it in the end?

[shda]

When call undo, innerNodeGUIDs are removed from the group, I solved this problem like this

in the BaseGraphView.ReloadView method:

void ReloadView()
{
	...
	var innerNodeGUIDs = graph.groups.Select(graphGroup => graphGroup.innerNodeGUIDs.ToList()).ToList();

	// Remove everything
	RemoveNodeViews();

	for (var i = 0; i < innerNodeGUIDs.Count; i++)
	{
		var innerNodeGuid = innerNodeGUIDs[i];
		graph.groups[i].innerNodeGUIDs = innerNodeGuid;
	}
	...
}