Browse files

🚑 web_website fixes after odoo update odoo/odoo@b6d32de

  • Loading branch information...
KolushovAlexandr committed Dec 3, 2018
1 parent 0bf2e41 commit d93c25fc51b691c7bc784ebad5d36bea81fa1280
Showing with 110 additions and 5 deletions.
  1. +106 −4 web_website/models/
  2. +4 −1 web_website/models/
@@ -1,8 +1,13 @@
# Copyright 2018 Ivan Yelizariev <>
# Copyright 2018 Kolushov Alexandr <>
# License LGPL-3.0 or later (
import logging
from odoo import models, fields, api
from odoo.addons.base.models.ir_property import TYPE2FIELD

_logger = logging.getLogger(__name__)

@@ -98,10 +103,107 @@ def get(self, name, model, res_id=False):

def get_multi(self, name, model, ids):
return super(IrProperty, self._check_website_dependent(
name, model,
)).get_multi(name, model, ids)
# Due to commit
# We override the super method to make it website dependent
# It has the same idea and structure, but sql request and set record value method are changed
if not ids:
return {}
website_id = self._context.get('website_id', None)
field = self.env[model]._fields[name]
field_id = self.env['ir.model.fields']._get(model, name).id
company_id = self._context.get('force_company') \
or self.env['']._company_default_get(model, field_id).id

if field.type == 'many2one':
comodel = self.env[field.comodel_name]
model_pos = len(model) + 2
value_pos = len(comodel._name) + 2
query = """
SELECT substr(p.res_id, %s)::integer,, p.company_id, p.website_id
FROM ir_property p
LEFT JOIN {} r ON substr(p.value_reference, %s)
WHERE p.fields_id=%s
AND (p.company_id=%s OR p.company_id IS NULL)
AND (p.website_id=%s OR p.website_id IS NULL)
AND (p.res_id IN %s OR p.res_id IS NULL)
ORDER BY p.website_id NULLS FIRST, p.company_id NULLS FIRST
params = [model_pos, value_pos, field_id, company_id, website_id]
elif field.type in TYPE2FIELD:
model_pos = len(model) + 2
query = """
SELECT substr(p.res_id, %s)::integer, p.{0}, p.company_id, p.website_id
FROM ir_property p
WHERE p.fields_id=%s
AND (p.company_id=%s OR p.company_id IS NULL)
AND (p.website_id=%s OR p.website_id IS NULL)
AND (p.res_id IN %s OR p.res_id IS NULL)
ORDER BY p.website_id NULLS FIRST, p.company_id NULLS FIRST
params = [model_pos, field_id, company_id, website_id]
return dict.fromkeys(ids, (False, False, False, False))

# retrieve values
cr =
result = {}
refs = {"%s,%s" % (model, id) for id in ids}
for sub_refs in cr.split_for_in_conditions(refs):
cr.execute(query, params + [sub_refs])
fetched_result = cr.fetchall()
_logger.debug('Properties for website=%s, company=%s, resource=%s:\n%s', website_id, company_id, sub_refs, fetched_result)
# fetched_result format: [(res_id, val, company_id, website_id), ...]
# fetched_result sorting:

# [(RES_ID, 'only_res', None, None),
# (None, 'only_field', None, None),

# (RES_ID, 'company_and_resource', COMPANY, None),
# (None, 'only_company', COMPANY, None),

# (RES_ID, 'website_and_resource', None, WEBSITE),
# (None, 'only_website', None, WEBSITE),

# (RES_ID, 'company_website', COMPANY, WEBSITE)]

res = {
i[0]: i for i in fetched_result

# result format: {res_id: (res_id, val, company_id, website_id), ...}
# from fetched result was taken only the last row for each res_id

default_value = result.pop(None, None)
default_company_id = default_value and default_value[2]
default_website_id = default_value and default_value[3]
for id in ids:
if id not in result:
# 5 Company, Resource and Website are empty (i.e. only Field is matched)
result[id] = default_value
result_website_id = result[id][3]
if result_website_id:
# 1 Website and Resource are matched
if default_website_id:
# 2 Website is matched, Resource is empty
result[id] = default_value
# No properties with website
result_company_id = result[id][2]
if result_company_id:
# 3 Company and Resource are matched, Website is empty
# no property for res and company
if default_company_id:
# 4 Company is matched, Resource and Website are empty
result[id] = default_value

for key, value in result.items():
# set data to appropriate form
result[key] = value[1]
# result format: {id: val, ...}
return result

def search_multi(self, name, model, operator, value):
@@ -68,7 +68,10 @@ def _force_default(self, field_name, prop_value):
value =
except AttributeError:

prop_value =
except AttributeError:
if value != prop_value:
vals['value'] = prop_value

0 comments on commit d93c25f

Please sign in to comment.