From 5daa3aa3f0daf5b6e48b5718117601fb0a9cf4ac Mon Sep 17 00:00:00 2001
From: Kaalleen <36401965+kaalleen@users.noreply.github.com>
Date: Mon, 25 Dec 2023 08:49:46 +0100
Subject: Remove previous troubleshoot layer (#2638)

... instead of deleting content
---
 lib/extensions/troubleshoot.py | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

(limited to 'lib/extensions/troubleshoot.py')

diff --git a/lib/extensions/troubleshoot.py b/lib/extensions/troubleshoot.py
index fdc7fa9e..01629069 100644
--- a/lib/extensions/troubleshoot.py
+++ b/lib/extensions/troubleshoot.py
@@ -100,16 +100,18 @@ class Troubleshoot(InkstitchExtension):
         svg = self.document.getroot()
         layer = svg.find(".//*[@id='__validation_layer__']")
 
-        if layer is None:
-            layer = inkex.Group(attrib={
-                'id': '__validation_layer__',
-                INKSCAPE_LABEL: _('Troubleshoot'),
-                INKSCAPE_GROUPMODE: 'layer',
-            })
-            svg.append(layer)
-        else:
-            # Clear out everything from the last run
-            del layer[:]
+        if layer is not None:
+            # Remove the old layer - they may have used tranfsorms
+            # or moved it into an other group (which could lead to more transforms)
+            # We don't want to deal with it.
+            layer.getparent().remove(layer)
+
+        layer = inkex.Group(attrib={
+            'id': '__validation_layer__',
+            INKSCAPE_LABEL: _('Troubleshoot'),
+            INKSCAPE_GROUPMODE: 'layer',
+        })
+        svg.append(layer)
 
         add_layer_commands(layer, ["ignore_layer"])
 
-- 
cgit v1.2.3