From 9e87a887cb8a85c1b9148598850f1b42b988ae84 Mon Sep 17 00:00:00 2001
From: Florent BEAUCHAMP <florent.beauchamp@vates.fr>
Date: Tue, 8 Nov 2022 10:06:04 +0100
Subject: [PATCH] fix(xo-web/backup): cleanup settings correctly when
 deselecting health check (#6515)

Fix #6501
---
 CHANGELOG.unreleased.md                               | 1 +
 packages/xo-web/src/xo-app/backup/new/new-schedule.js | 1 +
 2 files changed, 2 insertions(+)

diff --git a/CHANGELOG.unreleased.md b/CHANGELOG.unreleased.md
index e0257ee14..6f81d52a8 100644
--- a/CHANGELOG.unreleased.md
+++ b/CHANGELOG.unreleased.md
@@ -20,6 +20,7 @@
 - [Pool] Add tooltip on "no XCP-ng Pro support" warning icon (PR [#6505](https://github.com/vatesfr/xen-orchestra/pull/6505))
 - [Backup] Respect HTTP proxy setting when connecting to XCP-ng/XenServer pools
 - [Dashboard/Health] Fix `an error has occurred` in case of unknown default SR (PR [#6508](https://github.com/vatesfr/xen-orchestra/pull/6508))
+- [Backup] Really disable Healthcheck when unchecking settings [#6501](https://github.com/vatesfr/xen-orchestra/issues/6501) (PR [#6515](https://github.com/vatesfr/xen-orchestra/pull/6515))
 
 ### Packages to release
 
diff --git a/packages/xo-web/src/xo-app/backup/new/new-schedule.js b/packages/xo-web/src/xo-app/backup/new/new-schedule.js
index 68b8e2603..658a07f4c 100644
--- a/packages/xo-web/src/xo-app/backup/new/new-schedule.js
+++ b/packages/xo-web/src/xo-app/backup/new/new-schedule.js
@@ -80,6 +80,7 @@ const New = decorate([
       toggleHealthCheck({ setSchedule }, { target: { checked } }) {
         setSchedule({
           healthCheckVmsWithTags: checked ? [] : undefined,
+          healthCheckSr: checked ? this.state.healthCheckSr : undefined,
         })
       },
       setHealthCheckSr({ setSchedule }, sr) {