From bb88b420c148b667be463e4c537f4f6c44ee2bff Mon Sep 17 00:00:00 2001
From: badrAZ <azizbibadr@gmail.com>
Date: Wed, 12 Sep 2018 13:53:37 +0200
Subject: [PATCH] chore(xo-web/backup-ng/new): make "setSchedule" support
 multiple params (#3404)

---
 .../src/xo-app/backup-ng/new/new-schedule.js  | 38 ++++++++-----------
 1 file changed, 16 insertions(+), 22 deletions(-)

diff --git a/packages/xo-web/src/xo-app/backup-ng/new/new-schedule.js b/packages/xo-web/src/xo-app/backup-ng/new/new-schedule.js
index 269a1f0ed..1485f9b19 100644
--- a/packages/xo-web/src/xo-app/backup-ng/new/new-schedule.js
+++ b/packages/xo-web/src/xo-app/backup-ng/new/new-schedule.js
@@ -27,48 +27,42 @@ export default [
       schedule: undefined,
     }),
     effects: {
-      setSchedule: (_, { name, value }) => ({
+      setSchedule: (_, params) => ({
         tmpSchedule = DEFAULT_SCHEDULE,
         schedule = tmpSchedule,
       }) => ({
         schedule: {
           ...schedule,
-          [name]: value,
+          ...params,
         },
       }),
-      setExportRetention: ({ setSchedule }, value) => () => {
+      setExportRetention: ({ setSchedule }, exportRetention) => () => {
         setSchedule({
-          name: 'exportRetention',
-          value,
+          exportRetention,
         })
       },
-      setCopyRetention: ({ setSchedule }, value) => () => {
+      setCopyRetention: ({ setSchedule }, copyRetention) => () => {
         setSchedule({
-          name: 'copyRetention',
-          value,
+          copyRetention,
         })
       },
-      setSnapshotRetention: ({ setSchedule }, value) => () => {
+      setSnapshotRetention: ({ setSchedule }, snapshotRetention) => () => {
         setSchedule({
-          name: 'snapshotRetention',
-          value,
+          snapshotRetention,
         })
       },
-      setCronTimezone: ({ setSchedule }, { cronPattern, timezone }) => () => {
+      setCronTimezone: (
+        { setSchedule },
+        { cronPattern: cron, timezone }
+      ) => () => {
         setSchedule({
-          name: 'cron',
-          value: cronPattern,
-        }).then(() =>
-          setSchedule({
-            name: 'timezone',
-            value: timezone,
-          })
-        )
+          cron,
+          timezone,
+        })
       },
       setName: ({ setSchedule }, { target: { value } }) => () => {
         setSchedule({
-          name: 'name',
-          value: value.trim() === '' ? null : value,
+          name: value.trim() === '' ? null : value,
         })
       },
     },