Yaltik/account-financial-tools
2
0
Fork 0
Code Pull Requests Activity

[FIX] account_move_name_sequence: call flush before _fetch_duplicate_supplier_reference method

Browse Source 
same issue https://github.com/OCA/account-financial-tools/issues/1501
this fix not working for v16 https://github.com/OCA/account-financial-tools/pull/1514
This commit is contained in:
Rodrigo 2022-12-02 12:57:26 +01:00
parent 346eb9d085
commit ab8e59d69d
2 changed files with 102 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
account_move_name_sequence/models/account_move.py
View File

@ -64,3 +64,9 @@ class AccountMove(models.Model):
if not invoices_other_sequences and invoices_no_gap_sequences: if not invoices_other_sequences and invoices_no_gap_sequences:
return False return False
return super(AccountMove, invoices_other_sequences)._is_end_of_seq_chain() return super(AccountMove, invoices_other_sequences)._is_end_of_seq_chain()
def _fetch_duplicate_supplier_reference(self, only_posted=False):
moves = self.filtered(lambda m: m.is_purchase_document() and m.ref)
if moves:
self.flush_model(["name", "journal_id", "move_type", "state"])
return super()._fetch_duplicate_supplier_reference(only_posted=only_posted)

96
account_move_name_sequence/tests/test_account_move_name_seq.py
View File

@ -138,6 +138,102 @@ class TestAccountMoveNameSequence(TransactionCase):
move.action_post() move.action_post()
self.assertEqual(move.name, "TEST-2022-07-0001") self.assertEqual(move.name, "TEST-2022-07-0001")
def test_in_invoice_and_refund(self):
in_invoice = self.env["account.move"].create(
{
"journal_id": self.purchase_journal.id,
"invoice_date": self.date,
"partner_id": self.env.ref("base.res_partner_3").id,
"move_type": "in_invoice",
"invoice_line_ids": [
(
0,
0,
{
"account_id": self.account1.id,
"price_unit": 42.0,
"quantity": 12,
},
),
(
0,
0,
{
"account_id": self.account1.id,
"price_unit": 48.0,
"quantity": 10,
},
),
],
}
)
self.assertEqual(in_invoice.name, "/")
in_invoice.action_post()
move_reversal = (
self.env["account.move.reversal"]
.with_context(active_model="account.move", active_ids=in_invoice.ids)
.create(
{
"journal_id": in_invoice.journal_id.id,
"reason": "no reason",
"refund_method": "cancel",
}
)
)
reversal = move_reversal.reverse_moves()
reversed_move = self.env["account.move"].browse(reversal["res_id"])
self.assertTrue(reversed_move)
self.assertEqual(reversed_move.state, "posted")
in_invoice = in_invoice.copy(
{
"invoice_date": self.date,
}
)
in_invoice.action_post()
move_reversal = (
self.env["account.move.reversal"]
.with_context(active_model="account.move", active_ids=in_invoice.ids)
.create(
{
"journal_id": in_invoice.journal_id.id,
"reason": "no reason",
"refund_method": "modify",
}
)
)
reversal = move_reversal.reverse_moves()
draft_invoice = self.env["account.move"].browse(reversal["res_id"])
self.assertTrue(draft_invoice)
self.assertEqual(draft_invoice.state, "draft")
self.assertEqual(draft_invoice.move_type, "in_invoice")
in_invoice = in_invoice.copy(
{
"invoice_date": self.date,
}
)
in_invoice.action_post()
move_reversal = (
self.env["account.move.reversal"]
.with_context(active_model="account.move", active_ids=in_invoice.ids)
.create(
{
"journal_id": in_invoice.journal_id.id,
"reason": "no reason",
"refund_method": "refund",
}
)
)
reversal = move_reversal.reverse_moves()
draft_reversed_move = self.env["account.move"].browse(reversal["res_id"])
self.assertTrue(draft_reversed_move)
self.assertEqual(draft_reversed_move.state, "draft")
self.assertEqual(draft_reversed_move.move_type, "in_refund")
def test_in_refund(self): def test_in_refund(self):
in_refund_invoice = self.env["account.move"].create( in_refund_invoice = self.env["account.move"].create(
{ {