Ruff-format fixes

Author: Mtk112

arho_feature_template/core/create_land_use_plan.py

@@ -2,7 +2,7 @@
 from qgis.PyQt.QtGui import QStandardItemModel
 from qgis.utils import iface
 
-from .feature_template_library import TemplateGeometryDigitizeMapTool
+from arho_feature_template.core.feature_template_library import TemplateGeometryDigitizeMapTool
 
 
 class LandUsePlanTemplater:
@@ -21,24 +21,19 @@ def start_digitizing_for_layer(self, layer: QgsVectorLayer):
 
     def create_feature(self, feature):
         """Creates a new feature using stored dialog attributes."""
-        print("create_feature called with feature:", feature)
 
         if not self.plan_id or not self.plan_name or not self.plan_type:
-            print("Plan information not available")
             return
 
         layer = self.digitize_map_tool.layer()
 
         if not layer:
-            print("Layer is not set for digitizing.")
             return
 
-        print("Layer found:", layer.name(), "Editable:", layer.isEditable())
         new_feature = QgsFeature(layer.fields())
 
         geometry = feature.geometry()
         if not isinstance(geometry, QgsGeometry):
-            print("Expected geometry to be of type QgsGeometry, got:", type(geometry))
             return
 
         new_feature.setGeometry(geometry)
@@ -58,8 +53,5 @@ def create_feature(self, feature):
             layer.startEditing()
 
         layer.beginEditCommand("Create land use plan feature.")
-        if layer.addFeature(new_feature):
-            print(f"Feature with ID {self.plan_id}, name {self.plan_name}, and type {plan_type_value} created successfully.")
-        else:
-            print("Failed to add feature to the layer.")
+        layer.addFeature(new_feature)
         layer.commitChanges()

arho_feature_template/core/update_plan.py

@@ -1,7 +1,7 @@
 from dataclasses import dataclass
-from typing import List
 
 from qgis.core import QgsMapLayer, QgsProject, QgsVectorLayer
+from qgis.utils import iface
 
 
 # To be extended and moved
@@ -10,18 +10,21 @@ class LandUsePlan:
     id: str
     name: str
 
+
 # To be replaced later
 LAYER_PLAN_ID_MAP = {
     "land_use_plan": "id",
     "land_use_area": "plan_id",
     "Osa-alue": "plan_id",
 }
 
+
 def update_selected_plan(new_plan: LandUsePlan):
     """Update the project layers based on the selected land use plan."""
-    id = new_plan.id
+    plan_id = new_plan.id
     for layer_name, field_name in LAYER_PLAN_ID_MAP.items():
-        set_filter_for_vector_layer(layer_name, field_name, id)
+        set_filter_for_vector_layer(layer_name, field_name, plan_id)
+
 
 def set_filter_for_vector_layer(layer_name: str, field_name: str, field_value: str):
     # Get layer and perform checks
@@ -36,18 +39,20 @@ def set_filter_for_vector_layer(layer_name: str, field_name: str, field_value: s
     query = f"{field_name} = {field_value}"
     if not layer.setSubsetString(query):
         # TODO: Convert to log msg?
-        print(f"ERROR: Failed to filter layer {layer_name} with query {query}")
+        iface.messageBar().pushMessage("Error", f"Failed to filter layer {layer_name} with query {query}", level=3)
+
 
 def _check_layer_count(layers: list) -> bool:
     """Check if any layers are returned."""
     if not layers:
-        print("ERROR: No layers found with the specified name.")
+        iface.messageBar().pushMessage("Error", "ERROR: No layers found with the specified name.", level=3)
         return False
     return True
 
+
 def _check_vector_layer(layer: QgsMapLayer) -> bool:
     """Check if the given layer is a vector layer."""
     if not isinstance(layer, QgsVectorLayer):
-        print(f"ERROR: Layer {layer.name()} is not a vector layer: {type(layer)}")
+        iface.messageBar().pushMessage("Error", f"Layer {layer.name()} is not a vector layer: {type(layer)}", level=3)
         return False
     return True

arho_feature_template/gui/new_land_use_plan_form.py

@@ -17,9 +17,9 @@ def __init__(self, parent=None):
 
         self.plan_widgets = defaultdict(dict)
 
-        self.plan_widgets['id'] = self.lineEdit_plan_id
-        self.plan_widgets['name'] = self.lineEdit_plan_name
-        self.plan_widgets['type'] = self.comboBox_plan_type
+        self.plan_widgets["id"] = self.lineEdit_plan_id
+        self.plan_widgets["name"] = self.lineEdit_plan_name
+        self.plan_widgets["type"] = self.comboBox_plan_type
 
     @property
     def plan_id(self):

arho_feature_template/gui/new_land_use_plan_form.ui

@@ -104,4 +104,4 @@
    <slot>reject()</slot>
   </connection>
  </connections>
-</ui>
+</ui>