From 307b2608a5d9b20ca114033178ed0bd1a1ff726c Mon Sep 17 00:00:00 2001
From: Dominik Csapak <d.csapak@proxmox.com>
Date: Wed, 17 Nov 2021 15:21:01 +0100
Subject: [PATCH] api: backup: normalize 'dow' format when converting

the old web ui sends the days as seperate parameters, which will
be concatenated by a null-byte in the api, causing it to land it this
way in the jobs.cfg

to fix this, split+join the list to get a well-formed dow list

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 PVE/API2/Backup.pm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm
index 3b1d5299..c4970bc3 100644
--- a/PVE/API2/Backup.pm
+++ b/PVE/API2/Backup.pm
@@ -52,6 +52,7 @@ my $convert_to_schedule = sub {
 
     my $starttime = $job->{starttime};
     my $dow = $job->{dow};
+    $dow = join(',', PVE::Tools::split_list($dow)) if defined($dow);
 
     if (!$dow || $dow eq ALL_DAYS) {
 	return "$starttime";