openerp-dev-web team mailing list archive

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

gpa(Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/fixed_error_course_copy 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/fixed_error_course_copy/+merge/54816

fixed error of the when duplicate the course
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/fixed_error_course_copy/+merge/54816
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/fixed_error_course_copy.

=== modified file 'training/training.py'
--- training/training.py	2011-03-23 12:26:56 +0000
+++ training/training.py	2011-03-25 07:31:01 +0000
@@ -522,16 +522,21 @@
             values['parent_id'] = proxy.browse(cr, uid, values['category_id'], context).analytic_account_id.id
         return super(training_course, self).write(cr, uid, ids, values, context)
 
+
     def copy_data(self, cr, uid, id, default=None, context=None):
         if context is None:
-            context = {}
-        data, trans_data = super(training_course, self).copy_data(cr, uid, id, default=default, context=context)
-        to_delete = [ 'theme_ids', 'questionnaire_ids', 'purchase_line_ids', 'pending_ids', 'line_ids', 'lecturer_ids', 'budget_intra_line_ids', 'attachment_ids' ]
-        for key in to_delete:
-            if key in data:
-                del data[key]
-        return data, trans_data
-
+            context={}
+
+        training_name = self.read(cr, uid, id, ['name'], context=context)
+
+        if not default:
+            default = {}
+
+        default = default.copy()
+        default['name'] = training_name['name'] + _(' (copy)')
+
+        default.update({'theme_ids':[], 'questionnaire_ids':[], 'pending_ids':[], 'line_ids':[], 'purchase_line_ids':[], 'lecturer_ids':[],'budget_intra_line_ids':[], 'attachment_ids':[]})
+        return super(training_course, self).copy_data(cr, uid, id, default=default, context=context)
 
     def on_change_reference(self, cr, uid, ids, reference_id, context=None):
         if context is None: