From e089d6bafbad0dba1c2f31dfbf92e2ec71acbd88 Mon Sep 17 00:00:00 2001
From: Ankush Menat <ankush@frappe.io>
Date: Mon, 13 Dec 2021 22:28:38 +0530
Subject: [PATCH] fix: adding a new child expands all childs

The refresh method is causing deep reload of parent node. Doesn't seem
like it's of any use here.

ref: https://github.com/frappe/frappe/pull/4805

This fix had side effect of expanding all parent node recursively.
---
 frappe/public/js/frappe/ui/tree.js        | 1 -
 frappe/public/js/frappe/views/treeview.js | 1 +
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/frappe/public/js/frappe/ui/tree.js b/frappe/public/js/frappe/ui/tree.js
index 4b11b092eb..c32d92aa32 100644
--- a/frappe/public/js/frappe/ui/tree.js
+++ b/frappe/public/js/frappe/ui/tree.js
@@ -299,7 +299,6 @@ frappe.ui.Tree = class {
 				.appendTo($toolbar);
 			$link.on('click', () => {
 				obj.click(node);
-				this.refresh();
 			});
 		});
 
diff --git a/frappe/public/js/frappe/views/treeview.js b/frappe/public/js/frappe/views/treeview.js
index 30da212f0a..44bbaaeeaf 100644
--- a/frappe/public/js/frappe/views/treeview.js
+++ b/frappe/public/js/frappe/views/treeview.js
@@ -242,6 +242,7 @@ frappe.views.TreeView = class TreeView {
 					frappe.model.rename_doc(me.doctype, node.label, function(new_name) {
 						node.$tree_link.find('a').text(new_name);
 						node.label = new_name;
+						me.tree.refresh();
 					});
 				},
 				btnClass: "hidden-xs"