From 1f77c2d6814f7ffa3911f3b3cd3ef344f7695eb1 Mon Sep 17 00:00:00 2001
From: Raphael Collet <rco@openerp.com>
Date: Fri, 24 Feb 2017 11:45:15 +0100
Subject: [PATCH] [FIX] models: add warning for inconsistent `compute_sudo` on
 fields (#15184)

Check that fields using the same compute method have the same value for
parameter `compute_sudo`.
---
 openerp/models.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/openerp/models.py b/openerp/models.py
index 2be2de2196bea..85c6a45c65b75 100644
--- a/openerp/models.py
+++ b/openerp/models.py
@@ -3077,6 +3077,11 @@ def _setup_fields(self, partial):
             if field.compute:
                 cls._field_computed[field] = group = groups[field.compute]
                 group.append(field)
+        for fields in groups.itervalues():
+            compute_sudo = fields[0].compute_sudo
+            if not all(field.compute_sudo == compute_sudo for field in fields):
+                _logger.warning("%s: inconsistent 'compute_sudo' for computed fields: %s",
+                                self._name, ", ".join(field.name for field in fields))
 
     @api.model
     def _setup_complete(self):