Merge pull request #4413 from nabinhait/bank_transfer_exchange_rate
Bank transfer exchange rate and account balance in COA
This commit is contained in:
commit
39e603a154
@ -368,10 +368,6 @@ frappe.ui.form.on("Journal Entry Account", {
|
|||||||
|
|
||||||
credit: function(frm, dt, dn) {
|
credit: function(frm, dt, dn) {
|
||||||
cur_frm.cscript.update_totals(frm.doc);
|
cur_frm.cscript.update_totals(frm.doc);
|
||||||
},
|
|
||||||
|
|
||||||
exchange_rate: function(frm, cdt, cdn) {
|
|
||||||
erpnext.journal_entry.set_debit_credit_in_company_currency(frm, cdt, cdn);
|
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
|
|
||||||
@ -418,7 +414,7 @@ $.extend(erpnext.journal_entry, {
|
|||||||
|
|
||||||
if(row.account_currency == company_currency || !frm.doc.multi_currency) {
|
if(row.account_currency == company_currency || !frm.doc.multi_currency) {
|
||||||
frappe.model.set_value(cdt, cdn, "exchange_rate", 1);
|
frappe.model.set_value(cdt, cdn, "exchange_rate", 1);
|
||||||
} else if (!row.exchange_rate || row.account_type == "Bank") {
|
} else if (!row.exchange_rate || row.exchange_rate == 1 || row.account_type == "Bank") {
|
||||||
frappe.call({
|
frappe.call({
|
||||||
method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_exchange_rate",
|
method: "erpnext.accounts.doctype.journal_entry.journal_entry.get_exchange_rate",
|
||||||
args: {
|
args: {
|
||||||
|
@ -295,7 +295,7 @@ class JournalEntry(AccountsController):
|
|||||||
for d in self.get("accounts"):
|
for d in self.get("accounts"):
|
||||||
if d.account_currency == self.company_currency:
|
if d.account_currency == self.company_currency:
|
||||||
d.exchange_rate = 1
|
d.exchange_rate = 1
|
||||||
elif not d.exchange_rate or d.account_type=="Bank" or \
|
elif not d.exchange_rate or d.exchange_rate == 1 or \
|
||||||
(d.reference_type in ("Sales Invoice", "Purchase Invoice") and d.reference_name):
|
(d.reference_type in ("Sales Invoice", "Purchase Invoice") and d.reference_name):
|
||||||
d.exchange_rate = get_exchange_rate(d.account, d.account_currency, self.company,
|
d.exchange_rate = get_exchange_rate(d.account, d.account_currency, self.company,
|
||||||
d.reference_type, d.reference_name, d.debit, d.credit, d.exchange_rate)
|
d.reference_type, d.reference_name, d.debit, d.credit, d.exchange_rate)
|
||||||
|
@ -166,7 +166,10 @@ erpnext.AccountsChart = Class.extend({
|
|||||||
var dr_or_cr = node.data.balance < 0 ? "Cr" : "Dr";
|
var dr_or_cr = node.data.balance < 0 ? "Cr" : "Dr";
|
||||||
if (me.ctype == 'Account' && node.data && node.data.balance!==undefined) {
|
if (me.ctype == 'Account' && node.data && node.data.balance!==undefined) {
|
||||||
$('<span class="balance-area pull-right text-muted small">'
|
$('<span class="balance-area pull-right text-muted small">'
|
||||||
+ format_currency(Math.abs(node.data.balance), node.data.account_currency)
|
+ (node.data.balance_in_account_currency ?
|
||||||
|
(format_currency(Math.abs(node.data.balance_in_account_currency),
|
||||||
|
node.data.account_currency) + " / ") : "")
|
||||||
|
+ format_currency(Math.abs(node.data.balance), node.data.company_currency)
|
||||||
+ " " + dr_or_cr
|
+ " " + dr_or_cr
|
||||||
+ '</span>').insertBefore(node.$ul);
|
+ '</span>').insertBefore(node.$ul);
|
||||||
}
|
}
|
||||||
|
@ -44,7 +44,12 @@ def get_children():
|
|||||||
args['parent'], as_dict=1)
|
args['parent'], as_dict=1)
|
||||||
|
|
||||||
if ctype == 'Account':
|
if ctype == 'Account':
|
||||||
|
company_currency = frappe.db.get_value("Company", company, "default_currency")
|
||||||
for each in acc:
|
for each in acc:
|
||||||
each["balance"] = flt(get_balance_on(each.get("value")))
|
each["company_currency"] = company_currency
|
||||||
|
each["balance"] = flt(get_balance_on(each.get("value"), in_account_currency=False))
|
||||||
|
|
||||||
|
if each.account_currency != company_currency:
|
||||||
|
each["balance_in_account_currency"] = flt(get_balance_on(each.get("value")))
|
||||||
|
|
||||||
return acc
|
return acc
|
||||||
|
Loading…
x
Reference in New Issue
Block a user