From edac5550ec4cf38236f54278c9f2e11a59c3ebbf Mon Sep 17 00:00:00 2001 From: Romain Derie Date: Thu, 25 Jan 2018 13:09:31 +0100 Subject: [FIX] website_blog: copy dict correctly The 'copy_vals' dict would refer to the same 'vals' dict since list are mutables. This was not the expected behavior. --- addons/website_blog/models/website_blog.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'addons/website_blog/models') diff --git a/addons/website_blog/models/website_blog.py b/addons/website_blog/models/website_blog.py index e5c39e8b594..94c423b3153 100644 --- a/addons/website_blog/models/website_blog.py +++ b/addons/website_blog/models/website_blog.py @@ -209,7 +209,7 @@ class BlogPost(models.Model): def write(self, vals): result = True for post in self: - copy_vals = vals + copy_vals = dict(vals) if 'website_published' in vals and 'published_date' not in vals and post.published_date <= fields.Datetime.now(): copy_vals['published_date'] = vals['website_published'] and fields.Datetime.now() or False result &= super(BlogPost, self).write(copy_vals) -- cgit v1.2.3