fix(xo-web/backupNg/new): make the default retention equals 1 (#2872)

See #2711

packages/xo-web/src/xo-app/backup-ng/new/index.js

@@ -371,6 +371,13 @@ export default [
         _,
         { cron, timezone, exportRetention, snapshotRetention }
       ) => async (state, props) => {
+        if (!state.exportMode) {
+          exportRetention = 0
+        }
+        if (!state.snapshotMode) {
+          snapshotRetention = 0
+        }
+
         if (state.editionMode === 'creation') {
           return {
             ...state,

packages/xo-web/src/xo-app/backup-ng/new/new-schedule.js

@@ -43,8 +43,8 @@ export default [
     initialState: ({
       schedule: {
         cron = '0 0 * * *',
-        exportRetention = 0,
+        exportRetention = 1,
-        snapshotRetention = 0,
+        snapshotRetention = 1,
         timezone = moment.tz.guess(),
       },
     }) => ({
@@ -76,14 +76,27 @@ export default [
       }),
     },
     computed: {
-      isScheduleInvalid: ({
+      isScheduleInvalid: ({ retentionNeeded, scheduleNotEdited }) =>
-        cron,
+        retentionNeeded || scheduleNotEdited,
+      retentionNeeded: ({
+        exportMode,
         exportRetention,
+        snapshotMode,
+        snapshotRetention,
+      }) =>
+        !(
+          (exportMode && exportRetention !== 0) ||
+          (snapshotMode && snapshotRetention !== 0)
+        ),
+      scheduleNotEdited: ({
+        cron,
+        editionMode,
+        exportRetention,
+        oldSchedule,
         snapshotRetention,
         timezone,
-        oldSchedule,
       }) =>
-        (snapshotRetention === 0 && exportRetention === 0) ||
+        editionMode !== 'creation' &&
         isEqual(oldSchedule, {
           cron,
           exportRetention,