From 7d4b5f34cce9e95ec193f144381c931c329b7083 Mon Sep 17 00:00:00 2001 From: Russell Johnson <47639332+Russ4262@users.noreply.github.com> Date: Mon, 12 Jul 2021 23:03:29 -0500 Subject: [PATCH] Path: Remove `Side` assignment to `Outside` for full model profiles The profile operation is forcing cut side to `Outside` when profiling whole models. This change in code also allows for profiling `Inside` of entire models. Apart from limiting profiling capabilities of entire models, this override could cause problems with profiling of selected faces or edges in very specific use cases. Also, this fix removes the property assignment of a key control property within the execution code, which should not happen, apart from feedback properties. --- src/Mod/Path/PathScripts/PathProfile.py | 1 - 1 file changed, 1 deletion(-) diff --git a/src/Mod/Path/PathScripts/PathProfile.py b/src/Mod/Path/PathScripts/PathProfile.py index f1ee7ce7b5..39f65f9956 100644 --- a/src/Mod/Path/PathScripts/PathProfile.py +++ b/src/Mod/Path/PathScripts/PathProfile.py @@ -380,7 +380,6 @@ class ObjectProfile(PathAreaOp.ObjectOp): else: # Try to build targets from the job models # No base geometry selected, so treating operation like a exterior contour operation self.opUpdateDepths(obj) - obj.Side = 'Outside' # Force outside for whole model profile if 1 == len(self.model) and hasattr(self.model[0], "Proxy"): if isinstance(self.model[0].Proxy, ArchPanel.PanelSheet): # process the sheet