Skip to content

Commit 4bfea2d

Browse files
committed
[FIX] stock_move_backdating: update date backdating when stock is backdate only
1 parent d47e842 commit 4bfea2d

File tree

2 files changed

+29
-19
lines changed

2 files changed

+29
-19
lines changed

stock_move_backdating/models/stock_picking.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,15 @@ def _backdating_update_stock_valuation_layers_date(self):
4747

4848
def _action_done(self):
4949
result = super()._action_done()
50-
for picking in self:
50+
51+
pickings_backdate = self.filtered_domain(
52+
[
53+
"|",
54+
("date_backdating", "!=", False),
55+
("move_ids.move_line_ids.date_backdating", "!=", False),
56+
]
57+
)
58+
for picking in pickings_backdate:
5159
picking._backdating_update_picking_date()
5260
picking._backdating_update_stock_valuation_layers_date()
5361
return result

stock_move_backdating/tests/test_picking.py

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,28 +4,30 @@
44
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
55

66

7+
from odoo.exceptions import UserError, ValidationError
8+
79
from .common import TestCommon
810

911

1012
class TestPicking(TestCommon):
11-
# def test_date_backdating_yesterday(self):
12-
# date_backdating = self._get_datetime_backdating(1)
13-
# self._transfer_picking_with_dates(date_backdating)
14-
15-
# def test_date_backdating_last_month(self):
16-
# date_backdating = self._get_datetime_backdating(31)
17-
# self._transfer_picking_with_dates(date_backdating)
18-
19-
# def test_date_backdating_future_wizard(self):
20-
# date_backdating = self._get_datetime_backdating(-1)
21-
# with self.assertRaises(ValidationError):
22-
# self._transfer_picking_with_dates(date_backdating)
23-
24-
# def test_date_backdating_future(self):
25-
# date_backdating_1 = self._get_datetime_backdating(-1)
26-
# date_backdating_2 = self._get_datetime_backdating(-2)
27-
# with self.assertRaises(UserError):
28-
# self._transfer_picking_with_dates(date_backdating_1, date_backdating_2)
13+
def test_date_backdating_yesterday(self):
14+
date_backdating = self._get_datetime_backdating(1)
15+
self._transfer_picking_with_dates(date_backdating)
16+
17+
def test_date_backdating_last_month(self):
18+
date_backdating = self._get_datetime_backdating(31)
19+
self._transfer_picking_with_dates(date_backdating)
20+
21+
def test_date_backdating_future_wizard(self):
22+
date_backdating = self._get_datetime_backdating(-1)
23+
with self.assertRaises(ValidationError):
24+
self._transfer_picking_with_dates(date_backdating)
25+
26+
def test_date_backdating_future(self):
27+
date_backdating_1 = self._get_datetime_backdating(-1)
28+
date_backdating_2 = self._get_datetime_backdating(-2)
29+
with self.assertRaises(UserError):
30+
self._transfer_picking_with_dates(date_backdating_1, date_backdating_2)
2931

3032
def test_different_dates_backdating(self):
3133
date_backdating_1 = self._get_datetime_backdating(1)

0 commit comments

Comments
 (0)