Skip to content

Commit

Permalink
fix: Use get_list instead of get_all to avoid perm issues
Browse files Browse the repository at this point in the history
  • Loading branch information
deepeshgarg007 committed Oct 5, 2021
1 parent 0660d6e commit 9507b2d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions erpnext/setup/setup_wizard/operations/taxes_setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ def get_or_create_account(company_name, account):
default_root_type = 'Liability'
root_type = account.get('root_type', default_root_type)

existing_accounts = frappe.get_list('Account',
existing_accounts = frappe.get_all('Account',
filters={
'company': company_name,
'root_type': root_type
Expand Down Expand Up @@ -257,7 +257,7 @@ def get_or_create_tax_group(company_name, root_type):

# Create a new group account named 'Duties and Taxes' or 'Tax Assets' just
# below the root account
root_account = frappe.get_list('Account', {
root_account = frappe.get_all('Account', {
'is_group': 1,
'root_type': root_type,
'company': company_name,
Expand Down

0 comments on commit 9507b2d

Please sign in to comment.