[FIX] website_event, website_sale: _order on is_published

As fbbf24f58a introduced multiwebsite and `website_published` became a
related and/or a computed field (depending of the mixin), we should not order
on this field as it is not on DB anymore.

We now have the field `is_published` that replace `website_published`.
This commit is contained in:
Romain Derie
2018-08-14 22:25:36 +02:00
parent fd9583982e
commit 75f3d8f1d7
3 changed files with 3 additions and 3 deletions
+1 -1
View File
@@ -123,7 +123,7 @@ class WebsiteEventController(http.Controller):
order = 'date_begin desc'
if searches["country"] != 'all': # if we are looking for a specific country
order = 'is_online, ' + order # show physical events first
order = 'website_published desc, ' + order
order = 'is_published desc, ' + order
events = Event.search(dom_without("none"), limit=step, offset=pager['offset'], order=order)
values = {
+1 -1
View File
@@ -174,7 +174,7 @@ class WebsiteSale(http.Controller):
def _get_search_order(self, post):
# OrderBy will be parsed in orm and so no direct sql injection
# id is added to be sure that order is a unique sort key
return 'website_published desc,%s , id desc' % post.get('order', 'website_sequence desc')
return 'is_published desc,%s , id desc' % post.get('order', 'website_sequence desc')
def _get_search_domain(self, search, category, attrib_values):
domain = request.website.sale_product_domain()
+1 -1
View File
@@ -106,7 +106,7 @@ class ProductPublicCategory(models.Model):
class ProductTemplate(models.Model):
_inherit = ["product.template", "website.seo.metadata", 'website.published.multi.mixin', 'rating.mixin']
_order = 'website_published desc, website_sequence desc, name'
_order = 'is_published desc, website_sequence desc, name'
_name = 'product.template'
_mail_post_access = 'read'