Merge lp:~vauxoo/openerp-venezuela-localization/ovl70-imex-context-rev-kty into lp:openerp-venezuela-localization

[Katherine Zaoral (Vauxoo)]

[MERGE] change context into l10n_ve_imex for properly use.

[hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha)]

review: Approve

=== modified file 'l10n_ve_imex/model/customs_form.py'
--- l10n_ve_imex/model/customs_form.py	2013-05-10 21:24:05 +0000
+++ l10n_ve_imex/model/customs_form.py	2013-06-14 13:45:37 +0000
@@ -148,6 +148,7 @@
         cfl_ids.imex_tax_line and get debit account from account_tax model
         """
         lines = []
+        context = context or {}
         company_id = context.get('f86_company_id')
         f86_cfg = context.get('f86_config')
 
@@ -189,9 +190,8 @@
         return lines
 
     def create_account_move(self, cr, uid, ids, context=None):
-        if context is None:
-            context = {}
-        so_brw = self.browse(cr, uid, ids, context={})
+        context = context or {}
+        so_brw = self.browse(cr, uid, ids, context=context)
         for f86 in so_brw:
             if f86.move_id:  # ~ The move is already done, nothing to do
                 return []
@@ -201,7 +201,8 @@
             cr, uid, uid, context=context).company_id.id
         company = self.pool.get('res.company').browse(cr, uid, company_id,
                                                       context=context)
-        cfg_id = obj_cfg.search(cr, uid, [('company_id', '=', company_id)])
+        cfg_id = obj_cfg.search(cr, uid, [('company_id', '=', company_id)],
+                                context=context)
         if cfg_id:
             f86_cfg = obj_cfg.browse(cr, uid, cfg_id[0], context=context)
         else:
@@ -227,38 +228,44 @@
                 str_inv = _('\n\tSupplier: %-40s Reference: %s') % \
                 (inv.partner_id.name, inv.supplier_invoice_number)
                 move['narration'] = '%s%s' % (move['narration'], str_inv)
-            lines = self.create_account_move_lines(cr, uid, f86, context)
+            lines = self.create_account_move_lines(cr, uid, f86,
+                                                   context=context)
             if lines:
                 move.update({'line_id': lines})
-                move_id = obj_move.create(cr, uid, move, context)
+                move_id = obj_move.create(cr, uid, move, context=context)
                 obj_move.post(cr, uid, [move_id], context=context)
                 if move_id:
                     move_ids.append(move_id)
-                    self.write(cr, uid, f86.id, {'move_id': move_id}, context)
+                    self.write(cr, uid, f86.id, {'move_id': move_id},
+                               context=context)
         return move_ids
 
     def button_draft(self, cr, uid, ids, context=None):
+        context = context or {}
         vals = {'state': 'draft'}
-        return self.write(cr, uid, ids, vals, context)
+        return self.write(cr, uid, ids, vals, context=context)
 
     def button_open(self, cr, uid, ids, context=None):
-        self.create_account_move(cr, uid, ids, context)
+        context = context or {}
+        self.create_account_move(cr, uid, ids, context=context)
         vals = {'state': 'open'}
-        return self.write(cr, uid, ids, vals, context)
+        return self.write(cr, uid, ids, vals, context=context)
 
     def button_done(self, cr, uid, ids, context=None):
+        context = context or {}
         vals = {'state': 'done'}
-        return self.write(cr, uid, ids, vals, context)
+        return self.write(cr, uid, ids, vals, context=context)
 
     def button_cancel(self, cr, uid, ids, context=None):
+        context = context or {}
         f86 = self.browse(cr, uid, ids[0], context=context)
         f86_move_id = f86.move_id.id if f86 and f86.move_id else False
         vals = {'state': 'cancel', 'move_id': 0}
-        res = self.write(cr, uid, ids, vals, context)
+        res = self.write(cr, uid, ids, vals, context=context)
         if f86_move_id:
             self.pool.get('account.move').unlink(cr, uid, [f86_move_id],
-                                                 context)
-        return self.write(cr, uid, ids, vals, context)
+                                                 context=context)
+        return self.write(cr, uid, ids, vals, context=context)
 
     def test_draft(self, cr, uid, ids, *args):
         return True
@@ -302,7 +309,7 @@
             raise osv.except_osv(
                 _('Error!'),
                 _('Multiple operations not allowed'))
-        for f86 in self.browse(cr, uid, ids, context=None):
+        for f86 in self.browse(cr, uid, ids, context={}):
             #~ Validate account_move.state != draft
             if f86.move_id and f86.move_id.state != 'draft':
                 raise osv.except_osv(

=== modified file 'l10n_ve_imex/model/customs_form_config.py'
--- l10n_ve_imex/model/customs_form_config.py	2013-04-29 17:17:15 +0000
+++ l10n_ve_imex/model/customs_form_config.py	2013-06-14 13:45:37 +0000
@@ -109,11 +109,12 @@
     _description = ''
     _order = 'sequence'
 
-    def name_get(self, cr, uid, ids, context):
+    def name_get(self, cr, uid, ids, context=None):
+        context = context or {}
         if not len(ids):
             return []
         res = []
-        so_brw = self.browse(cr, uid, ids, context)
+        so_brw = self.browse(cr, uid, ids, context=context)
         for item in so_brw:
             res.append((item.id, '[%s] %s - %s' % (
                 item.code, item.ref, item.name)))

=== modified file 'l10n_ve_imex/model/invoice.py'
--- l10n_ve_imex/model/invoice.py	2013-05-10 21:24:05 +0000
+++ l10n_ve_imex/model/invoice.py	2013-06-14 13:45:37 +0000
@@ -52,12 +52,13 @@
                                    on Book"),
     }
 
-    def on_change_customs_form_id(self, cr, uid, ids, customs_form_id):
+    def on_change_customs_form_id(self, cr, uid, ids, customs_form_id, context=None):
+        context = context or {}
         res = {}
         if customs_form_id:
             imp = self.pool.get('customs.form').browse(cr, uid,
                                                          customs_form_id,
-                                                         context=None)
+                                                         context=context)
             res = {'value': {'num_import_form': imp.name,
                              'import_invo': imp.date_liq}}
         return res
@@ -118,12 +119,14 @@
             #~ res = {'domain': {'invoice_id': [('id','in',invoices)]}}
         #~ return res
 
-    def on_change_amount(self, cr, uid, ids, tax_id, base_amount, tax_amount):
+    def on_change_amount(self, cr, uid, ids, tax_id, base_amount, tax_amount,
+                         context=None):
         """ To autocompute base or tax, only for percent based taxes. """
+        context = context or {}
         res = {}
         if tax_id:
             obj_vat = self.pool.get('account.tax')
-            vat = obj_vat.browse(cr, uid, tax_id)
+            vat = obj_vat.browse(cr, uid, tax_id, context=context)
             if vat.type == 'percent':
                 if base_amount == 0 and tax_amount > 0:
                     base_amount = round(tax_amount / vat.amount, 2)
@@ -138,11 +141,12 @@
 
         return res
 
-    def on_change_invoice_id(self, cr, uid, ids, invoice_id):
+    def on_change_invoice_id(self, cr, uid, ids, invoice_id, context=None):
+        context = context or {}
         res = {}
         if invoice_id:
             obj_inv = self.pool.get('account.invoice')
-            inv = obj_inv.browse(cr, uid, invoice_id)
+            inv = obj_inv.browse(cr, uid, invoice_id, context=context)
             res = {'value': {'partner_id': inv.partner_id.id,
                              'supplier_invoice_number': inv.supplier_invoice_number}}
         return res