From 08ea710c8f42d58b31aff0d63781c4dff8599cf5 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Wed, 17 May 2017 19:43:12 +0530 Subject: [PATCH] Minor fix in creating chart of accounts based on existing (#8890) --- .../doctype/account/chart_of_accounts/chart_of_accounts.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/erpnext/accounts/doctype/account/chart_of_accounts/chart_of_accounts.py b/erpnext/accounts/doctype/account/chart_of_accounts/chart_of_accounts.py index c7d1291521..a5c551fc8c 100644 --- a/erpnext/accounts/doctype/account/chart_of_accounts/chart_of_accounts.py +++ b/erpnext/accounts/doctype/account/chart_of_accounts/chart_of_accounts.py @@ -125,13 +125,14 @@ def get_account_tree_from_existing_company(existing_company): account_tree = {} # fill in tree starting with root accounts (those with no parent) - build_account_tree(account_tree, None, all_accounts) + if all_accounts: + build_account_tree(account_tree, None, all_accounts) return account_tree def build_account_tree(tree, parent, all_accounts): # find children - parent_account = parent.name if parent else None - children = [acc for acc in all_accounts if acc.parent_account == parent_account] + parent_account = parent.name if parent else "" + children = [acc for acc in all_accounts if cstr(acc.parent_account) == parent_account] # if no children, but a group account if not children and parent.is_group: