From 2a85c4c5d9e9892916a998ce540eaebf397ae01e Mon Sep 17 00:00:00 2001 From: barredterra <14891507+barredterra@users.noreply.github.com> Date: Tue, 13 Feb 2024 11:29:30 +0100 Subject: [PATCH] refactor: no inline if-statement Co-authored-by: Akhil Narang --- frappe/utils/backups.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/frappe/utils/backups.py b/frappe/utils/backups.py index b69ef799b1..a916621616 100644 --- a/frappe/utils/backups.py +++ b/frappe/utils/backups.py @@ -131,8 +131,10 @@ class BackupGenerator: if not self.ignore_conf: backup_conf = frappe.conf.get("backup", {}) if not self.backup_includes: - specified_tables = get_tables(backup_conf.get("includes", [])) - self.backup_includes = (specified_tables + base_tables) if specified_tables else [] + if specified_tables := get_tables(backup_conf.get("includes", [])): + self.backup_includes = specified_tables + base_tables + else: + self.backup_includes = [] if not self.backup_excludes: self.backup_excludes = get_tables(backup_conf.get("excludes", []))