fix/small-fixes: drobné úpravy stylu a čitelnosti kódu (#54)

* small fixes

* fix: refactoring
rozbaleny listy, doplněny trailing commas

amcr_viewer/amcr_dialog.py

@@ -44,16 +44,19 @@ class UpdateCodelistsTask(QgsTask):
             refresh_globals()
             QgsMessageLog.logMessage(
                 "Hesláře AMČR byly úspěšně aktualizovány.",
-                "AMČR", Qgis.Info)
+                "AMČR", Qgis.Info
+            )
         else:
             if self.isCanceled():
                 QgsMessageLog.logMessage(
                     "Aktualizace heslářů byla zrušena.",
-                    "AMČR", Qgis.Warning)
+                    "AMČR", Qgis.Warning
+                )
             else:
                 QgsMessageLog.logMessage(
                     f"Chyba aktualizace: {self.exception}",
-                    "AMČR", Qgis.Critical)
+                    "AMČR", Qgis.Critical
+                )
 
 
 class FilterableSelectionDialog(QDialog):
@@ -151,10 +154,20 @@ class AmcrFilterDialog(QDialog):
 
         # Cache dictionary to store selected codes for each category
         self.selection_cache = {
-            'organizace': [], 'kraj': [], 'obdobi': [], 'areal': [],
+            'organizace': [],
-            'typ_akce': [], 'okres': [], 'katastr': [], 'vedouci': [],
+            'kraj': [],
-            'pian_presnost': [], 'pristupnost': [], 'typ_lokality': [],
+            'obdobi': [],
-            'druh_lokality': [], 'jistota': [], 'lokalita_zachovalost': []
+            'areal': [],
+            'typ_akce': [],
+            'okres': [],
+            'katastr': [],
+            'vedouci': [],
+            'pian_presnost': [],
+            'pristupnost': [],
+            'typ_lokality': [],
+            'druh_lokality': [],
+            'jistota': [],
+            'lokalita_zachovalost': [],
         }
 
         layout = QVBoxLayout()
@@ -363,7 +376,7 @@ class AmcrFilterDialog(QDialog):
             self.selection_cache[cache_key] = [
                 'HES-000861',
                 'HES-000862',
-                'HES-000863'
+                'HES-000863',
             ]
 
         btn.clicked.connect(open_dialog)

amcr_viewer/amcr_tools.py

@@ -283,7 +283,7 @@ def load_amcr_data(canvas, bb, filters=None,
         base_params = {
             "mapa": "true",
             "sort": "ident_cely asc",
-            "entity": typ_dat
+            "entity": typ_dat,
         }
 
         # Restrict search to map window if requested
@@ -479,7 +479,7 @@ def load_amcr_data(canvas, bb, filters=None,
                 "katastr": g_list(doc, 'katastr'),
                 "dalsi_katastr": dalsi_kat_str,
                 "pristupnost": g(doc, 'pristupnost'),
-                "loc": g_list(doc, 'loc')
+                "loc": g_list(doc, 'loc'),
             }
 
             # Add entity-specific metadata
@@ -668,8 +668,10 @@ def load_amcr_data(canvas, bb, filters=None,
         )
 
         fl_pian = [
-            "ident_cely", "pian_typ",
+            "ident_cely",
-            "pian_chranene_udaje", "pian_presnost"
+            "pian_typ",
+            "pian_chranene_udaje",
+            "pian_presnost",
         ]
 
         for i in range(0, total_pians, BATCH_PIAN):
@@ -682,7 +684,7 @@ def load_amcr_data(canvas, bb, filters=None,
                 "entity": "pian",
                 "q": fq_pian,
                 "rows": len(batch),
-                "fl": ",".join(fl_pian)
+                "fl": ",".join(fl_pian),
             }
             try:
                 QApplication.processEvents()
@@ -739,7 +741,7 @@ def load_amcr_data(canvas, bb, filters=None,
             QgsField("odkaz_do_digiarchivu", QMetaType.Type.QString),
             QgsField("okres", QMetaType.Type.QString),
             QgsField("katastr", QMetaType.Type.QString),
-            QgsField("dalsi_katastry", QMetaType.Type.QString)
+            QgsField("dalsi_katastry", QMetaType.Type.QString),
         ]
 
         # Extend table based on data type
@@ -763,7 +765,7 @@ def load_amcr_data(canvas, bb, filters=None,
                 QgsField("popis_lokality", QMetaType.Type.QString),
                 QgsField("typ_lokality", QMetaType.Type.QString),
                 QgsField("druh_lokality", QMetaType.Type.QString),
-                QgsField("zachovalost", QMetaType.Type.QString)
+                QgsField("zachovalost", QMetaType.Type.QString),
             ]
 
         cols.append(QgsField("Přístupnost", QMetaType.Type.QString))
@@ -920,7 +922,7 @@ def load_amcr_data(canvas, bb, filters=None,
                                 + meta['ident_cely'],
                                 meta['az_okres'],
                                 meta['katastr'],
-                                meta['dalsi_katastr']
+                                meta['dalsi_katastr'],
                             ]
                             if is_akce:
                                 atributy.extend([
@@ -942,7 +944,7 @@ def load_amcr_data(canvas, bb, filters=None,
                                     meta['lokalita_popis'],
                                     meta['lokalita_typ'],
                                     meta['lokalita_druh'],
-                                    meta['lokalita_zachovalost']
+                                    meta['lokalita_zachovalost'],
                                 ])
 
                             atributy.append(meta['pristupnost'])
@@ -972,12 +974,12 @@ def load_amcr_data(canvas, bb, filters=None,
             (feats_pt, vl_point, "Body"),
         ]
 
-        for f, l, n in layers_to_process:
+        for f, L, n in layers_to_process:
             if f:
-                l.dataProvider().addFeatures(f)
+                L.dataProvider().addFeatures(f)
-                l.updateExtents()
+                L.updateExtents()
-                l.setName(f"AMCR_{archeologicky_zaznam}_{n}")
+                L.setName(f"AMCR_{archeologicky_zaznam}_{n}")
-                proj.addMapLayer(l)
+                proj.addMapLayer(L)
                 added += len(f)
 
         if network_error: