From 04b077a89ff0c447343726e2a3d045991a136b08 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Wed, 20 Jul 2022 10:55:10 +0530 Subject: [PATCH] fix: set status on submit/cancel --- .../doctype/period_closing_voucher/period_closing_voucher.py | 4 ++-- .../period_closing_voucher/test_period_closing_voucher.py | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/erpnext/accounts/doctype/period_closing_voucher/period_closing_voucher.py b/erpnext/accounts/doctype/period_closing_voucher/period_closing_voucher.py index a76cc18c69e..866a94d04b1 100644 --- a/erpnext/accounts/doctype/period_closing_voucher/period_closing_voucher.py +++ b/erpnext/accounts/doctype/period_closing_voucher/period_closing_voucher.py @@ -19,11 +19,11 @@ class PeriodClosingVoucher(AccountsController): self.validate_posting_date() def on_submit(self): - self.gle_processing_status = "In Progress" + self.db_set("gle_processing_status", "In Progress") self.make_gl_entries() def on_cancel(self): - self.gle_processing_status = "In Progress" + self.db_set("gle_processing_status", "In Progress") self.ignore_linked_doctypes = ("GL Entry", "Stock Ledger Entry") gle_count = frappe.db.count( "GL Entry", diff --git a/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py b/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py index 7999160a5eb..93869ed6c04 100644 --- a/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py +++ b/erpnext/accounts/doctype/period_closing_voucher/test_period_closing_voucher.py @@ -59,7 +59,8 @@ class TestPeriodClosingVoucher(unittest.TestCase): """, (pcv.name), ) - self.assetEqual(pcv.gle_processing_status, "Completed") + pcv.reload() + self.assertEqual(pcv.gle_processing_status, "Completed") self.assertEqual(pcv_gle, expected_gle) def test_cost_center_wise_posting(self):