From db2d518aa2e9d40a102e1fc7c017756f9dc0bbea 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 173aaaa401..b4b1e724ff 100644 --- a/src/Mod/Path/PathScripts/PathProfile.py +++ b/src/Mod/Path/PathScripts/PathProfile.py @@ -448,7 +448,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