Merge pull request #33502 from ruthra-kumar/toggle_account_balances_in_coa

feat: Toggle display of Account Balance in Chart of Accounts

erpnext/accounts/doctype/account/account_tree.js

@@ -56,36 +56,41 @@ frappe.treeview_settings["Account"] = {
 			accounts = nodes;
 		}
 
-		const get_balances = frappe.call({
+		frappe.db.get_single_value("Accounts Settings", "show_balance_in_coa").then((value) => {
-			method: 'erpnext.accounts.utils.get_account_balances',
+			if(value) {
-			args: {
-				accounts: accounts,
-				company: cur_tree.args.company
-			},
-		});
 
-		get_balances.then(r => {
+				const get_balances = frappe.call({
-			if (!r.message || r.message.length == 0) return;
+					method: 'erpnext.accounts.utils.get_account_balances',
+					args: {
+						accounts: accounts,
+						company: cur_tree.args.company
+					},
+				});
 
-			for (let account of r.message) {
+				get_balances.then(r => {
+					if (!r.message || r.message.length == 0) return;
 
-				const node = cur_tree.nodes && cur_tree.nodes[account.value];
+					for (let account of r.message) {
-				if (!node || node.is_root) continue;
 
-				// show Dr if positive since balance is calculated as debit - credit else show Cr
+						const node = cur_tree.nodes && cur_tree.nodes[account.value];
-				const balance = account.balance_in_account_currency || account.balance;
+						if (!node || node.is_root) continue;
-				const dr_or_cr = balance > 0 ? "Dr": "Cr";
-				const format = (value, currency) => format_currency(Math.abs(value), currency);
 
-				if (account.balance!==undefined) {
+						// show Dr if positive since balance is calculated as debit - credit else show Cr
-					node.parent && node.parent.find('.balance-area').remove();
+						const balance = account.balance_in_account_currency || account.balance;
-					$('<span class="balance-area pull-right">'
+						const dr_or_cr = balance > 0 ? "Dr": "Cr";
-						+ (account.balance_in_account_currency ?
+						const format = (value, currency) => format_currency(Math.abs(value), currency);
-							(format(account.balance_in_account_currency, account.account_currency) + " / ") : "")
+
-						+ format(account.balance, account.company_currency)
+						if (account.balance!==undefined) {
-						+ " " + dr_or_cr
+							node.parent && node.parent.find('.balance-area').remove();
-						+ '</span>').insertBefore(node.$ul);
+							$('<span class="balance-area pull-right">'
-				}
+							  + (account.balance_in_account_currency ?
+							     (format(account.balance_in_account_currency, account.account_currency) + " / ") : "")
+							  + format(account.balance, account.company_currency)
+							  + " " + dr_or_cr
+							  + '</span>').insertBefore(node.$ul);
+						}
+					}
+				});
 			}
 		});
 	},

erpnext/accounts/doctype/accounts_settings/accounts_settings.json

@@ -56,7 +56,9 @@
   "acc_frozen_upto",
   "column_break_25",
   "frozen_accounts_modifier",
-  "report_settings_sb"
+  "report_settings_sb",
+  "tab_break_dpet",
+  "show_balance_in_coa"
  ],
  "fields": [
   {
@@ -347,6 +349,17 @@
    "fieldname": "allow_multi_currency_invoices_against_single_party_account",
    "fieldtype": "Check",
    "label": "Allow multi-currency invoices against single party account "
+  },
+  {
+   "fieldname": "tab_break_dpet",
+   "fieldtype": "Tab Break",
+   "label": "Chart Of Accounts"
+  },
+  {
+   "default": "1",
+   "fieldname": "show_balance_in_coa",
+   "fieldtype": "Check",
+   "label": "Show Balances in Chart Of Accounts"
   }
  ],
  "icon": "icon-cog",
@@ -354,7 +367,7 @@
  "index_web_pages_for_search": 1,
  "issingle": 1,
  "links": [],
- "modified": "2022-11-27 21:49:52.538655",
+ "modified": "2023-01-02 12:07:42.434214",
  "modified_by": "Administrator",
  "module": "Accounts",
  "name": "Accounts Settings",