mirror of
https://github.com/frappe/erpnext.git
synced 2026-06-03 20:29:09 +00:00
Merge pull request #49805 from frappe/mergify/bp/version-15-hotfix/pr-49639
fix(Credit-limit): consider current voucher for credit limit validation (backport #49639)
This commit is contained in:
@@ -189,8 +189,8 @@ class JournalEntry(AccountsController):
|
|||||||
|
|
||||||
def on_submit(self):
|
def on_submit(self):
|
||||||
self.validate_cheque_info()
|
self.validate_cheque_info()
|
||||||
self.check_credit_limit()
|
|
||||||
self.make_gl_entries()
|
self.make_gl_entries()
|
||||||
|
self.check_credit_limit()
|
||||||
self.update_asset_value()
|
self.update_asset_value()
|
||||||
self.update_inter_company_jv()
|
self.update_inter_company_jv()
|
||||||
self.update_invoice_discounting()
|
self.update_invoice_discounting()
|
||||||
|
|||||||
@@ -11,6 +11,7 @@ from frappe.utils import flt, nowdate
|
|||||||
from erpnext.accounts.doctype.account.test_account import get_inventory_account
|
from erpnext.accounts.doctype.account.test_account import get_inventory_account
|
||||||
from erpnext.accounts.doctype.journal_entry.journal_entry import StockAccountInvalidTransaction
|
from erpnext.accounts.doctype.journal_entry.journal_entry import StockAccountInvalidTransaction
|
||||||
from erpnext.exceptions import InvalidAccountCurrency
|
from erpnext.exceptions import InvalidAccountCurrency
|
||||||
|
from erpnext.selling.doctype.customer.test_customer import make_customer, set_credit_limit
|
||||||
|
|
||||||
|
|
||||||
class TestJournalEntry(unittest.TestCase):
|
class TestJournalEntry(unittest.TestCase):
|
||||||
@@ -592,6 +593,15 @@ class TestJournalEntry(unittest.TestCase):
|
|||||||
|
|
||||||
self.assertEqual(jv.pay_to_recd_from, "_Test Receiver 2")
|
self.assertEqual(jv.pay_to_recd_from, "_Test Receiver 2")
|
||||||
|
|
||||||
|
def test_credit_limit_for_customer(self):
|
||||||
|
customer = make_customer("_Test New Customer")
|
||||||
|
set_credit_limit("_Test New Customer", "_Test Company", 50)
|
||||||
|
jv = make_journal_entry(account1="Debtors - _TC", account2="_Test Cash - _TC", amount=100, save=False)
|
||||||
|
jv.accounts[0].party_type = "Customer"
|
||||||
|
jv.accounts[0].party = customer
|
||||||
|
jv.save()
|
||||||
|
self.assertRaises(frappe.ValidationError, jv.submit)
|
||||||
|
|
||||||
|
|
||||||
def make_journal_entry(
|
def make_journal_entry(
|
||||||
account1,
|
account1,
|
||||||
|
|||||||
@@ -443,3 +443,14 @@ def create_internal_customer(customer_name=None, represents_company=None, allowe
|
|||||||
customer_name = frappe.db.get_value("Customer", customer_name)
|
customer_name = frappe.db.get_value("Customer", customer_name)
|
||||||
|
|
||||||
return customer_name
|
return customer_name
|
||||||
|
|
||||||
|
|
||||||
|
def make_customer(customer_name):
|
||||||
|
if not frappe.db.exists("Customer", customer_name):
|
||||||
|
customer = frappe.new_doc("Customer")
|
||||||
|
customer.customer_name = customer_name
|
||||||
|
customer.customer_type = "Individual"
|
||||||
|
customer.insert()
|
||||||
|
return customer.name
|
||||||
|
else:
|
||||||
|
return customer_name
|
||||||
|
|||||||
Reference in New Issue
Block a user