Remove deprecated normalization method for backup cloud max backups and streamline retrieval of max_backups in Nextcloud and Samba modules. This change ensures consistent handling of max_backups directly from the configuration object.

Author: Sleepwalker86

packages/helpermodules/update_config.py

@@ -710,9 +710,6 @@ def update(self):
         try:
             # erst breaking changes auflösen, sonst sind alte Topics schon gelöscht
             self.__solve_breaking_changes()
-            # Normalisiere neue Felder auch im normalen Update-Pfad, nicht nur in
-            # upgrade_datastore_* (UI kann aktuell bei "Feld geleert" null übermitteln).
-            self.__normalize_backup_cloud_max_backups()
             self.__remove_outdated_topics()
             self._remove_invalid_topics()
             self.__pub_missing_defaults()
@@ -813,31 +810,6 @@ def __solve_breaking_changes(self) -> None:
                 pub_system_message(
                     {}, "Fehler bei der Aktualisierung der Konfiguration des Brokers.", MessageType.ERROR)
 
-    def __normalize_backup_cloud_max_backups(self) -> None:
-        """
-        Serverseitige Normalisierung für `configuration.max_backups` im
-        normalen Update-Pfad.
-
-        Semantik:
-        - `0` => automatische Löschung deaktiviert
-        """
-        for topic, payload in list(self.all_received_topics.items()):
-            if topic != "openWB/system/backup_cloud/config":
-                continue
-
-            configuration_payload = decode_payload(payload)
-            if not isinstance(configuration_payload, dict):
-                continue
-
-            cloud_type = configuration_payload.get("type")
-            if cloud_type not in ("nextcloud", "samba"):
-                continue
-
-            configuration_payload.setdefault("configuration", {})
-            if configuration_payload["configuration"].get("max_backups") is None:
-                configuration_payload["configuration"]["max_backups"] = 0
-                self.__update_topic(topic, configuration_payload)
-
     def _loop_all_received_topics(self, callback) -> None:
         modified_topics = {}
         for topic, payload in self.all_received_topics.items():

packages/modules/backup_clouds/nextcloud/backup_cloud.py

@@ -43,7 +43,7 @@ def _list_backups(config: NextcloudBackupCloudConfiguration,
     Listet alle vorhandenen Backupdateien, die zum gleichen OpenWB-Suffix gehören
     (Pattern-Match am Dateinamen) und gibt eine nach Dateinamen sortierte Liste zurück.
     """
-    max_backups = getattr(config, "max_backups", None)
+    max_backups = config.max_backups
     if not max_backups or max_backups <= 0:
         return []
 
@@ -105,7 +105,7 @@ def _enforce_retention(config: NextcloudBackupCloudConfiguration, backup_filenam
     gleichen OpenWB-Suffix (Pattern-Match) übrig bleiben. Sortierung erfolgt nach
     Dateinamen, es bleiben die letzten N erhalten.
     """
-    max_backups = getattr(config, "max_backups", None)
+    max_backups = config.max_backups
     if not max_backups or max_backups <= 0:
         return
 

packages/modules/backup_clouds/samba/backup_cloud.py

@@ -35,7 +35,7 @@ def _enforce_retention(conn, config: SambaBackupCloudConfiguration, backup_filen
     ".openwb-backup.gpg" enden (stabiler Filter; der Teil vor der Endung ist
     timestamps-/versionspezifisch und darf nicht zum Gruppieren verwendet werden).
     """
-    max_backups = getattr(config, "max_backups", None)
+    max_backups = config.max_backups
     if not max_backups or max_backups <= 0:
         return