Credit Note: Set total amount and print format fix
diff --git a/erpnext/accounts/doctype/journal_voucher/journal_voucher.py b/erpnext/accounts/doctype/journal_voucher/journal_voucher.py
index f9f6df1..c6299ed 100644
--- a/erpnext/accounts/doctype/journal_voucher/journal_voucher.py
+++ b/erpnext/accounts/doctype/journal_voucher/journal_voucher.py
@@ -289,26 +289,25 @@
def set_print_format_fields(self):
for d in self.get('entries'):
- result = frappe.db.get_value("Account", d.account,
- ["account_type", "master_type"])
+ acc = frappe.db.get_value("Account", d.account, ["account_type", "master_type"], as_dict=1)
- if not result:
- continue
+ if not acc: continue
- account_type, master_type = result
-
- if master_type in ['Supplier', 'Customer']:
+ if acc.master_type in ['Supplier', 'Customer']:
if not self.pay_to_recd_from:
- self.pay_to_recd_from = frappe.db.get_value(master_type,
- ' - '.join(d.account.split(' - ')[:-1]),
- master_type == 'Customer' and 'customer_name' or 'supplier_name')
+ self.pay_to_recd_from = frappe.db.get_value(acc.master_type, ' - '.join(d.account.split(' - ')[:-1]),
+ acc.master_type == 'Customer' and 'customer_name' or 'supplier_name')
+ if self.voucher_type in ["Credit Note", "Debit Note"]:
+ self.set_total_amount(d.debit or d.credit)
- if account_type in ['Bank', 'Cash']:
- company_currency = get_company_currency(self.company)
- amt = flt(d.debit) and d.debit or d.credit
- self.total_amount = fmt_money(amt, currency=company_currency)
- from frappe.utils import money_in_words
- self.total_amount_in_words = money_in_words(amt, company_currency)
+ if acc.account_type in ['Bank', 'Cash']:
+ self.set_total_amount(d.debit or d.credit)
+
+ def set_total_amount(self, amt):
+ company_currency = get_company_currency(self.company)
+ self.total_amount = fmt_money(amt, currency=company_currency)
+ from frappe.utils import money_in_words
+ self.total_amount_in_words = money_in_words(amt, company_currency)
def check_credit_days(self):
date_diff = 0
diff --git a/erpnext/accounts/print_format/credit_note/credit_note.json b/erpnext/accounts/print_format/credit_note/credit_note.json
index 31507b4..ac0de75 100644
--- a/erpnext/accounts/print_format/credit_note/credit_note.json
+++ b/erpnext/accounts/print_format/credit_note/credit_note.json
@@ -4,9 +4,9 @@
"doc_type": "Journal Voucher",
"docstatus": 0,
"doctype": "Print Format",
- "html": "{%- from \"templates/print_formats/standard_macros.html\" import add_header -%}\n\n<div class=\"page-break\">\n {%- if not doc.get(\"print_heading\") and not doc.get(\"select_print_heading\") \n and doc.set(\"select_print_heading\", _(\"Credit Note\")) -%}{%- endif -%}\n {{ add_header(0, 1, doc, letter_head, no_letterhead) }}\n\n {%- for label, value in (\n (_(\"Credit To\"), doc.pay_to_recd_from),\n (_(\"Date\"), frappe.utils.formatdate(doc.voucher_date)),\n (_(\"Amount\"), \"<strong>\" + doc.total_amount + \"</strong><br>\" + (doc.total_amount_in_words or \"\") + \"<br>\"),\n (_(\"Remarks\"), doc.remark)\n ) -%}\n\n <div class=\"row\">\n <div class=\"col-sm-3\"><label class=\"text-right\">{{ label }}</label></div>\n <div class=\"col-sm-9\">{{ value }}</div>\n </div>\n\n {%- endfor -%}\n\n <hr>\n <br>\n <p class=\"strong\">\n {{ _(\"For\") }} {{ doc.company }},<br>\n <br>\n <br>\n <br>\n {{ _(\"Authorized Signatory\") }}\n </p>\n</div>\n\n\n",
+ "html": "{%- from \"templates/print_formats/standard_macros.html\" import add_header -%}\n\n<div class=\"page-break\">\n {%- if not doc.get(\"print_heading\") and not doc.get(\"select_print_heading\") \n and doc.set(\"select_print_heading\", _(\"Credit Note\")) -%}{%- endif -%}\n {{ add_header(0, 1, doc, letter_head, no_letterhead) }}\n\n {%- for label, value in (\n (_(\"Credit To\"), doc.pay_to_recd_from),\n (_(\"Date\"), frappe.utils.formatdate(doc.voucher_date)),\n (_(\"Amount\"), \"<strong>\" + frappe.utils.cstr(doc.total_amount) + \"</strong><br>\" + (doc.total_amount_in_words or \"\") + \"<br>\"),\n (_(\"Remarks\"), doc.remark)\n ) -%}\n\n <div class=\"row\">\n <div class=\"col-sm-3\"><label class=\"text-right\">{{ label }}</label></div>\n <div class=\"col-sm-9\">{{ value }}</div>\n </div>\n\n {%- endfor -%}\n\n <hr>\n <br>\n <p class=\"strong\">\n {{ _(\"For\") }} {{ doc.company }},<br>\n <br>\n <br>\n <br>\n {{ _(\"Authorized Signatory\") }}\n </p>\n</div>\n\n\n",
"idx": 2,
- "modified": "2014-08-29 13:20:15.789533",
+ "modified": "2014-10-17 17:20:02.740340",
"modified_by": "Administrator",
"module": "Accounts",
"name": "Credit Note",