Nur noch OGC: Aenderungen speichern in der Seitenleiste, GeoJSON-Weg entfernt
Erik, 08.09.2026: "in die GUI das Speichern via OGC einbauen, GeoJSON nicht mehr verwenden." - Seitenleiste hat jetzt zwei Bloecke: "Layer" (Auswahl, laden, Aenderungen speichern) und "Darstellung" (Stil, Vorlage-Haken). - Neuer Knopf "Aenderungen speichern" -> VlnKartenPlugin.run_save_edits(): schliesst die offenen Bearbeitungssitzungen ALLER geladenen OAPIF-Layer auf einmal; ohne offene Bearbeitung kommt ein Hinweis statt einer Aktion. Zurueckgeschrieben wird ohnehin je Feature (Part 4), der Knopf beendet nur die Sitzung. - Der GeoJSON-Weg ist raus: die vier Datensatz-Knoepfe, "Aktiven Layer hochladen", DATASETS, run_load_dataset, run_upload_active_layer, _confirm_replace_existing, vln_karten/layer_manager.py sowie KartenApiClient.load_layer_complete/save_feature_collection. - Stil-Hinweise angepasst: den Stil des eigenen Verfahrens darf jeder speichern, die Vorlage nur die Gruppe MODUL_KARTENSTIL (Server-PR #565). Geprueft headless mit QGIS 4.2 gegen die Live-API (TG Test 27998): ohne Bearbeitung Hinweis, mit Aenderung Sitzung geschlossen, API zaehlt +1, Feature sichtbar, danach wieder aufgeraeumt. Panel-Test deckt die neuen Knoepfe ab. metadata.txt 0.5.0. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
erik
+35
-187
@@ -18,14 +18,6 @@ from qgis.core import Qgis, QgsDataSourceUri, QgsProject
|
||||
|
||||
from .api_client import ApiError, AuthError, KartenApiClient
|
||||
from .panel import VlnPanel
|
||||
from .layer_manager import (
|
||||
PROP_DATASET,
|
||||
PROP_VERFAHREN,
|
||||
feature_collection_to_layers,
|
||||
layer_api_path,
|
||||
layers_to_feature_collection,
|
||||
plugin_layers,
|
||||
)
|
||||
from .login_dialog import LoginDialog, SETTINGS_GROUP
|
||||
from . import ogc_layers
|
||||
|
||||
@@ -35,36 +27,6 @@ MENU_TITLE = "&VLN Karten"
|
||||
PROJECT_SCOPE = "vln_karten"
|
||||
PROJECT_KEY_VKZ = "/vkz"
|
||||
|
||||
# Karten-Layer der VLN-Manager-API (map.php). Weitere Layer (st, we)
|
||||
# wären je ein Eintrag mehr. {vkz} = Verfahrenskennzeichen der TG.
|
||||
DATASETS = {
|
||||
"umringe": {
|
||||
"label": "Verfahrensgebiet",
|
||||
"path": "/maps/umringe/{vkz}",
|
||||
"geometry": "MultiPolygon",
|
||||
},
|
||||
"p41": {
|
||||
"label": "Plan 41 (Wege- und Gewässerplan)",
|
||||
"path": "/maps/p41/{vkz}",
|
||||
"geometry": "MultiPolygon",
|
||||
},
|
||||
# DB-Art KAS: enthält, was im Web-GIS über "in Datenbank speichern"
|
||||
# abgelegt wurde — kann je VKZ leer sein, solange dort noch nichts
|
||||
# gespeichert wurde.
|
||||
"kas": {
|
||||
"label": "Karte alter Stand (KAS)",
|
||||
"path": "/maps/kas/{vkz}",
|
||||
"geometry": "MultiPolygon",
|
||||
},
|
||||
# DB-Art WE: Wertermittlung (Web-GIS-Modul we.js).
|
||||
"we": {
|
||||
"label": "Wertermittlung (WE)",
|
||||
"path": "/maps/we/{vkz}",
|
||||
"geometry": "MultiPolygon",
|
||||
},
|
||||
}
|
||||
|
||||
|
||||
class VlnKartenPlugin:
|
||||
def __init__(self, iface):
|
||||
self.iface = iface
|
||||
@@ -77,9 +39,9 @@ class VlnKartenPlugin:
|
||||
# ------------------------------------------------------------------
|
||||
|
||||
def initGui(self):
|
||||
# Seitenleiste statt Werkzeugleiste (Erik, 08.09.2026). Die OGC-Knöpfe stehen oben:
|
||||
# sie laden native, editierbare OAPIF-Layer, die QGIS selbst zurückschreibt (Part 4).
|
||||
# Die GeoJSON-Knöpfe darunter sind der klassische Weg (Kopie laden, später hochladen).
|
||||
# Seitenleiste statt Werkzeugleiste (Erik, 08.09.2026). Geladen und gespeichert wird
|
||||
# ausschließlich über die OGC API: die Layer sind nativ editierbar, QGIS schreibt jede
|
||||
# Änderung einzeln zurück (Part 4). Der frühere GeoJSON-Weg ist entfallen.
|
||||
self.panel = VlnPanel(
|
||||
self.iface.mainWindow(),
|
||||
callbacks={
|
||||
@@ -87,11 +49,9 @@ class VlnKartenPlugin:
|
||||
"logout": self.run_logout,
|
||||
"verfahren_gewaehlt": self.on_verfahren_changed,
|
||||
"load_ogc": self.run_load_ogc,
|
||||
"save_edits": self.run_save_edits,
|
||||
"save_style": self.run_save_style,
|
||||
"load_dataset": self.run_load_dataset,
|
||||
"upload": self.run_upload_active_layer,
|
||||
},
|
||||
datensaetze=[(key, dataset["label"]) for key, dataset in DATASETS.items()],
|
||||
)
|
||||
self.iface.addDockWidget(Qt.DockWidgetArea.RightDockWidgetArea, self.panel)
|
||||
|
||||
@@ -250,149 +210,6 @@ class VlnKartenPlugin:
|
||||
# Laden & Hochladen
|
||||
# ------------------------------------------------------------------
|
||||
|
||||
def run_load_dataset(self, dataset_key):
|
||||
if not self._ensure_logged_in():
|
||||
return
|
||||
vkz = self._current_vkz()
|
||||
if not vkz:
|
||||
self._show_error(
|
||||
"Kein Verfahren gewählt",
|
||||
"Bitte zuerst in der Auswahlliste ein Verfahren wählen.",
|
||||
)
|
||||
return
|
||||
|
||||
dataset = DATASETS[dataset_key]
|
||||
if not self._confirm_replace_existing(dataset_key, vkz, dataset["label"]):
|
||||
return
|
||||
|
||||
path = dataset["path"].format(vkz=vkz)
|
||||
try:
|
||||
# Vollständig aus KARTE_OBJEKT: Listen-Endpunkt mit Paging
|
||||
# (wie das Web-GIS, aber ohne 2000er-Limit).
|
||||
feature_collection = self.client.load_layer_complete(dataset_key, vkz)
|
||||
layers = feature_collection_to_layers(
|
||||
feature_collection,
|
||||
"%s %s" % (dataset["label"], vkz),
|
||||
dataset["geometry"],
|
||||
dataset_key,
|
||||
vkz,
|
||||
path,
|
||||
)
|
||||
except AuthError:
|
||||
self._handle_session_expired()
|
||||
return
|
||||
except (ApiError, RuntimeError) as exc:
|
||||
self._show_error("Laden fehlgeschlagen", str(exc))
|
||||
return
|
||||
|
||||
total = sum(layer.featureCount() for layer in layers)
|
||||
if total == 0:
|
||||
self.iface.messageBar().pushMessage(
|
||||
"VLN Karten",
|
||||
"Der Server hat für VKZ %s keine Objekte im Layer "
|
||||
"'%s' — es wurde ein leerer Layer zum Digitalisieren "
|
||||
"angelegt." % (vkz, dataset["label"]),
|
||||
level=Qgis.Warning,
|
||||
duration=10,
|
||||
)
|
||||
return
|
||||
|
||||
detail = ", ".join(
|
||||
"%s: %d" % (layer.name(), layer.featureCount()) for layer in layers
|
||||
)
|
||||
self.iface.messageBar().pushMessage(
|
||||
"VLN Karten",
|
||||
"%d Layer geladen (%s)." % (len(layers), detail),
|
||||
level=Qgis.Success,
|
||||
duration=5,
|
||||
)
|
||||
|
||||
def _confirm_replace_existing(self, dataset_key, vkz, label):
|
||||
"""Ersetzt bereits geladene Layer dieses Datensatzes nach
|
||||
Rückfrage — doppelte Layer-Sätze würden beim Hochladen alle
|
||||
Objekte dupliziert an den Server schicken."""
|
||||
existing = plugin_layers(dataset_key, vkz)
|
||||
if not existing:
|
||||
return True
|
||||
answer = QMessageBox.question(
|
||||
self.iface.mainWindow(),
|
||||
"Layer ersetzen",
|
||||
"%s für VKZ %s ist bereits im Projekt geladen (%s).\n"
|
||||
"Beim Neuladen werden diese Layer ersetzt — nicht "
|
||||
"hochgeladene Änderungen gehen verloren. Fortfahren?"
|
||||
% (label, vkz, ", ".join(layer.name() for layer in existing)),
|
||||
)
|
||||
if answer != QMessageBox.StandardButton.Yes:
|
||||
return False
|
||||
QgsProject.instance().removeMapLayers([layer.id() for layer in existing])
|
||||
return True
|
||||
|
||||
def run_upload_active_layer(self):
|
||||
if not self._ensure_logged_in():
|
||||
return
|
||||
|
||||
active = self.iface.activeLayer()
|
||||
path = layer_api_path(active) if active else None
|
||||
if not path:
|
||||
self._show_error(
|
||||
"Hochladen nicht möglich",
|
||||
"Der aktive Layer wurde nicht über dieses Plugin geladen. "
|
||||
"Bitte einen über 'VLN Karten' geladenen Layer auswählen.",
|
||||
)
|
||||
return
|
||||
|
||||
# Der PUT ersetzt den kompletten Layer-Bestand der VKZ auf dem
|
||||
# Server. Deshalb werden immer alle Teil-Layer dieses Datensatzes
|
||||
# (Punkte/Linien/Flächen) gemeinsam hochgeladen — nur den aktiven
|
||||
# zu senden, würde die anderen Geometrietypen serverseitig löschen.
|
||||
dataset_key = active.customProperty(PROP_DATASET)
|
||||
vkz = active.customProperty(PROP_VERFAHREN)
|
||||
siblings = plugin_layers(dataset_key, vkz)
|
||||
|
||||
for layer in siblings:
|
||||
if layer.isEditable() and not layer.commitChanges():
|
||||
self._show_error(
|
||||
"Hochladen nicht möglich",
|
||||
"Die offene Bearbeitungssitzung von '%s' konnte nicht "
|
||||
"gespeichert werden." % layer.name(),
|
||||
)
|
||||
return
|
||||
|
||||
dataset = DATASETS.get(dataset_key, {})
|
||||
total = sum(layer.featureCount() for layer in siblings)
|
||||
listing = "\n".join(
|
||||
"• %s (%d Objekte)" % (layer.name(), layer.featureCount())
|
||||
for layer in siblings
|
||||
)
|
||||
answer = QMessageBox.question(
|
||||
self.iface.mainWindow(),
|
||||
"Zum Server hochladen",
|
||||
"%s für VKZ %s übertragen?\n\n%s\n\n"
|
||||
"Die Layer werden zusammengeführt (%d Objekte); der bisherige "
|
||||
"Bestand dieser VKZ auf dem Server wird ersetzt."
|
||||
% (dataset.get("label", dataset_key), vkz, listing, total),
|
||||
)
|
||||
if answer != QMessageBox.StandardButton.Yes:
|
||||
return
|
||||
|
||||
try:
|
||||
feature_collection = layers_to_feature_collection(siblings)
|
||||
self.client.save_feature_collection(path, feature_collection)
|
||||
except AuthError:
|
||||
self._handle_session_expired()
|
||||
return
|
||||
except ApiError as exc:
|
||||
self._show_error("Hochladen fehlgeschlagen", str(exc))
|
||||
return
|
||||
|
||||
self.iface.messageBar().pushMessage(
|
||||
"VLN Karten",
|
||||
"%d Objekte aus %d Layer(n) erfolgreich hochgeladen."
|
||||
% (total, len(siblings)),
|
||||
level=Qgis.Success,
|
||||
duration=5,
|
||||
)
|
||||
|
||||
def run_load_ogc(self):
|
||||
"""Alle Sammlungen des gewählten Verfahrens (Verfahrens-Layer, Vorerhebung,
|
||||
Maßnahmenblätter) als OAPIF-Layer laden — editierbar in QGIS, Rückschreiben je Feature."""
|
||||
@@ -440,6 +257,37 @@ class VlnKartenPlugin:
|
||||
self.iface, self.client.base_url, vkz, self.panel.aktueller_text(), collections,
|
||||
authcfg, api_key=None if authcfg else self.client.api_key)
|
||||
|
||||
def run_save_edits(self):
|
||||
"""Offene Bearbeitungssitzungen aller geladenen VLN-Layer schließen.
|
||||
|
||||
Über die OGC API schreibt QGIS jede Änderung einzeln zurück (Part 4) — dieser Knopf
|
||||
beendet nur die Bearbeitung, so wie „Layeränderungen speichern" in QGIS, aber für
|
||||
alle Layer des Verfahrens auf einmal.
|
||||
"""
|
||||
layer = [l for l in QgsProject.instance().mapLayers().values()
|
||||
if getattr(l, "providerType", None) and l.providerType() == "OAPIF"]
|
||||
offen = [l for l in layer if l.isEditable()]
|
||||
if not offen:
|
||||
self.iface.messageBar().pushMessage(
|
||||
"VLN Karten", "Keine offene Bearbeitung — es gibt nichts zu speichern.",
|
||||
level=Qgis.Info, duration=5)
|
||||
return
|
||||
gespeichert, fehler = [], []
|
||||
for l in offen:
|
||||
if l.commitChanges():
|
||||
gespeichert.append(l.name())
|
||||
else:
|
||||
fehler.append("%s: %s" % (l.name(), "; ".join(l.commitErrors())[:120]))
|
||||
if fehler:
|
||||
self._show_error("Nicht alles gespeichert", "\n".join(fehler))
|
||||
if gespeichert:
|
||||
self.iface.messageBar().pushMessage(
|
||||
"VLN Karten", "%d Layer gespeichert (%s)." % (len(gespeichert),
|
||||
", ".join(gespeichert[:4])),
|
||||
level=Qgis.Success, duration=6)
|
||||
if self.panel is not None:
|
||||
self.panel.set_status("%d von %d Layern gespeichert." % (len(gespeichert), len(offen)))
|
||||
|
||||
def run_save_style(self):
|
||||
"""Den Stil des aktiven OGC-Layers auf den Server zurückschreiben.
|
||||
|
||||
|
||||
Reference in New Issue
Block a user