From 479e256b1e099eb2f1ca90eb78562d46c563826f Mon Sep 17 00:00:00 2001 From: Anton Vakhrushev Date: Sat, 20 Dec 2025 10:36:19 +0300 Subject: [PATCH] Backups: use constants for file names --- files/backups/backup-all.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/files/backups/backup-all.py b/files/backups/backup-all.py index 279e305..2d80286 100644 --- a/files/backups/backup-all.py +++ b/files/backups/backup-all.py @@ -39,6 +39,13 @@ TELEGRAM_BOT_TOKEN = config.get("telegram", "TELEGRAM_BOT_TOKEN") TELEGRAM_CHAT_ID = config.get("telegram", "TELEGRAM_CHAT_ID") NOTIFICATIONS_NAME = config.get("telegram", "NOTIFICATIONS_NAME") +# File name to store directories and files to back up +BACKUP_TARGETS_FILE = "backup-targets" + +# Default directory fo backups (relative to app dir) +# Used when backup-targets file not exists +BACKUP_DEFAULT_DIR = "backups" + class BackupManager: def __init__(self): @@ -146,7 +153,7 @@ class BackupManager: for app_dir_str, _ in app_dirs: app_dir = Path(app_dir_str) - targets_file = app_dir / "backup-targets" + targets_file = app_dir / BACKUP_TARGETS_FILE resolved_targets: List[Path] = [] if targets_file.exists(): @@ -167,7 +174,7 @@ class BackupManager: self.warnings.append(warning_msg) else: # Fallback to default backups directory when no list is provided. - default_target = (app_dir / "backups").resolve() + default_target = (app_dir / BACKUP_DEFAULT_DIR).resolve() if default_target.exists(): resolved_targets.append(default_target) else: