From 54494409d2daa3d2ef14efd2432684b4dab0adec Mon Sep 17 00:00:00 2001
From: Goffi <goffi@goffi.org>
Date: Thu, 12 Nov 2020 14:53:15 +0100
Subject: [PATCH] tools (xml_tools): added `with_parent` and `with_children`
 argument to `elementCopy`

---
 sat/tools/xml_tools.py | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/sat/tools/xml_tools.py b/sat/tools/xml_tools.py
index ae36aa60..69f9c6a5 100644
--- a/sat/tools/xml_tools.py
+++ b/sat/tools/xml_tools.py
@@ -1676,22 +1676,28 @@ def deferConfirm(*args, **kwargs):
 
 # Misc other funtions
 
-def elementCopy(element):
+def elementCopy(
+    element: domish.Element,
+    with_parent: bool = True,
+    with_children: bool = True
+) -> domish.Element:
     """Make a copy of a domish.Element
 
     The copy will have its own children list, so other elements
     can be added as direct children without modifying orignal one.
     Children are not deeply copied, so if an element is added to a child or grandchild,
     it will also affect original element.
-    @param element(domish.Element): Element to clone
+    @param element: Element to clone
     """
     new_elt = domish.Element(
         (element.uri, element.name),
         defaultUri = element.defaultUri,
         attribs = element.attributes,
         localPrefixes = element.localPrefixes)
-    new_elt.parent = element.parent
-    new_elt.children = element.children[:]
+    if with_parent:
+        new_elt.parent = element.parent
+    if with_children:
+        new_elt.children = element.children[:]
     return new_elt