🌲 Override IDs should not be processed twice

converter/context.py

@@ -80,6 +80,8 @@ def used_fonts(self) -> Dict[Tuple[str, str], Tuple[IO[bytes], str]]:
 
     def find_symbol(self, sid: Sequence[int]) -> dict:
         symbol = self.fig_node(sid)
+        sid = symbol["guid"]
+
         if not self._component_symbols.get(sid, True):
             # The symbol is in the component page and has not been converted yet, do it now
             from . import tree

converter/instance.py

@@ -26,9 +26,12 @@ def convert(fig_instance):
         else:
             utils.log_conversion_warning("SYM002", fig_instance, props=unsupported)
 
+    # Use always the GUID of the master for the symbolID
+    # The instance symbolID can refer to the overrideKey instead
+    fig_master = context.find_symbol(fig_instance["symbolData"]["symbolID"])
     obj = SymbolInstance(
         **base.base_styled(fig_instance),
-        symbolID=utils.gen_object_id(fig_instance["symbolData"]["symbolID"]),
+        symbolID=utils.gen_object_id(fig_master["guid"]),
         overrideValues=sketch_overrides,
     )
     # Replace style

converter/shape_path.py

@@ -164,7 +164,7 @@ def use_segment(i: int) -> int:
     if unused_segments:
         regions += [
             {
-                "style": {"fillPaints": []},
+                "style": {},
                 "windingRule": "NONZERO",
                 "loops": reorder_segments(
                     [vector_network["segments"][i] for i in unused_segments]

figformat/fig2tree.py

@@ -20,6 +20,7 @@ def convert_fig(path: str, output: ZipFile) -> Tuple[dict, Dict[Sequence[int], d
 
     # Load all nodes into a map
     id_map = {}
+    override_map = {}
     root = None
 
     for node in fig["nodeChanges"]:
@@ -28,7 +29,7 @@ def convert_fig(path: str, output: ZipFile) -> Tuple[dict, Dict[Sequence[int], d
         id_map[node_id] = node
 
         if "overrideKey" in node:
-            id_map[node["overrideKey"]] = node
+            override_map[node["overrideKey"]] = node
 
         if not root:
             root = node_id
@@ -45,6 +46,8 @@ def convert_fig(path: str, output: ZipFile) -> Tuple[dict, Dict[Sequence[int], d
     for node in id_map.values():
         node["children"].sort(key=lambda n: n["parent"]["position"])
 
+    id_map.update(override_map)
+
     return tree, id_map