kindred/create
1
1
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

[Gui] Fix "Select dependent objects" with cycles

Browse Source 
Doing "Add dependent objects to selection" in the context menu of an
object that has cyclic dependencies triggered an infinite recursive
loop.  This has been solved by using the function to get an outlist
recursively.
This commit is contained in:
Pieter Hijma
2025-04-22 15:43:36 +02:00
committed by Adrián Insaurralde Avalos
parent 54549a4a96
commit 01effaaab0
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/Gui/Tree.cpp
View File

@@ -1303,11 +1303,11 @@ void TreeWidget::addDependentToSelection(App::Document* doc, App::DocumentObject
{
// add the docObject to the selection
Selection().addSelection(doc->getName(), docObject->getNameInDocument());
// get the dependent
auto subObjectList = docObject->getOutList();
// the dependent can in turn have dependents, thus add them recursively
for (auto itDepend = subObjectList.begin(); itDepend != subObjectList.end(); ++itDepend)
addDependentToSelection(doc, (*itDepend));
// get the dependent objects recursively
auto subObjectList = docObject->getOutListRecursive();
for (auto itDepend = subObjectList.begin(); itDepend != subObjectList.end(); ++itDepend) {
Selection().addSelection(doc->getName(), (*itDepend)->getNameInDocument());
}
}
// add dependents of the selected tree object to selection