Skip to content

Commit adc6d3e

Browse files
committed
[FIX] stock_move_backdating: update date backdating when stock is backdate only
1 parent 9e30116 commit adc6d3e

File tree

2 files changed

+26
-19
lines changed

2 files changed

+26
-19
lines changed

stock_move_backdating/models/stock_picking.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,13 @@ 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+
('date_backdating', '!=', False),
54+
('move_ids.move_line_ids.date_backdating', '!=', False),
55+
])
56+
for picking in pickings_backdate:
5157
picking._backdating_update_picking_date()
5258
picking._backdating_update_stock_valuation_layers_date()
5359
return result

stock_move_backdating/tests/test_picking.py

Lines changed: 19 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -5,27 +5,28 @@
55

66

77
from .common import TestCommon
8+
from odoo.exceptions import ValidationError, UserError
89

910

1011
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)
12+
def test_date_backdating_yesterday(self):
13+
date_backdating = self._get_datetime_backdating(1)
14+
self._transfer_picking_with_dates(date_backdating)
15+
16+
def test_date_backdating_last_month(self):
17+
date_backdating = self._get_datetime_backdating(31)
18+
self._transfer_picking_with_dates(date_backdating)
19+
20+
def test_date_backdating_future_wizard(self):
21+
date_backdating = self._get_datetime_backdating(-1)
22+
with self.assertRaises(ValidationError):
23+
self._transfer_picking_with_dates(date_backdating)
24+
25+
def test_date_backdating_future(self):
26+
date_backdating_1 = self._get_datetime_backdating(-1)
27+
date_backdating_2 = self._get_datetime_backdating(-2)
28+
with self.assertRaises(UserError):
29+
self._transfer_picking_with_dates(date_backdating_1, date_backdating_2)
2930

3031
def test_different_dates_backdating(self):
3132
date_backdating_1 = self._get_datetime_backdating(1)

0 commit comments

Comments
 (0)