Make propertys a dict

atopile · Sep 13, 2024 · e666bf2 · e666bf2
1 parent 209b2d7
commit e666bf2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 9 deletions.
diff --git a/src/faebryk/libs/kicad/fileformats_sch.py b/src/faebryk/libs/kicad/fileformats_sch.py
@@ -175,8 +175,9 @@ class C_power:
 
                 name: str = field(**sexp_field(positional=True))
                 power: Optional[C_power] = None
-                propertys: list[C_property] = field(
-                    **sexp_field(multidict=True), default_factory=list
+                propertys: dict[str, C_property] = field(
+                    **sexp_field(multidict=True, key=lambda x: x.name),
+                    default_factory=dict,
                 )
                 pin_numbers: Optional[E_hide] = None
                 pin_names: Optional[C_pin_names] = None
@@ -205,8 +206,9 @@ class C_pin:
             in_bom: bool
             on_board: bool
             fields_autoplaced: bool = True
-            propertys: list[C_property] = field(
-                **sexp_field(multidict=True), default_factory=list
+            propertys: dict[str, C_property] = field(
+                **sexp_field(multidict=True, key=lambda x: x.name),
+                default_factory=dict,
             )
             pins: list[C_pin] = field(
                 **sexp_field(multidict=True), default_factory=list
@@ -257,8 +259,9 @@ class E_type(SymEnum):
             fill: C_fill
             uuid: UUID
             fields_autoplaced: bool = True
-            propertys: list[C_property] = field(
-                **sexp_field(multidict=True), default_factory=list
+            propertys: dict[str, C_property] = field(
+                **sexp_field(multidict=True, key=lambda x: x.name),
+                default_factory=dict,
             )
             pins: list[C_pin] = field(
                 **sexp_field(multidict=True), default_factory=list
@@ -284,8 +287,9 @@ class E_shape(SymEnum):
             uuid: UUID
             text: str = field(**sexp_field(positional=True))
             fields_autoplaced: bool = True
-            propertys: list[C_property] = field(
-                **sexp_field(multidict=True), default_factory=list
+            propertys: dict[str, C_property] = field(
+                **sexp_field(multidict=True, key=lambda x: x.name),
+                default_factory=dict,
             )
 
         # TODO: inheritance

diff --git a/test/libs/kicad/test_fileformats.py b/test/libs/kicad/test_fileformats.py
@@ -117,7 +117,7 @@ def test_parser(self):
 
         self.assertEqual(
             sch.kicad_sch.lib_symbols.symbol["Amplifier_Audio:LM4990ITL"]
-            .propertys[3]
+            .propertys["Datasheet"]
             .value,
             "http://www.ti.com/lit/ds/symlink/lm4990.pdf",
         )