diff --git a/account_move_post_date_user/__manifest__.py b/account_move_post_date_user/__manifest__.py index 3dd3802fc..3aaccb440 100644 --- a/account_move_post_date_user/__manifest__.py +++ b/account_move_post_date_user/__manifest__.py @@ -3,12 +3,12 @@ { "name": "Account Move Post Date User", "summary": "Trace journal entry posting date and user.", - "version": "16.0.1.0.0", + "version": "13.0.1.0.0", "category": "Accounting", "website": "https://github.com/OCA/account-financial-tools", "author": "ForgeFlow S.L., Odoo Community Association (OCA)", "maintainers": ["JordiMForgeFlow"], "license": "LGPL-3", - "depends": ["account",], + "depends": ["account"], "data": ["views/account_move.xml"], } diff --git a/account_move_post_date_user/models/account_move.py b/account_move_post_date_user/models/account_move.py index c72f418a4..ed5aa6233 100644 --- a/account_move_post_date_user/models/account_move.py +++ b/account_move_post_date_user/models/account_move.py @@ -7,20 +7,19 @@ from odoo import fields, models class AccountMove(models.Model): _inherit = "account.move" - def _post(self, soft=True): - res = super()._post(soft) - if res: - res.write( - { - "last_post_date": fields.Datetime.now(), - "last_post_uid": self.env.user.id, - } - ) - return res - last_post_date = fields.Datetime( string="Last Posted on", readonly=True, tracking=True ) last_post_uid = fields.Many2one( "res.users", string="Last Posted by", readonly=True, tracking=True ) + + def post(self): + res = super().post() + self.write( + { + "last_post_date": fields.Datetime.now(), + "last_post_uid": self.env.user.id, + } + ) + return res diff --git a/account_move_post_date_user/readme/CONTRIBUTORS.rst b/account_move_post_date_user/readme/CONTRIBUTORS.rst index 0c9417fb5..d4ab92953 100644 --- a/account_move_post_date_user/readme/CONTRIBUTORS.rst +++ b/account_move_post_date_user/readme/CONTRIBUTORS.rst @@ -1,3 +1,4 @@ * `ForgeFlow S.L. `__: * Jordi Masvidal + * Guillem Casassas diff --git a/account_move_post_date_user/tests/test_account_move_post_date_user.py b/account_move_post_date_user/tests/test_account_move_post_date_user.py index c12763669..9a92becbb 100644 --- a/account_move_post_date_user/tests/test_account_move_post_date_user.py +++ b/account_move_post_date_user/tests/test_account_move_post_date_user.py @@ -4,7 +4,7 @@ from odoo import fields from odoo.tests import tagged -from odoo.addons.account.tests.common import AccountTestInvoicingCommon +from odoo.addons.account.tests.account_test_savepoint import AccountTestInvoicingCommon @tagged("post_install", "-at_install") @@ -47,5 +47,6 @@ class TestAccountMovePostDateUser(AccountTestInvoicingCommon): def test_account_move_post_date_user(self): self.move.action_post() + self.assertEqual(self.move.state, "posted") self.assertEqual(self.move.last_post_date.date(), fields.Date.today()) self.assertEqual(self.move.last_post_uid, self.env.user) diff --git a/account_move_post_date_user/views/account_move.xml b/account_move_post_date_user/views/account_move.xml index 0878aaf9d..e32e222dd 100644 --- a/account_move_post_date_user/views/account_move.xml +++ b/account_move_post_date_user/views/account_move.xml @@ -1,8 +1,8 @@ - account.move account.move.form - account_move_post_date_user + account.move