From 6ae31fc19bd51dde5fdd1935716f49feac62d89d Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Fri, 6 May 2016 08:48:29 +0200 Subject: [PATCH] fix #979: check always for migrated vm we only checked for grouping attributes for moved vms, but this is only true in server view e.g. in pool view the grouping attributes is pool and the item is not reselected, resulting in a "file not found" error (it looks for the conf on the wrong node) Signed-off-by: Dominik Csapak --- www/manager6/tree/ResourceTree.js | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/www/manager6/tree/ResourceTree.js b/www/manager6/tree/ResourceTree.js index feecf90a..d94a63ce 100644 --- a/www/manager6/tree/ResourceTree.js +++ b/www/manager6/tree/ResourceTree.js @@ -266,6 +266,7 @@ Ext.define('PVE.tree.ResourceTree', { if (item) { // test if any grouping attributes changed // this will also catch migrated nodes + // in server view var i, len; for (i = 0, len = groups.length; i < len; i++) { var attr = groups[i]; @@ -276,6 +277,13 @@ Ext.define('PVE.tree.ResourceTree', { } } + // explicitely check for node, since + // in some views, node is not a grouping + // attribute + if (!moved && item.data.node !== olditem.data.node) { + moved = true; + } + // tree item has been updated if ((item.data.text !== olditem.data.text) || (item.data.running !== olditem.data.running) ||