From 28407b4acf17aa8fd7a210713503a9beaaac918d Mon Sep 17 00:00:00 2001 From: Fabien BOURGEOIS Date: Tue, 16 Jan 2018 10:53:16 +0100 Subject: [PATCH] [FIX]GOLEM SEASON : use update for onchange + unit test warning on membership creation --- golem_season/models/golem_season.py | 6 ++++-- golem_season/tests/test_golem_season.py | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/golem_season/models/golem_season.py b/golem_season/models/golem_season.py index 97f3aa16..ed7f35ec 100644 --- a/golem_season/models/golem_season.py +++ b/golem_season/models/golem_season.py @@ -44,8 +44,10 @@ class GolemSeason(models.Model): for season in self: if season.membership_id: if not season.date_start: - season.date_start = season.membership_id.membership_date_from - season.date_end = season.membership_id.membership_date_to + season.update({ + 'date_start': season.membership_id.membership_date_from, + 'date_end': season.membership_id.membership_date_to + }) @api.constrains('date_start', 'date_end') def _check_period(self): diff --git a/golem_season/tests/test_golem_season.py b/golem_season/tests/test_golem_season.py index 1eafd4e8..59e93c3b 100644 --- a/golem_season/tests/test_golem_season.py +++ b/golem_season/tests/test_golem_season.py @@ -58,7 +58,7 @@ class TestGolemSeason(TransactionCase): def test__onchange_season_dates(self): """ Test if membership date """ membership = self.env['product.template'].create({'name': 'Name', - 'membership_id': 'truc', + 'membership': True, 'membership_date_from': '2017-11-01', 'membership_date_to': '2018-11-01'})