From 4f2f1247a6b5ef5415793f1cd9f60ca83fd95261 Mon Sep 17 00:00:00 2001 From: Raphael Collet Date: Thu, 26 May 2016 13:30:46 +0200 Subject: [PATCH] [IMP] api: make decorator `@returns` work when `model=None` Specifically, the `upgrade` function simply returns the given value. --- openerp/api.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/openerp/api.py b/openerp/api.py index 0ec3342cd74..a846de540d1 100644 --- a/openerp/api.py +++ b/openerp/api.py @@ -293,10 +293,10 @@ def get_upgrade(method): return upgrade elif model == 'self': return lambda self, *args, **kwargs: self.browse(args[0]) - else: + elif model: return lambda self, *args, **kwargs: self.env[model].browse(args[0]) - else: - return lambda self, *args, **kwargs: args[0] + + return lambda self, *args, **kwargs: args[0] def get_aggregate(method): @@ -309,10 +309,10 @@ def get_aggregate(method): model, _, _ = spec if model == 'self': return lambda self, value: sum(value, self.browse()) - else: + elif model: return lambda self, value: sum(value, self.env[model].browse()) - else: - return lambda self, value: value + + return lambda self, value: value def get_context_split(method):