fix: remove parent_field parameter from calls to rebuild_tree (#39146)
fix: remove parent_field parameter from calls to rebuild_tree The parent_field parameter became redundant with https://github.com/frappe/frappe/pull/24107
This commit is contained in:
parent
e912e9597d
commit
68c0e188e8
@ -74,7 +74,7 @@ def create_charts(
|
|||||||
# after all accounts are already inserted.
|
# after all accounts are already inserted.
|
||||||
frappe.local.flags.ignore_update_nsm = True
|
frappe.local.flags.ignore_update_nsm = True
|
||||||
_import_accounts(chart, None, None, root_account=True)
|
_import_accounts(chart, None, None, root_account=True)
|
||||||
rebuild_tree("Account", "parent_account")
|
rebuild_tree("Account")
|
||||||
frappe.local.flags.ignore_update_nsm = False
|
frappe.local.flags.ignore_update_nsm = False
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@ -35,7 +35,7 @@ def execute():
|
|||||||
# append list of new department for each company
|
# append list of new department for each company
|
||||||
comp_dict[company.name][department.name] = copy_doc.name
|
comp_dict[company.name][department.name] = copy_doc.name
|
||||||
|
|
||||||
rebuild_tree("Department", "parent_department")
|
rebuild_tree("Department")
|
||||||
doctypes = ["Asset", "Employee", "Payroll Entry", "Staffing Plan", "Job Opening"]
|
doctypes = ["Asset", "Employee", "Payroll Entry", "Staffing Plan", "Job Opening"]
|
||||||
|
|
||||||
for d in doctypes:
|
for d in doctypes:
|
||||||
|
|||||||
@ -27,7 +27,7 @@ def execute():
|
|||||||
except frappe.DuplicateEntryError:
|
except frappe.DuplicateEntryError:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
rebuild_tree("Location", "parent_location")
|
rebuild_tree("Location")
|
||||||
|
|
||||||
|
|
||||||
def get_parent_warehouse_name(warehouse):
|
def get_parent_warehouse_name(warehouse):
|
||||||
|
|||||||
@ -4,4 +4,4 @@ from frappe.utils.nestedset import rebuild_tree
|
|||||||
|
|
||||||
def execute():
|
def execute():
|
||||||
frappe.reload_doc("setup", "doctype", "company")
|
frappe.reload_doc("setup", "doctype", "company")
|
||||||
rebuild_tree("Company", "parent_company")
|
rebuild_tree("Company")
|
||||||
|
|||||||
@ -41,4 +41,4 @@ def build_tree():
|
|||||||
}
|
}
|
||||||
).insert(ignore_permissions=True)
|
).insert(ignore_permissions=True)
|
||||||
|
|
||||||
rebuild_tree("Supplier Group", "parent_supplier_group")
|
rebuild_tree("Supplier Group")
|
||||||
|
|||||||
@ -18,4 +18,4 @@ def execute():
|
|||||||
)
|
)
|
||||||
)
|
)
|
||||||
|
|
||||||
rebuild_tree("Department", "parent_department")
|
rebuild_tree("Department")
|
||||||
|
|||||||
@ -249,7 +249,7 @@ class Company(NestedSet):
|
|||||||
if frappe.flags.parent_company_changed:
|
if frappe.flags.parent_company_changed:
|
||||||
from frappe.utils.nestedset import rebuild_tree
|
from frappe.utils.nestedset import rebuild_tree
|
||||||
|
|
||||||
rebuild_tree("Company", "parent_company")
|
rebuild_tree("Company")
|
||||||
|
|
||||||
frappe.clear_cache()
|
frappe.clear_cache()
|
||||||
|
|
||||||
@ -397,7 +397,7 @@ class Company(NestedSet):
|
|||||||
frappe.local.flags.ignore_update_nsm = True
|
frappe.local.flags.ignore_update_nsm = True
|
||||||
make_records(records)
|
make_records(records)
|
||||||
frappe.local.flags.ignore_update_nsm = False
|
frappe.local.flags.ignore_update_nsm = False
|
||||||
rebuild_tree("Department", "parent_department")
|
rebuild_tree("Department")
|
||||||
|
|
||||||
def validate_coa_input(self):
|
def validate_coa_input(self):
|
||||||
if self.create_chart_of_accounts_based_on == "Existing Company":
|
if self.create_chart_of_accounts_based_on == "Existing Company":
|
||||||
|
|||||||
@ -79,7 +79,7 @@ class TestItem(unittest.TestCase):
|
|||||||
group_b.save()
|
group_b.save()
|
||||||
|
|
||||||
def test_rebuild_tree(self):
|
def test_rebuild_tree(self):
|
||||||
rebuild_tree("Item Group", "parent_item_group")
|
rebuild_tree("Item Group")
|
||||||
self.test_basic_tree()
|
self.test_basic_tree()
|
||||||
|
|
||||||
def move_it_back(self):
|
def move_it_back(self):
|
||||||
|
|||||||
@ -59,7 +59,7 @@ def get_warehouse_account(warehouse, warehouse_account=None):
|
|||||||
else:
|
else:
|
||||||
from frappe.utils.nestedset import rebuild_tree
|
from frappe.utils.nestedset import rebuild_tree
|
||||||
|
|
||||||
rebuild_tree("Warehouse", "parent_warehouse")
|
rebuild_tree("Warehouse")
|
||||||
else:
|
else:
|
||||||
account = frappe.db.sql(
|
account = frappe.db.sql(
|
||||||
"""
|
"""
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user