From 642fce4a5b7fae952e1572bb33f7e67cd2d18271 Mon Sep 17 00:00:00 2001 From: Fabien Pinckaers Date: Tue, 3 Dec 2013 18:10:33 +0100 Subject: [PATCH] [IMP] speed improvement + small bug bzr revid: fp@tinyerp.com-20131203171033-oe6pkzb9jal3ril8 --- addons/website/models/website.py | 8 +++++--- addons/website_sale/views/website_sale.xml | 4 ++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/addons/website/models/website.py b/addons/website/models/website.py index 738c8d460ae..2f58cb206bd 100644 --- a/addons/website/models/website.py +++ b/addons/website/models/website.py @@ -181,10 +181,12 @@ class website(osv.osv): return self.pool['website'].browse(cr, uid, 1, context=context) def preprocess_request(self, cr, uid, ids, request, context=None): + # TODO FP: is_website_publisher and editable in context should be removed + # for performance reasons (1 query per image to load) but also to be cleaner + # I propose to replace this by a group 'base.group_website_publisher' on the + # view that requires it. Access = request.registry['ir.model.access'] - is_website_publisher = ( - Access.check_groups(cr, uid, 'base.group_website_publisher') - and Access.check(cr, uid, 'ir.ui.view', 'write', False, context)) + is_website_publisher = Access.check(cr, uid, 'ir.ui.view', 'write', False, context) lang = request.context['lang'] is_master_lang = lang == request.website.default_lang_code diff --git a/addons/website_sale/views/website_sale.xml b/addons/website_sale/views/website_sale.xml index b3fc9862f69..282d401b271 100644 --- a/addons/website_sale/views/website_sale.xml +++ b/addons/website_sale/views/website_sale.xml @@ -42,8 +42,8 @@