Merge lp:~therp-nl/openobject-server/7.0_lp1321675_crash_on_copy into lp:openobject-server/7.0

[Ronald Portier (Therp)]

Fixes lp1321675.

Prevent crash when copying record with (not stored) one2many function field.

Unmerged revisions

5307. By Ronald Portier (Therp) on 2014-05-21

[FIX] Prevents exception when copying record with one2many function field.

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2014-05-19 16:12:41 +0000
+++ openerp/osv/orm.py	2014-05-21 10:59:15 +0000
@@ -5066,13 +5066,20 @@
             old_record, new_record = self.browse(cr, uid, [old_id, new_id], context=context_wo_lang)
             # we must recursively copy the translations for o2o and o2m
             if field_def['type'] == 'one2many':
-                target_obj = self.pool.get(field_def['relation'])
-                # here we rely on the order of the ids to match the translations
-                # as foreseen in copy_data()
-                old_children = sorted(r.id for r in old_record[field_name])
-                new_children = sorted(r.id for r in new_record[field_name])
-                for (old_child, new_child) in zip(old_children, new_children):
-                    target_obj.copy_translations(cr, uid, old_child, new_child, context=context)
+                # do not copy function fields, unless they are stored:
+                if ((not 'function' in field_def)
+                        or ('store' in field_def and field_def['store'])):
+                    target_obj = self.pool.get(field_def['relation'])
+                    # here we rely on the order of the ids to match the
+                    # translations as foreseen in copy_data()
+                    old_children = sorted(
+                        r.id for r in old_record[field_name])
+                    new_children = sorted(
+                        r.id for r in new_record[field_name])
+                    for (old_child, new_child) in zip(
+                            old_children, new_children):
+                        target_obj.copy_translations(
+                            cr, uid, old_child, new_child, context=context)
             # and for translatable fields we keep them for copy
             elif field_def.get('translate'):
                 if field_name in self._columns: