[pve-devel] [PATCH manager] fix doubleclick on vm with spice console

Dominik Csapak d.csapak at proxmox.com
Mon Jul 30 11:37:32 CEST 2018


allowSpice needs to be a real boolean, because of a
'=== true' check to open a spice console if allowed, so make it one here

this fixes the issue that a doubleclick on a tree item only opened
the novnc console even if spice is the default and the vm is spice
enabled

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 www/manager6/Utils.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index 21153ece..c062b591 100644
--- a/www/manager6/Utils.js
+++ b/www/manager6/Utils.js
@@ -880,7 +880,7 @@ Ext.define('PVE.Utils', { utilities: {
 		    Ext.Msg.alert('Error', response.htmlStatus);
 		},
 		success: function(response, opts) {
-		    var allowSpice = response.result.data.spice;
+		    var allowSpice = !!response.result.data.spice;
 		    PVE.Utils.openDefaultConsoleWindow(allowSpice, 'kvm', vmid, nodename, vmname);
 		}
 	    });
-- 
2.11.0





More information about the pve-devel mailing list