[pve-devel] applied: [PATCH manager] ui: vm/HDEdit: fix backup checkbox default
Thomas Lamprecht
t.lamprecht at proxmox.com
Mon Dec 9 15:45:55 CET 2019
The recent change transformed this checkbox from a negative "No
Backup" to a positive "Backup" did not accounted for changing the
default fallback for the parseBoolean call.
As this would had suggested that disk are backed-up which aren't
backup, it could lead to pretty devastating results...
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
www/manager6/qemu/HDEdit.js | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
index 8d469f6f..40055442 100644
--- a/www/manager6/qemu/HDEdit.js
+++ b/www/manager6/qemu/HDEdit.js
@@ -130,7 +130,7 @@ Ext.define('PVE.qemu.HDInputPanel', {
}
values.hdimage = drive.file;
- values.backup = PVE.Parser.parseBoolean(drive.backup, 1);
+ values.backup = PVE.Parser.parseBoolean(drive.backup, 0);
values.noreplicate = !PVE.Parser.parseBoolean(drive.replicate, 1);
values.diskformat = drive.format || 'raw';
values.cache = drive.cache || '__default__';
@@ -296,7 +296,6 @@ Ext.define('PVE.qemu.HDInputPanel', {
tag: 'div',
'data-qtip': gettext('Include volume in backup job'),
},
- uncheckedValue: 'no',
labelWidth: labelWidth,
name: 'backup',
bind: {
--
2.20.1
More information about the pve-devel
mailing list