From e6c12d9ac74b8c950ac76a5d71486f3506e0e88d Mon Sep 17 00:00:00 2001
From: pappapisshu <39009553+pappapisshu@users.noreply.github.com>
Date: Fri, 23 Dec 2022 06:39:36 +0100
Subject: [PATCH] Backup configuration saving mechanism is fixed

The region value used for backups on S3-compatible services
is now saved correctly even if the user changes the configuration
from backup of type S3 to another.
---
 management/templates/system-backup.html | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/management/templates/system-backup.html b/management/templates/system-backup.html
index e73fdabf..f9a11264 100644
--- a/management/templates/system-backup.html
+++ b/management/templates/system-backup.html
@@ -285,11 +285,13 @@ function set_custom_backup() {
   var target_pass = $("#backup-target-pass").val();
 
   var target;
-  if (target_type == "local" || target_type == "off")
+  var target_region = '';
+  if (target_type == "local" || target_type == "off") {
     target = target_type;
-  else if (target_type == "s3")
+  } else if (target_type == "s3") {
     target = "s3://" + $("#backup-target-s3-host").val() + "/" + $("#backup-target-s3-path").val();
-  else if (target_type == "rsync") {
+    target_region = $("#backup-target-s3-region").val();
+  } else if (target_type == "rsync") {
     target = "rsync://" + $("#backup-target-rsync-user").val() + "@" + $("#backup-target-rsync-host").val()
                                                                 + "/" + $("#backup-target-rsync-path").val();
     target_user = '';
@@ -300,10 +302,8 @@ function set_custom_backup() {
     target_pass = '';
   }
 
-
-  var target_region = $("#backup-target-s3-region").val();
-
   var min_age = $("#min-age").val();
+
   api(
     "/system/backup/config",
     "POST",