[pve-devel] [PATCH v5 manager 7/7] ui: backup window: also reset removal fields when defaults API call fails

Fabian Ebner f.ebner at proxmox.com
Fri May 7 08:56:52 CEST 2021


to avoid showing settings for the wrong storage.

Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---

Sorry for the delayed fix-up/follow-up to the series.

Could also be squashed into patch #6.

 www/manager6/window/Backup.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/www/manager6/window/Backup.js b/www/manager6/window/Backup.js
index 7048c366..77be6caf 100644
--- a/www/manager6/window/Backup.js
+++ b/www/manager6/window/Backup.js
@@ -141,6 +141,11 @@ Ext.define('PVE.window.Backup', {
 			},
 			failure: function(response, opts) {
 			    initialDefaults = true;
+
+			    removeCheckbox.setHidden(true);
+			    removeCheckbox.setValue(false);
+			    pruneSettings.forEach(field => field.reset());
+
 			    Ext.Msg.alert(gettext('Error'), response.htmlStatus);
 			},
 		    });
-- 
2.20.1





More information about the pve-devel mailing list