From 7c432c8b65f19f2bbe28c3054ebe862b3a09cf65 Mon Sep 17 00:00:00 2001 From: Thomas Groutars Date: Thu, 4 Feb 2016 11:06:01 -0800 Subject: [PATCH] [FIX] Fix get_default_ method decorator and definition In sale.config.settings, the method get_default_sale_config has api.multi decorator. However, the method definition should be def get_default_sale_config(self, cr, uid, fields, context) (see https://github.com/odoo/odoo/blob/9.0/openerp/addons/base/res/res_config.py#L534 ). The api.multi decorator results in the model being browsed with fields as ids, which results in that kind of queries: select * from sale_config_settings where id in ('generate_sales_team_alias') This change corrects the method definition to fix that. --- addons/sale_stock/res_config.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/sale_stock/res_config.py b/addons/sale_stock/res_config.py index 6cba692be88..ef3cf1af33e 100644 --- a/addons/sale_stock/res_config.py +++ b/addons/sale_stock/res_config.py @@ -28,8 +28,8 @@ class SaleConfiguration(models.TransientModel): ], "Order Routing", implied_group='sale_stock.group_route_so_lines') - @api.multi - def get_default_sale_config(self): + @api.model + def get_default_sale_config(self, fields): default_picking_policy = self.env['ir.values'].get_default('sale.order', 'picking_policy') return { 'default_picking_policy': 1 if default_picking_policy == 'one' else 0,