adjust based on review:

- remove parent_regulation from yaml
- remove letter_code from yaml
- small refactor

Author: nmaarnio

arho_feature_template/core/plan_regulation_config.py

@@ -88,9 +88,7 @@ class PlanRegulationConfig:
     """Describes the configuration of a plan regulation."""
 
     regulation_code: str
-    parent_regulation: str | None
-    heading_only: bool
-    letter_code: str | None
+    category_only: bool
     value_type: ValueType | None
     unit: Unit | None
     child_regulations: list[PlanRegulationConfig] | None
@@ -104,14 +102,8 @@ def from_dict(cls, data: dict) -> PlanRegulationConfig:
         """
         return cls(
             regulation_code=data["regulation_code"],
-            parent_regulation=data.get("parent_regulation"),
-            heading_only=data.get("heading_only", False),
-            letter_code=data.get("letter_code"),
+            category_only=data.get("category_only", False),
             value_type=ValueType(data["value_type"]) if "value_type" in data else None,
             unit=Unit(data["unit"]) if "unit" in data else None,
-            child_regulations=(
-                [PlanRegulationConfig.from_dict(config) for config in data["child_regulations"]]
-                if "child_regulations" in data
-                else None
-            ),
+            child_regulations=[PlanRegulationConfig.from_dict(config) for config in data.get("child_regulations", [])],
         )

arho_feature_template/gui/new_plan_regulation_group_form.py

@@ -53,7 +53,7 @@ def initialize_plan_regulations(self):
 
     def add_selected_plan_regulation(self, item: QTreeWidgetItem, column: int):
         config: PlanRegulationConfig = item.data(column, Qt.UserRole)  # Retrieve the associated config
-        if config.heading_only:
+        if config.category_only:
             return
         self.add_plan_regulation(config)
 

arho_feature_template/resources/kaavamaaraykset.yaml

@@ -2,123 +2,89 @@ version: 1
 plan_regulations:
 # ASUMISEN ALUE (valmis?)
   - regulation_code: asumisenAlue
-    letter_code: [A, AA]
     child_regulations:
 
     - regulation_code: asuinPientaloAlue
-      parent_regulation: asumisenAlue
-      letter_code: AP
       child_regulations:
 
       - regulation_code: erillistenAsuinpientalojenAlue
-        parent_regulation: asuinPientaloAlue
-        letter_code: AO
 
       - regulation_code: rivitalojenJaMuidenKytkettyjenAsuinpientalojenAlue
-        parent_regulation: asuinPientaloAlue
-        letter_code: AR
 
     - regulation_code: asuinkerrostaloalue
-      parent_regulation: asumisenAlue
-      letter_code: AK
 
     - regulation_code: asumistaPalvelevaYhteiskayttoinenAlue
-      parent_regulation: asumisenAlue
-      letter_code: AH
 
     - regulation_code: maatilanTalouskeskuksenAlue
-      parent_regulation: asumisenAlue
-      letter_code: AM
 
     - regulation_code: kylaAlue
-      parent_regulation: asumisenAlue
-      letter_code: AT
 
 # VIHERALUE (kesken)
   - regulation_code: viheralue
-    heading_only: true
+    category_only: true
     child_regulations:
 
     - regulation_code: virkistysalue
-      parent_regulation: viheralue
-      letter_code: V
 
 # VESIALUE (valmis?)
   - regulation_code: vesialue
-    letter_code: W
     child_regulations:
 
     - regulation_code: pohjavesialue
-      parent_regulation: vesialue
-      letter_code: pv
 
 # RAKENTAMISEN MÄÄRÄ (valmis?)
   - regulation_code: rakentamisenMaara
-    heading_only: true
+    category_only: true
     child_regulations:
 
     - regulation_code: sallittuKerrosala
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
       unit: k-m2
 
     - regulation_code: sallittuRakennustilavuus
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
       unit: m3
 
     - regulation_code: tehokkuusluku
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen desimaali
       unit: k-m2/m2
       child_regulations:
 
       - regulation_code: rakentamisenSuhdeAlueenPintaAlaan
-        parent_regulation: tehokkuusluku
         value_type: positiivinen kokonaisluku
         unit: prosentti
 
     - regulation_code: valjyysluku
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen desimaali
       unit: m2/k-m2
 
     - regulation_code: maanpaallinenKerroslukuLuku
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
 
     - regulation_code: maanpaallinenKerroslukuArvovali
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku arvoväli
 
     - regulation_code: maanalainenKerroslukuLuku
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
 
     - regulation_code: maanalainenKerroslukuArvovali
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku arvoväli
 
     - regulation_code: kellarinSallittuOsuusKerrosalasta
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
       unit: prosentti
 
     - regulation_code: ullakonSallittuOsuusKerrosalasta
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
       unit: prosentti
 
     - regulation_code: asuntojenMaara
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
 
     - regulation_code: rakennuspaikkojenMaara
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
 
     - regulation_code: tuulivoimaloidenMaara
-      parent_regulation: rakentamisenMaara
       value_type: positiivinen kokonaisluku
 
 # SANALLINEN MÄÄRÄYS (valmis?)
@@ -127,36 +93,25 @@ plan_regulations:
     child_regulations:
 
     - regulation_code: rakentamisrajoitusYleiskaava
-      parent_regulation: sanallinenMaarays
-      letter_code: yk-rr
       value_type: kieliversioitu teksti
 
     - regulation_code: rakentamisrajoitusMaakuntakaava
-      parent_regulation: sanallinenMaarays
       value_type: kieliversioitu teksti
 
     - regulation_code: maaraAikainenRakentamisrajoitus
-      parent_regulation: sanallinenMaarays
       value_type: kieliversioitu teksti
 
     - regulation_code: toimenpiderajoitus
-      parent_regulation: sanallinenMaarays
-      letter_code: tr
       value_type: kieliversioitu teksti
 
     - regulation_code: maaraAikainenKieltoRakennuksenRakentamiseksi
-      parent_regulation: sanallinenMaarays
-      letter_code: mrk
       value_type: kieliversioitu teksti
 
     - regulation_code: suunnittelumaarays
-      parent_regulation: sanallinenMaarays
       value_type: kieliversioitu teksti
 
     - regulation_code: rakentamismaarays
-      parent_regulation: sanallinenMaarays
       value_type: kieliversioitu teksti
 
     - regulation_code: suojelumaarays
-      parent_regulation: sanallinenMaarays
       value_type: kieliversioitu teksti