chore: remove debugging statements and fixing failing unit tests
diff --git a/erpnext/controllers/accounts_controller.py b/erpnext/controllers/accounts_controller.py
index 5597b51..2548bdc 100644
--- a/erpnext/controllers/accounts_controller.py
+++ b/erpnext/controllers/accounts_controller.py
@@ -977,26 +977,25 @@
if self.get("doctype") == "Journal Entry":
if args:
for arg in args:
- print(arg)
- if arg.get("difference_amount") != 0 and arg.get("difference_account"):
+ if arg.get("difference_amount", 0) != 0 and arg.get("difference_account"):
journal_entry = frappe.new_doc("Journal Entry")
journal_entry.voucher_type = "Exchange Gain Or Loss"
journal_entry.company = self.company
journal_entry.posting_date = nowdate()
journal_entry.multi_currency = 1
- party_account = arg.account
+ party_account = arg.get("account")
party_account_currency = frappe.get_cached_value(
"Account", party_account, "account_currency"
)
- dr_or_cr = "debit" if arg.difference_amount > 0 else "credit"
+ dr_or_cr = "debit" if arg.get("difference_amount") > 0 else "credit"
if arg.reference_doctype == "Purchase Invoice":
dr_or_cr = "debit" if dr_or_cr == "credit" else "credit"
reverse_dr_or_cr = "debit" if dr_or_cr == "credit" else "credit"
- gain_loss_account = arg.difference_account
+ gain_loss_account = arg.get("difference_account")
if not gain_loss_account:
frappe.throw(
@@ -1014,14 +1013,14 @@
journal_account = frappe._dict(
{
"account": party_account,
- "party_type": arg.party_type,
- "party": arg.party,
+ "party_type": arg.get("party_type"),
+ "party": arg.get("party"),
"account_currency": party_account_currency,
"exchange_rate": 0,
"cost_center": erpnext.get_default_cost_center(self.company),
- "reference_type": arg.against_voucher_type,
- "reference_name": arg.against_voucher,
- "reference_detail_no": arg.idx,
+ "reference_type": arg.get("against_voucher_type"),
+ "reference_name": arg.get("against_voucher"),
+ "reference_detail_no": arg.get("idx"),
dr_or_cr: abs(arg.difference_amount),
dr_or_cr + "_in_account_currency": 0,
}
@@ -1039,8 +1038,8 @@
# "reference_type": self.doctype,
# "reference_name": self.name,
# "reference_detail_no": arg.idx,
- reverse_dr_or_cr + "_in_account_currency": abs(arg.difference_amount),
- reverse_dr_or_cr: abs(arg.difference_amount),
+ reverse_dr_or_cr + "_in_account_currency": abs(arg.get("difference_amount")),
+ reverse_dr_or_cr: abs(arg.get("difference_amount")),
}
)