openerp-dev-web team mailing list archive

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

Yogesh(Open ERP) has proposed merging lp:~openerp-dev/openobject-addons/addons-training-intra-fix-onchange-method-ysa into lp:~openobject-training/openobject-addons/training.

Requested reviews:
  OpenObject Training (openobject-training)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/addons-training-intra-fix-onchange-method-ysa/+merge/59321

[FIX] training_intra :- return value in o2m field instead of id of created record. fix problem in on_change_budget_intra_template method in training.offer object.
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/addons-training-intra-fix-onchange-method-ysa/+merge/59321
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-addons/addons-training-intra-fix-onchange-method-ysa.

=== modified file 'training_intra/training_intra.py'
--- training_intra/training_intra.py	2011-04-11 08:57:50 +0000
+++ training_intra/training_intra.py	2011-04-28 06:22:23 +0000
@@ -429,28 +429,25 @@
         return {'value' : {}}
 
     def on_change_budget_intra_template(self, cr, uid, ids, tpl_id, context=None):
+        if context is None:
+            context = {}
         if not tpl_id:
             return False
-        if not ids:
-            return False
-        his = self.browse(cr, uid, ids[0], context=context)
         tpl_line_proxy = self.pool.get('training.intra.budget.tpl.line')
-        line_proxy = self.pool.get('training.intra.budget.line')
         line_ids = tpl_line_proxy.search(cr, uid, [('template_id', '=', tpl_id)], context=context)
-        new_line_ids = []
+        new_line_values = []
         for line in tpl_line_proxy.browse(cr, uid, line_ids, context=context):
             values = {
                 'product_id' : line.product_id.id,
                 'product_uom' : line.product_uom.id,
                 'product_qty' : line.product_qty,
                 'unit_price' : line.unit_price,
-                'offer_id' : ids[0],
                 'type' : line.type,
             }
-            new_line_ids.append(line_proxy.create(cr, uid, values, context=context))
+            new_line_values.append(values)
         return {
             'value' : {
-                'budget_intra_line_ids': new_line_ids
+                'budget_intra_line_ids': new_line_values
             }
         }