Skip to content

Commit

Permalink
Path: Fix Import feature of Base Geometry
Browse files Browse the repository at this point in the history
Error was thrown if FreeCADGui.Selection list had more than one item.  Solution is to clear selection before adding  selection from import operation.  I also removed the `for` loop since the selection should be singular.
  • Loading branch information
Russ4262 committed Dec 23, 2020
1 parent d89cd95 commit 1d99e00
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/Mod/Path/PathScripts/PathOpGui.py
Expand Up @@ -601,12 +601,13 @@ def clearBase(self):
def importBaseGeometry(self):
opLabel = str(self.form.geometryImportList.currentText())
ops = FreeCAD.ActiveDocument.getObjectsByLabel(opLabel)
if ops.__len__() > 1:
if len(ops) > 1:
msg = translate('PathOpGui', 'Mulitiple operations are labeled as')
msg += " {}\n".format(opLabel)
FreeCAD.Console.PrintWarning(msg)
for (base, subList) in ops[0].Base:
FreeCADGui.Selection.addSelection(base, subList)
(base, subList) = ops[0].Base[0]
FreeCADGui.Selection.clearSelection()
FreeCADGui.Selection.addSelection(base, subList)
self.addBase()

def registerSignalHandlers(self, obj):
Expand Down

0 comments on commit 1d99e00

Please sign in to comment.