From 9cfd1eefd895cb21a600cd7c9f9c490ad637049a Mon Sep 17 00:00:00 2001 From: Jeremy Kersten Date: Thu, 17 Jan 2019 08:43:35 +0000 Subject: [PATCH] [FIX] website: don't copy is_published Before this commit, if you duplicate a published record on the website, the copy was already published, while in most of case you need to update it to adapt to the new content. There doesn't make senses to duplicate the same content on a website and let the duplicate content. Task-1930058 --- addons/website/models/ir_ui_view.py | 3 ++- addons/website/models/website.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/addons/website/models/ir_ui_view.py b/addons/website/models/ir_ui_view.py index c427dd70086..cbfb525431f 100644 --- a/addons/website/models/ir_ui_view.py +++ b/addons/website/models/ir_ui_view.py @@ -73,7 +73,7 @@ class View(models.Model): website_specific_view = view.copy(copy_vals) view._create_website_specific_pages_for_view(website_specific_view, - view.env['website'].browse(current_website_id)) + view.env['website'].browse(current_website_id)) for inherit_child in view.inherit_children_ids.filter_duplicate().sorted(key=lambda v: (v.priority, v.id)): if inherit_child.website_id.id == current_website_id: @@ -167,6 +167,7 @@ class View(models.Model): # create new pages for this view page.copy({ 'view_id': new_view.id, + 'is_published': page.is_published, }) @api.model diff --git a/addons/website/models/website.py b/addons/website/models/website.py index 252ec58ee14..8dc06a4f972 100644 --- a/addons/website/models/website.py +++ b/addons/website/models/website.py @@ -840,7 +840,7 @@ class WebsitePublishedMixin(models.AbstractModel): _description = 'Website Published Mixin' website_published = fields.Boolean('Visible on current website', related='is_published', readonly=False) - is_published = fields.Boolean('Is published') + is_published = fields.Boolean('Is published', copy=False) website_url = fields.Char('Website URL', compute='_compute_website_url', help='The full URL to access the document through the website.') @api.multi