feat: remove attributes elements before featureType insertion to avoid custom attributes checkbox set

Author: f-necas

backend/maelstro/core/copy_manager.py

@@ -13,7 +13,7 @@
 from maelstro.common.exceptions import ParamError
 from .georchestra import GeorchestraHandler
 from .operations import raise_for_status
-
+import xml.etree.ElementTree as ET
 
 logger = logging.getLogger()
 
@@ -356,25 +356,28 @@ def copy_layer(
         )
         resource = gs_src.rest_client.get(xml_resource_route)
 
+        # Clean <attributes> element to avoid "Custom attributes" checkbox being set
+        # See issue #94
+        cleaned_content = self.remove_attributes_element(resource.content)
         if has_resource.status_code == 200:
             if has_layer.status_code != 200:
                 resp = self.gs_dst.rest_client.delete(resource_route)
                 raise_for_status(resp)
                 resp = self.gs_dst.rest_client.post(
                     resource_post_route,
-                    data=resource.content,
+                    data=cleaned_content,
                     headers={"content-type": "application/xml"},
                 )
             else:
                 resp = self.gs_dst.rest_client.put(
                     xml_resource_route,
-                    data=resource.content,
+                    data=cleaned_content,
                     headers={"content-type": "application/xml"},
                 )
         else:
             resp = self.gs_dst.rest_client.post(
                 resource_post_route,
-                data=resource.content,
+                data=cleaned_content,
                 headers={"content-type": "application/xml"},
             )
             if resp.status_code == 404:
@@ -425,3 +428,13 @@ def copy_style(
                 headers={"content-type": style_def.headers["content-type"]},
             )
             raise_for_status(dst_style_def)
+
+    def remove_attributes_element(self, xml_bytes: bytes) -> Any:
+        root = ET.fromstring(xml_bytes)
+        if root is not None:
+            attributes = root.find("attributes")
+            if attributes is not None:
+                root.remove(attributes)
+            return ET.tostring(root, encoding="utf-8")
+        # If root is None, return the original bytes or handle as needed
+        return ET.tostring(root, encoding="utf-8")