vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: new and add child button for empty tree doctype

Browse source 
- The second check in treeview.js is useless
- If the node being created is root-node(i.e. doctype(name) is parent
  then don't set parent)
This commit is contained in:
Rutwik Hiwalkar 2024-05-03 23:27:21 +05:30
parent 8e76a94a62
commit 3b89c3f664
2 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/desk/treeview.py
View file

@ -82,6 +82,8 @@ def make_tree_args(**kwarg):
if kwarg["is_root"] == "true":
kwarg["is_root"] = True
kwarg.update({parent_field: kwarg.get("parent") or kwarg.get(parent_field)})
parent = kwarg.get("parent") or kwarg.get(parent_field)
if doctype != parent:
kwarg.update({parent_field: parent})
return frappe._dict(kwarg)

8
frappe/public/js/frappe/views/treeview.js
View file

@ -180,12 +180,12 @@ frappe.views.TreeView = class TreeView {
args: me.args,
callback: function (r) {
if (r.message) {
if (r.message.length > 1) {
me.root_label = me.doctype;
me.root_value = "";
} else {
if (r.message.length == 1) {
me.root_label = r.message[0]["value"];
me.root_value = me.root_label;
} else {
me.root_label = me.doctype;
me.root_value = "";
}
me.make_tree();
}