From b1247201efc90aef70a65d417dfd3e4789715b25 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Wed, 27 Mar 2019 00:28:57 +0100
Subject: [PATCH] Raise NotImplementedError when setting Function field without
 setter

issue7985
review265101002
---
 proteus/__init__.py | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/proteus/__init__.py b/proteus/__init__.py
index 14d18d2849..cecfc99167 100644
--- a/proteus/__init__.py
+++ b/proteus/__init__.py
@@ -885,8 +885,15 @@ class Model(object):
         'Return dictionary values'
         if fields is None:
             fields = self._values.keys()
-        return dict((x, getattr(self, '__%s_value' % x)) for x in fields
-                if x not in ('id', '_timestamp'))
+        values = {}
+        for name in fields:
+            if name in ['id', '_timestamp']:
+                continue
+            definition = self._fields[name]
+            if definition.get('readonly') and definition['type'] != 'one2many':
+                continue
+            values[name] = getattr(self, '__%s_value' % name)
+        return values
 
     @property
     def _timestamp(self):