]> git.proxmox.com Git - pve-manager.git/commitdiff
ext6migrate: fix a long running bug where a logout from the GUI would break the workspace
authorEmmanuel Kasper <e.kasper@proxmox.com>
Mon, 25 Jan 2016 14:19:28 +0000 (15:19 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Tue, 26 Jan 2016 09:54:32 +0000 (10:54 +0100)
If we pass true to the removeAll() , ExtJS6 will try to erase the leafes of a tree,
and then try to to iterate throuch these leaves in onNodeRemove() ( this obviously do not work)

The true parameter does not seem to be needed anyway anymore.
After a call to removeAll, the childNodes[] property of the rootNode is empty

www/manager6/tree/ResourceTree.js

index 48c1da6603026aba4475206fa54c06985dacf72d..711a38616a662e10fd15dec7d54e9da3908ba75d 100644 (file)
@@ -366,7 +366,7 @@ Ext.define('PVE.tree.ResourceTree', {
                pdata.updateCount = 0;
                var rootnode = me.store.getRootNode();
                rootnode.collapse();
-               rootnode.removeAll(true);
+               rootnode.removeAll();
                pdata.dataIndex = {};
                me.getSelectionModel().deselectAll();
            },