From 1d99e0010ee00caed82f9ae514b158e00529a823 Mon Sep 17 00:00:00 2001 From: Russell Johnson <47639332+Russ4262@users.noreply.github.com> Date: Wed, 23 Dec 2020 15:24:16 -0600 Subject: [PATCH] Path: Fix `Import` feature of Base Geometry 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. --- src/Mod/Path/PathScripts/PathOpGui.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Mod/Path/PathScripts/PathOpGui.py b/src/Mod/Path/PathScripts/PathOpGui.py index d0d5c22d83..ea1e069f2c 100644 --- a/src/Mod/Path/PathScripts/PathOpGui.py +++ b/src/Mod/Path/PathScripts/PathOpGui.py @@ -601,12 +601,13 @@ class TaskPanelBaseGeometryPage(TaskPanelPage): 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):