openerp-dev-web team mailing list archive

["gpa\(Open ERP\)" <gpa@xxxxxxxxxxx>]

gpa(Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/copy_error_fix into lp:~openobject-training/openobject-addons/training.

Requested reviews:
  Bhumika (OpenERP) (sbh-openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/copy_error_fix/+merge/55892

Fixed key error: 0 when doing copy 
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/copy_error_fix/+merge/55892
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/copy_error_fix.

=== modified file 'training_intra/training_intra.py'
--- training_intra/training_intra.py	2011-03-21 09:06:32 +0000
+++ training_intra/training_intra.py	2011-04-01 09:57:29 +0000
@@ -294,12 +294,17 @@
         return r
 
     def copy_data(self, cr, uid, id, default=None, context=None):
+        if context is None:
+            context = {}
         res = super(training_offer, self).copy_data(cr, uid, id, default=default, context=context)
         if not res:
             return res
-        res[0].pop('sale_order_id', None)
-        res[0].pop('sale_order_create_date', None)
-        res[0].pop('sale_order_state', None)
+
+        to_delete = [ 'sale_order_id', 'sale_order_create_date', 'sale_order_state' ]
+        for key in to_delete:
+            if key in res:
+                del res[key]
+
         return res
 
     _columns = {