From ab9f0fe1b1ca576720b44d57c099bd894670f142 Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Mon, 30 Jul 2018 11:37:32 +0200 Subject: [PATCH] fix doubleclick on vm with spice console 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 --- 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); } });