pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] website_sale: propagate modified context only in useful methods

Browse Source 
The pricelist in the context is only useful when computing the price of the
product.
Do not modifiy request context but only a local copy.

active_id is only used for the computation of the extra price from variants
This commit is contained in:
Martin Trigaux
2016-03-17 17:18:55 +01:00
parent 7b1d655d8c
commit a1339c5aad
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/website_sale/controllers/main.py
+9 -8
View File
@@ -194,11 +194,12 @@ class website_sale(http.Controller):
keep = QueryURL('/shop', category=category and int(category), search=search, attrib=attrib_list)
if not context.get('pricelist'):
pricelist_context = dict(context)
if not pricelist_context.get('pricelist'):
pricelist = self.get_pricelist()
context['pricelist'] = int(pricelist)
pricelist_context['pricelist'] = int(pricelist)
else:
pricelist = pool.get('product.pricelist').browse(cr, uid, context['pricelist'], context)
pricelist = pool.get('product.pricelist').browse(cr, uid, pricelist_context['pricelist'], context)
url = "/shop"
if search:
post["search"] = search
@@ -229,7 +230,7 @@ class website_sale(http.Controller):
product_count = product_obj.search_count(cr, uid, domain, context=context)
pager = request.website.pager(url=url, total=product_count, page=page, step=ppg, scope=7, url_args=post)
product_ids = product_obj.search(cr, uid, domain, limit=ppg, offset=pager['offset'], order='website_published desc, website_sequence desc', context=context)
products = product_obj.browse(cr, uid, product_ids, context=context)
products = product_obj.browse(cr, uid, product_ids, context=pricelist_context)
attributes_obj = request.registry['product.attribute']
if product_ids:
@@ -269,7 +270,7 @@ class website_sale(http.Controller):
category_obj = pool['product.public.category']
template_obj = pool['product.template']
context.update(active_id=product.id)
product_context = dict(context, active_id=product.id)
if category:
category = category_obj.browse(cr, uid, int(category), context=context)
@@ -297,9 +298,9 @@ class website_sale(http.Controller):
rating_message_values = dict([(record.message_id.id, record.rating) for record in ratings])
rating_product = product.rating_get_stats([('website_published', '=', True)])
if not context.get('pricelist'):
context['pricelist'] = int(self.get_pricelist())
product = template_obj.browse(cr, uid, int(product), context=context)
if not product_context.get('pricelist'):
product_context['pricelist'] = int(self.get_pricelist())
product = template_obj.browse(cr, uid, int(product), context=product_context)
values = {
'search': search,