nodeview: null check on item
Hopefully this doesn't break behavior Fixes #1687 Fixes #1736
This commit is contained in:
@@ -1369,12 +1369,16 @@ void NodeView::RecursivelyRemoveFloatingNodeFromContext(MultiUndoCommand *comman
|
||||
|
||||
void NodeView::RecursivelyAddNodeToContext(MultiUndoCommand *command, Node *node, Node *context)
|
||||
{
|
||||
command->add_child(new NodeSetPositionCommand(node, context, GetEstimatedPositionForContext(scene_.item_map().value(node), context), false));
|
||||
NodeViewItem *item = scene_.item_map().value(node);
|
||||
|
||||
// Add dependency
|
||||
for (auto it=node->input_connections().cbegin(); it!=node->input_connections().cend(); it++) {
|
||||
Node *dependency = it->second.node();
|
||||
RecursivelyAddNodeToContext(command, dependency, context);
|
||||
if (item) {
|
||||
command->add_child(new NodeSetPositionCommand(node, context, GetEstimatedPositionForContext(item, context), false));
|
||||
|
||||
// Add dependency
|
||||
for (auto it=node->input_connections().cbegin(); it!=node->input_connections().cend(); it++) {
|
||||
Node *dependency = it->second.node();
|
||||
RecursivelyAddNodeToContext(command, dependency, context);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user