Merge lp:~vauxoo/openerp-mexico-localization/7.0_l10n_mx_isinstance_ids_dev_carlos into lp:openerp-mexico-localization/7.0

[Juan Carlos Hernandez]

se agrego guideline
if context is None:
    context = {}
ids = isinstance(ids, (int, long)) and [ids] or ids

[Isaac López Zúñiga (isaako34)]

Carlos, todo pinta bien, sin embargo al ejecutar los siguientes rgrep:

rgrep "context={}" . --include=*.py
rgrep "if not context:" . --include=*.py

Siguen apareciendo líneas sin el guideline, nos puedes apoyar en revisarlo por favor.

Saludos.

review: Needs Fixing

[lp:~vauxoo/openerp-mexico-localization/7.0_l10n_mx_isinstance_ids_dev_carlos]

325. By Juan Carlos Hernandez on 2013-11-08

[IMP[l10n_mx_facturae]modify context={} to context=None

[Juan Carlos Hernandez (openerp1)]

Isaac,
Cambios mencionados anteriormente ya quedaron.

review: Needs Resubmitting

[Juan Carlos Hernandez (openerp1)]

Isaac ya quedaron resueltos los comentarios que tu mencionaste

review: Needs Resubmitting

[Isaac López Zúñiga (isaako34)]

Carlos, por favor ejecuta: rgrep "if not context" . --include=*.py

Y obtendrás los siguientes resultados, esas líneas son las que falta corregir:

./l10n_mx_account_tax_category/invoice.py: if not context:
./l10n_mx_invoice_amount_to_text/invoice.py: if not context:
./l10n_mx_import_info/product.py: if not context:
./l10n_mx_invoice_datetime/invoice.py: if not context:
./l10n_mx_base_vat_split/base_vat.py: if not context:
./l10n_mx_facturae_seq/ir_sequence.py: if not context:
./l10n_mx_facturae_seq/invoice.py: if not context:
./l10n_mx_facturae_cer/res_company.py: if not context:
./l10n_mx_facturae_cer/invoice.py: if not context:
./l10n_mx_facturae_cbb/invoice.py: if not context:
./l10n_mx_facturae_cbb/invoice.py: if not context:

Saludos.

review: Needs Fixing

[lp:~vauxoo/openerp-mexico-localization/7.0_l10n_mx_isinstance_ids_dev_carlos]

326. By Juan Carlos Hernandez on 2013-11-14

[IMP][l10n_mx]corrected if not context

[Juan Carlos Hernandez (openerp1)]

Isaac cambios listos del if not context:

review: Needs Resubmitting

[lp:~vauxoo/openerp-mexico-localization/7.0_l10n_mx_isinstance_ids_dev_carlos]

327. By Juan Carlos Hernandez on 2013-11-20

[MERGE]

328. By Juan Carlos Hernandez on 2013-11-20

[IMP][l10n_mx_ir_attachment_facturae]add ids = isinstance(ids, (int, long)) and [ids] or ids

[Juan Carlos Hernandez (openerp1)]

Isaac ya está el merge de lo ultimo y más nuevo, y ya resolví los conflictos.

review: Needs Resubmitting

[Juan Carlos Hernandez (openerp1)]

Isaac ya quedo lista la tarea,puedes revisarla porfavor

review: Needs Resubmitting

[lp:~vauxoo/openerp-mexico-localization/7.0_l10n_mx_isinstance_ids_dev_carlos]

329. By Juan Carlos Hernandez on 2013-12-03

[MERGE]

330. By Juan Carlos Hernandez on 2013-12-04

[MERGE]

331. By Juan Carlos Hernandez on 2013-12-10

[DEL][l10n_mx_facturae_pac_sf]del folder report

332. By Juan Carlos Hernandez on 2013-12-10

[MERGE]

333. By Juan Carlos Hernandez on 2013-12-11

[IMP][l10n_mx_facturae_cer]add context=None

[Jorge Angel Naranjo Rogel - http://www.vauxoo.com (jorge-nr)]

Busque los
rgrep "context={}" . --include=*.py
rgrep "if not context:" . --include=*.py

y ya no encuentra ninguna coincidencia y el runbot marca en verde 7.0-agrinos-amx-bioderpac-mp-194270

review: Approve

=== modified file 'city/partner_address.py'
--- city/partner_address.py	2013-05-26 02:34:03 +0000
+++ city/partner_address.py	2013-12-16 18:29:27 +0000
@@ -68,7 +68,6 @@
         return {}
 
     def onchange_state_city(self, cr, uid, ids, state_id, city_id, context=None):
-
         res = super(res_partner, self).onchange_state(cr, uid, ids, state_id, context)
         if city_id and state_id and self.pool.get('res.country.state.city').browse(cr, uid, city_id, context).state_id.id != state_id:
             if res and 'value' in res:

=== modified file 'l10n_mx_account_tax_category/invoice.py'
--- l10n_mx_account_tax_category/invoice.py	2013-05-09 23:30:27 +0000
+++ l10n_mx_account_tax_category/invoice.py	2013-12-16 18:29:27 +0000
@@ -37,8 +37,8 @@
 class account_invoice_tax(osv.Model):
     _inherit = "account.invoice.tax"
 
-    def _get_tax_data(self, cr, uid, ids, field_names=None, arg=False, context={}):
-        if not context:
+    def _get_tax_data(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         for invoice_tax in self.browse(cr, uid, ids, context=context):

=== modified file 'l10n_mx_base_vat_split/base_vat.py'
--- l10n_mx_base_vat_split/base_vat.py	2013-05-09 23:34:38 +0000
+++ l10n_mx_base_vat_split/base_vat.py	2013-12-16 18:29:27 +0000
@@ -30,9 +30,8 @@
 class res_partner(osv.Model):
     _inherit = 'res.partner'
 
-    def _get_base_vat_split(self, cr, uid, ids, field_names=None, arg=False,
-                            context={}):
-        if not context:
+    def _get_base_vat_split(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         for partner in self.browse(cr, uid, ids, context=context):

=== modified file 'l10n_mx_cities/tools/read_write_xml.py'
--- l10n_mx_cities/tools/read_write_xml.py	2013-05-09 23:41:43 +0000
+++ l10n_mx_cities/tools/read_write_xml.py	2013-12-16 18:29:27 +0000
@@ -56,7 +56,6 @@
 main_node = add_node('data', {"noupdate": "True"}, nodeopenerp,
                     xml_doc, attrs_types={"noupdate": "attribute"})
 
-print '------------Generando xml'
 for elem in root[1:]:
     for a in elem:
         if a.tag == '{NewDataSet}c_mnpio':
@@ -113,9 +112,6 @@
                                     xml_doc, node_states_attrs_types, order)
         node_record.appendChild(node_field_states)
 
-
-print 'xml_doc to xml', xml_doc.toxml('UTF-8')
-print '\n------------xml generado con exito'
 data_full_path = os.path.join(data_path, 'l10n_mx_cities.xml')
 
 f = open(data_full_path, 'wb')

=== modified file 'l10n_mx_company_cif/wizard/installer.py'
--- l10n_mx_company_cif/wizard/installer.py	2013-05-09 23:46:20 +0000
+++ l10n_mx_company_cif/wizard/installer.py	2013-12-16 18:29:27 +0000
@@ -37,12 +37,16 @@
         }, context=context)
 
     def execute(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        if not ids:
+            return True
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         company_id = self.pool.get('res.users').browse(
             cr, uid, [uid], context)[0].company_id.partner_id.id
-        wiz_data = self.read(cr, uid, ids[0])
-        if wiz_data['cif_file']:
-            self._write_company(cr, uid, wiz_data[
-                                "cif_file"], company_id, context)
+        wiz_data = self.read(cr, uid, ids)
+        if wiz_data[0]['cif_file']:
+            self._write_company(cr, uid, wiz_data[0]["cif_file"], company_id, context)
 
     _columns = {
         'cif_file': fields.binary('CIF', help="Fiscal Identification Card"),

=== modified file 'l10n_mx_company_multi_address/invoice.py'
--- l10n_mx_company_multi_address/invoice.py	2013-05-31 21:36:22 +0000
+++ l10n_mx_company_multi_address/invoice.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,8 @@
     _inherit = 'account.invoice'
 
     def _get_address_issued_invoice(self, cr, uid, ids, name, args, context=None):
+        if context is None:
+            context = {}
         res = {}
         journal_obj = self.pool.get('account.journal')
         for id_ in ids:
@@ -52,6 +54,8 @@
         return res
 
     def _get_company_emitter_invoice(self, cr, uid, ids, name, args, context=None):
+        if context is None:
+            context = {}
         res = {}
         journal_obj = self.pool.get('account.journal')
         for id_ in ids:
@@ -77,6 +81,8 @@
     }
 
     def onchange_journal_id(self, cr, uid, ids, journal_id=False, context=None):
+        if context is None:
+            context = {}
         result = super(account_invoice, self).onchange_journal_id(
             cr, uid, ids, journal_id, context=context)
         address_id = journal_id and self.pool.get('account.journal').browse(

=== modified file 'l10n_mx_company_multi_address/res_company.py'
--- l10n_mx_company_multi_address/res_company.py	2013-05-10 14:44:46 +0000
+++ l10n_mx_company_multi_address/res_company.py	2013-12-16 18:29:27 +0000
@@ -35,8 +35,7 @@
 class res_company(osv.Model):
     _inherit = 'res.company'
 
-    def get_address_invoice_parent_company_id(self, cr, uid, ids, field, arg,
-        context=None):
+    def get_address_invoice_parent_company_id(self, cr, uid, ids, field, arg, context=None):
         if context is None:
             context = {}
         res = {}

=== modified file 'l10n_mx_diot_report/wizard/wizard_diot_report.py'
--- l10n_mx_diot_report/wizard/wizard_diot_report.py	2013-11-14 20:30:30 +0000
+++ l10n_mx_diot_report/wizard/wizard_diot_report.py	2013-12-16 18:29:27 +0000
@@ -64,6 +64,8 @@
         This function load in the wizard, the company used by the user, and
         the previous period to the current
         """
+        if context is None:
+            context = {}
         data = super(wizard_account_diot_mx, self).default_get(cr, uid,
             fields, context=context)
         time_now = datetime.date.today()+relativedelta(months=-1)
@@ -85,6 +87,7 @@
         """
         if context is None:
             context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         acc_move_line_obj = self.pool.get('account.move.line')
         acc_tax_obj = self.pool.get('account.tax')
         acc_tax_category_obj = self.pool.get('account.tax.category')

=== modified file 'l10n_mx_facturae/invoice.py'
--- l10n_mx_facturae/invoice.py	2013-12-09 19:11:26 +0000
+++ l10n_mx_facturae/invoice.py	2013-12-16 18:29:27 +0000
@@ -142,15 +142,14 @@
             # return (False,str(e))
         return (True, ret_file_name)
 
-    def create_report_pdf(self, cr, uid, ids, context={}):
-        if not context:
+    def create_report_pdf(self, cr, uid, ids, context=None):
+        if context is None:
             context = {}
-        id = ids[0]
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         (fileno, fname) = tempfile.mkstemp(
             '.pdf', 'openerp_' + (False or '') + '__facturae__')
         os.close(fileno)
-        file = self.create_report(cr, uid, [
-                                  id], "account.invoice.facturae.pdf", fname)
+        file = self.create_report(cr, uid, ids, "account.invoice.facturae.pdf", fname)
         is_file = file[0]
         fname = file[1]
         if is_file and os.path.isfile(fname):
@@ -163,7 +162,7 @@
                 'datas_fname': context.get('fname'),
                 'description': 'Factura-E PDF',
                 'res_model': self._name,
-                'res_id': id,
+                'res_id': ids,
             }
             self.pool.get('ir.attachment').create(
                 cr, uid, data_attach, context=context)
@@ -215,8 +214,8 @@
                           (ref, move_id))
         return True
 
-    def _get_fname_invoice(self, cr, uid, ids, field_names=None, arg=False, context={}):
-        if not context:
+    def _get_fname_invoice(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         sequence_obj = self.pool.get('ir.sequence')
@@ -247,6 +246,7 @@
         return res
 
     def action_cancel_draft(self, cr, uid, ids, *args):
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         self.write(cr, uid, ids, {
             'no_certificado': False,
             'certificado': False,
@@ -257,6 +257,9 @@
         return super(account_invoice, self).action_cancel_draft(cr, uid, ids, args)
 
     def action_cancel(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         self.write(cr, uid, ids, {
                    'date_invoice_cancel': time.strftime('%Y-%m-%d %H:%M:%S')})
         return super(account_invoice, self).action_cancel(cr, uid, ids, context=context)
@@ -351,8 +354,8 @@
         os.close(fileno)
         return fname
 
-    def _get_file_globals(self, cr, uid, ids, context={}):
-        if not context:
+    def _get_file_globals(self, cr, uid, ids, context=None):
+        if context is None:
             context = {}
         id = ids and ids[0] or False
         file_globals = {}
@@ -479,7 +482,9 @@
                         'cadenaoriginal_3_2_l.xslt') or ''
         return file_globals
 
-    def _____________get_facturae_invoice_txt_data(self, cr, uid, ids, context={}):
+    def _____________get_facturae_invoice_txt_data(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         # TODO: Transform date to fmt %d/%m/%Y %H:%M:%S
         certificate_lib = self.pool.get('facturae.certificate.library')
         fname_repmensual_xslt = self._get_file_globals(
@@ -500,7 +505,9 @@
                 rep_mensual += '\r\n'
         return rep_mensual, fname_tmp
 
-    def _get_facturae_invoice_txt_data(self, cr, uid, ids, context={}):
+    def _get_facturae_invoice_txt_data(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}    
         facturae_datas = self._get_facturae_invoice_dict_data(
             cr, uid, ids, context=context)
         facturae_data_txt_lists = []
@@ -587,7 +594,9 @@
             '%m%Y', time.strptime(fecha_promedio, '%Y-%m-%dT%H:%M:%S')) + '.txt'
         return cad, fname
 
-    def _get_folio(self, cr, uid, ids, context={}):
+    def _get_folio(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         folio_data = {}
         id = ids and ids[0] or False
         if id:
@@ -667,7 +676,7 @@
                     'Not found a sequence of configuration. %s !') % (msg2))
         return folio_data
 
-    def _dict_iteritems_sort(self, data_dict):  # cr=False, uid=False, ids=[], context={}):
+    def _dict_iteritems_sort(self, data_dict):  # cr=False, uid=False, ids=[], context=None):
         """
         @param data_dict : Dictionary with data from invoice
         """
@@ -730,8 +739,8 @@
             doc.appendChild(node)
         return doc
 
-    def _get_facturae_invoice_xml_data(self, cr, uid, ids, context={}):
-        if not context:
+    def _get_facturae_invoice_xml_data(self, cr, uid, ids, context=None):
+        if context is None:
             context = {}
         data_dict = self._get_facturae_invoice_dict_data(
             cr, uid, ids, context=context)[0]
@@ -840,15 +849,17 @@
                         raise osv.except_osv('Error al validar la estructura del xml!', 'Validación de XML versión %s:\n%s'%(facturae_version, result))
         return True
 
-    def write_cfd_data(self, cr, uid, ids, cfd_datas, context={}):
+    def write_cfd_data(self, cr, uid, ids, cfd_datas, context=None):
         """
         @param cfd_datas : Dictionary with data that is used in facturae CFD and CFDI
         """
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         if not cfd_datas:
             cfd_datas = {}
         # obtener cfd_data con varios ids
         # for id in ids:
-        id = ids[0]
         if True:
             data = {}
             cfd_data = cfd_datas
@@ -864,7 +875,7 @@
                 'sello': sello,
                 'cadena_original': cadena_original,
             }
-            self.write(cr, uid, [id], data, context=context)
+            self.write(cr, uid, ids, data, context=context)
         return True
 
     def _get_noCertificado(self, cr, uid, ids, fname_cer, pem=True, context=None):
@@ -903,7 +914,9 @@
             fname_out=fname_sign, encrypt=encrypt, type_key='PEM')
         return result
 
-    def _xml2cad_orig(self, cr=False, uid=False, ids=False, context={}):
+    def _xml2cad_orig(self, cr=False, uid=False, ids=False, context=None):
+        if context is None:
+            context = {}
         certificate_lib = self.pool.get('facturae.certificate.library')
         fname_tmp = certificate_lib.b64str_to_tempfile(cr, uid, ids, base64.encodestring(
             ''), file_suffix='.txt', file_prefix='openerp__' + (False or '') + \
@@ -941,7 +954,9 @@
         open(fname_cadorig_digest, "w").write(cadorig_digest)
         return cadorig_digest, fname_cadorig_digest
 
-    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context={}):
+    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         invoices = self.browse(cr, uid, ids, context=context)
         invoice_tax_obj = self.pool.get("account.invoice.tax")
         invoice_datas = []

=== modified file 'l10n_mx_facturae/ir_attachment_facturae.py'
--- l10n_mx_facturae/ir_attachment_facturae.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/ir_attachment_facturae.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,8 @@
     _inherit = 'ir.attachment.facturae.mx'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = super(ir_attachment_facturae_mx, self)._get_type(
             cr, uid, ids, context=context)
         types.extend([

=== modified file 'l10n_mx_facturae/ir_sequence_approval.py'
--- l10n_mx_facturae/ir_sequence_approval.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/ir_sequence_approval.py	2013-12-16 18:29:27 +0000
@@ -33,6 +33,8 @@
     _inherit = 'ir.sequence.approval'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = super(ir_sequence_approval, self)._get_type(
             cr, uid, ids, context=context)
         types.extend([

=== modified file 'l10n_mx_facturae/wizard/installer.py'
--- l10n_mx_facturae/wizard/installer.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/installer.py	2013-12-16 18:29:27 +0000
@@ -33,6 +33,8 @@
     _description = __doc__
 
     def default_get(self, cr, uid, fields_list=None, context=None):
+        if context is None:
+            context = {}
         defaults = super(facturae_config, self).default_get(
             cr, uid, fields_list=fields_list, context=context)
         logo = open(addons.get_module_resource(
@@ -45,7 +47,8 @@
         @param vat : VAT that will be set in the company
         @param company_id : Id from the company that the user works
         """
-        print 'company_id', company_id
+        if context is None:
+            context = {}
         partner_id = self.pool.get('res.company').browse(
             cr, uid, company_id).partner_id.id
         partner_obj = self.pool.get('res.partner')
@@ -55,11 +58,13 @@
             }, context=context)
 
     def execute(self, cr, uid, ids, context=None):
-        company_id = self.pool.get('res.users').browse(
-            cr, uid, [uid], context)[0].company_id.partner_id.id
-        wiz_data = self.read(cr, uid, ids[0])
-        if wiz_data['vat']:
-            self._assign_vat(cr, uid, wiz_data["vat"], company_id, context)
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
+        company_id = self.pool.get('res.users').browse(cr, uid, [uid], context)[0].company_id.partner_id.id
+        wiz_data = self.read(cr, uid, ids)
+        if wiz_data[0]['vat']:
+            self._assign_vat(cr, uid, wiz_data[0]["vat"], company_id, context)
 
     _columns = {
         'vat': fields.char('VAT', 64, help='Federal Register of Causes'),

=== modified file 'l10n_mx_facturae/wizard/wizard_create_facturae_file.py'
--- l10n_mx_facturae/wizard/wizard_create_facturae_file.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/wizard_create_facturae_file.py	2013-12-16 18:29:27 +0000
@@ -49,8 +49,8 @@
 # end_fields = {}
 
 
-def _create_facturae_file(self, cr, uid, data, context={}):
-    if not context:
+def _create_facturae_file(self, cr, uid, data, context=None):
+    if context is None:
         context = {}
     context.update({'date': data['form']['date']})
     pool = pooler.get_pool(cr.dbname)

=== modified file 'l10n_mx_facturae/wizard/wizard_invoice_facturae_txt.py'
--- l10n_mx_facturae/wizard/wizard_invoice_facturae_txt.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/wizard_invoice_facturae_txt.py	2013-12-16 18:29:27 +0000
@@ -82,8 +82,8 @@
 }
 
 
-def _create_facturae_txt(self, cr, uid, data, context={}):
-    if not context:
+def _create_facturae_txt(self, cr, uid, data, context=None):
+    if context is None:
         context = {}
     pool = pooler.get_pool(cr.dbname)
     invoice_obj = pool.get('account.invoice')
@@ -129,8 +129,8 @@
 }
 
 
-def _get_invoices_month(self, cr, uid, data, context={}):
-    if not context:
+def _get_invoices_month(self, cr, uid, data, context=None):
+    if context is None:
         context = {}
     pool = pooler.get_pool(cr.dbname)
     invoice_obj = pool.get('account.invoice')
@@ -168,9 +168,9 @@
     return {'invoice_ids': invoice_ids}
 
 
-def _get_invoices_date(self, cr, uid, data, context={}):
+def _get_invoices_date(self, cr, uid, data, context=None):
     # invoice_ids.append(19)
-    if not context:
+    if context is None:
         context = {}
     pool = pooler.get_pool(cr.dbname)
     invoice_obj = pool.get('account.invoice')

=== modified file 'l10n_mx_facturae/wizard/wizard_invoice_facturae_txt_v6.py'
--- l10n_mx_facturae/wizard/wizard_invoice_facturae_txt_v6.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/wizard_invoice_facturae_txt_v6.py	2013-12-16 18:29:27 +0000
@@ -104,7 +104,7 @@
         invoice_ids = []
         data = self.read(cr, uid, ids, context=context)[0]
         # invoice_ids.append(19)
-        if not context:
+        if context is None:
             context = {}
         invoice_obj = self.pool.get('account.invoice')
         if data['invoice_ids']:
@@ -142,7 +142,7 @@
     def get_invoices_month(self, cr, uid, ids, context=None):
         invoice_ids = []
         data = self.read(cr, uid, ids, context=context)[0]
-        if not context:
+        if context is None:
             context = {}
         invoice_obj = self.pool.get('account.invoice')
         invoice_ids = data['invoice_ids']
@@ -181,7 +181,7 @@
     def create_facturae_txt(self, cr, uid, ids, context=None):
         obj_model = self.pool.get('ir.model.data')
         data = self.read(cr, uid, ids, context=context)[0]
-        if not context:
+        if context is None:
             context = {}
         invoice_obj = self.pool.get('account.invoice')
         invoice_ids = data['invoice_ids']

=== modified file 'l10n_mx_facturae/wizard/wizard_invoice_facturae_xml.py'
--- l10n_mx_facturae/wizard/wizard_invoice_facturae_xml.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/wizard_invoice_facturae_xml.py	2013-12-16 18:29:27 +0000
@@ -43,8 +43,8 @@
 }
 
 
-def _get_invoice_facturae_xml(self, cr, uid, data, context={}):
-    if not context:
+def _get_invoice_facturae_xml(self, cr, uid, data, context=None):
+    if context is None:
         context = {}
     # context.update( {'date': data['form']['date']} )
     pool = pooler.get_pool(cr.dbname)

=== modified file 'l10n_mx_facturae/wizard/wizard_invoice_facturae_xml_v6.py'
--- l10n_mx_facturae/wizard/wizard_invoice_facturae_xml_v6.py	2013-05-10 16:24:03 +0000
+++ l10n_mx_facturae/wizard/wizard_invoice_facturae_xml_v6.py	2013-12-16 18:29:27 +0000
@@ -61,7 +61,7 @@
     }
 
     def _get_invoice_facturae_xml(self, cr, uid, data, context=None):
-        if not context:
+        if context is None:
             context = {}
         invoice_obj = self.pool.get('account.invoice')
         ids = data['active_ids']

=== modified file 'l10n_mx_facturae_cbb/invoice.py'
--- l10n_mx_facturae_cbb/invoice.py	2013-05-10 17:00:10 +0000
+++ l10n_mx_facturae_cbb/invoice.py	2013-12-16 18:29:27 +0000
@@ -41,9 +41,8 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_fname_invoice(self, cr, uid, ids, field_names=None, arg=False,
-        context={}):
-        if not context:
+    def _get_fname_invoice(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         sequence_obj = self.pool.get('ir.sequence')
@@ -87,7 +86,7 @@
         #'date_invoice': lambda *a: time.strftime('%Y-%m-%d %H:%M:%S'),
     }
 
-    def create_report(self, cr, uid, res_ids, report_name=False, file_name=False):
+    def create_report(self, cr, uid, res_ids, report_name=False, file_name=False, context=None):
         """
         @param report_name : Name of report with the name of object more type
             of report
@@ -95,6 +94,8 @@
             name of report that is 'openerp___facturae__' more six random
             characters for no files duplicate
         """
+        if context is None:
+            context = {}
         if not report_name or not res_ids:
             return (False, Exception('Report name and Resources ids are required !!!'))
         # try:
@@ -109,11 +110,10 @@
             # return (False,str(e))
         return (True, ret_file_name)
 
-    def create_report_pdf(self, cr, uid, ids, context={}):
-        if not context:
+    def create_report_pdf(self, cr, uid, ids, context=None):
+        if context is None:
             context = {}
-        id = ids[0]
-
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         (fileno, fname) = tempfile.mkstemp(
             '.pdf', 'openerp_' + (False or '') + '__facturae__')
         os.close(fileno)

=== modified file 'l10n_mx_facturae_cbb/ir_attachment_facturae.py'
--- l10n_mx_facturae_cbb/ir_attachment_facturae.py	2013-05-10 17:00:10 +0000
+++ l10n_mx_facturae_cbb/ir_attachment_facturae.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,8 @@
     _inherit = 'ir.attachment.facturae.mx'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = super(ir_attachment_facturae_mx, self)._get_type(
             cr, uid, ids, context=context)
         types.extend([

=== modified file 'l10n_mx_facturae_cbb/ir_sequence_approval.py'
--- l10n_mx_facturae_cbb/ir_sequence_approval.py	2013-05-10 17:00:10 +0000
+++ l10n_mx_facturae_cbb/ir_sequence_approval.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,8 @@
     _inherit = 'ir.sequence.approval'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = super(ir_sequence_approval, self)._get_type(
             cr, uid, ids, context=context)
         types.extend([

=== modified file 'l10n_mx_facturae_cer/invoice.py'
--- l10n_mx_facturae_cer/invoice.py	2013-05-10 17:27:23 +0000
+++ l10n_mx_facturae_cer/invoice.py	2013-12-16 18:29:27 +0000
@@ -33,9 +33,8 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_invoice_certificate(self, cr, uid, ids, field_names=None,
-        arg=False, context={}):
-        if not context:
+    def _get_invoice_certificate(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         company_obj = self.pool.get('res.company')
         certificate_obj = self.pool.get('res.company.facturae.certificate')

=== modified file 'l10n_mx_facturae_cer/res_company.py'
--- l10n_mx_facturae_cer/res_company.py	2013-12-04 17:13:45 +0000
+++ l10n_mx_facturae_cer/res_company.py	2013-12-16 18:29:27 +0000
@@ -85,6 +85,9 @@
     }
 
     def get_certificate_info(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         certificate = self.browse(cr, uid, ids, context=context)[0]
         cer_der_b64str = certificate.certificate_file
         key_der_b64str = certificate.certificate_key_file
@@ -133,6 +136,7 @@
                 fname_key_der, fname_tmp, fname_password, type_der='key')
             key_pem_b64 = base64.encodestring(key_pem)
 
+            # date_fmt_return='%Y-%m-%d %H:%M:%S'
             date_fmt_return = '%Y-%m-%d'
             serial = False
             try:
@@ -221,8 +225,9 @@
 class res_company(osv.Model):
     _inherit = 'res.company'
 
-    def ____get_current_certificate(self, cr, uid, ids, field_names=None,
-        arg=False, context={}):
+    def ____get_current_certificate(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
+            context = {}
         if not field_names:
             field_names = []
         res = {}
@@ -250,9 +255,8 @@
                         res[company.id][f] = certificate_id
         return res
 
-    def _get_current_certificate(self, cr, uid, ids, field_names=False,
-        arg=False, context={}):
-        if not context:
+    def _get_current_certificate(self, cr, uid, ids, field_names=False, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}.fromkeys(ids, False)
         certificate_obj = self.pool.get('res.company.facturae.certificate')
@@ -280,7 +284,7 @@
         return res
 
     """
-    def copy(self, cr, uid, id, default={}, context={}, done_list=[], local=False):
+    def copy(self, cr, uid, id, default={}, context=None, done_list=[], local=False):
         if not default:
             default = {}
         default = default.copy()

=== modified file 'l10n_mx_facturae_pac/invoice.py'
--- l10n_mx_facturae_pac/invoice.py	2013-11-08 23:34:19 +0000
+++ l10n_mx_facturae_pac/invoice.py	2013-12-16 18:29:27 +0000
@@ -44,7 +44,10 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context={}):
+    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         datas = super(account_invoice, self)._get_facturae_invoice_dict_data(
             cr, uid, ids, context=context)
         ir_seq_app_obj = self.pool.get('ir.sequence.approval')
@@ -103,9 +106,10 @@
                 data.update(dict({'cfdi:Comprobante': dict_comprobante}))
         return datas
 
-    def _get_facturae_invoice_xml_data(self, cr, uid, ids, context={}):
-        if not context:
+    def _get_facturae_invoice_xml_data(self, cr, uid, ids, context=None):
+        if context is None:
             context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         ir_seq_app_obj = self.pool.get('ir.sequence.approval')
         invoice = self.browse(cr, uid, ids[0], context=context)
         sequence_app_id = ir_seq_app_obj.search(cr, uid, [(

=== modified file 'l10n_mx_facturae_pac_sf/invoice.py'
--- l10n_mx_facturae_pac_sf/invoice.py	2013-08-25 09:18:53 +0000
+++ l10n_mx_facturae_pac_sf/invoice.py	2013-12-16 18:29:27 +0000
@@ -69,12 +69,13 @@
                                          help='Folio used in the electronic invoice'),
     }
 
-    def cfdi_data_write(self, cr, uid, ids, cfdi_data, context={}):
+    def cfdi_data_write(self, cr, uid, ids, cfdi_data, context=None):
         """
         @params cfdi_data : * TODO
         """
-        if not context:
+        if context is None:
             context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         attachment_obj = self.pool.get('ir.attachment')
         cfdi_xml = cfdi_data.pop('cfdi_xml')
         if cfdi_xml:
@@ -134,11 +135,11 @@
         return super(account_invoice, self).action_cancel_draft(cr, uid, ids, args)
     """
 
-    def _get_file(self, cr, uid, inv_ids, context={}):
-        if not context:
+    def _get_file(self, cr, uid, inv_ids, context=None):
+        if context is None:
             context = {}
-        id = inv_ids[0]
-        invoice = self.browse(cr, uid, [id], context=context)[0]
+        ids = isinstance(ids, (int, long)) and [ids] or ids
+        invoice = self.browse(cr, uid, ids, context=context)[0]
         fname_invoice = invoice.fname_invoice and invoice.fname_invoice + \
             '.xml' or ''
         aids = self.pool.get('ir.attachment').search(cr, uid, [(
@@ -190,11 +191,13 @@
         parent_node.appendChild(new_node)
         return new_node
 
-    def add_addenta_xml(self, cr, ids, xml_res_str=None, comprobante=None, context={}):
+    def add_addenta_xml(self, cr, ids, xml_res_str=None, comprobante=None, context=None):
         """
          @params xml_res_str : File XML
          @params comprobante : Name to the Node that contain the information the XML
         """
+        if context is None:
+            context = {}
         if xml_res_str:
             node_Addenda = xml_res_str.getElementsByTagName('cfdi:Addenda')
             if len(node_Addenda) == 0:
@@ -230,6 +233,8 @@
         return xml_res_str
 
     def _get_type_sequence(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         ir_seq_app_obj = self.pool.get('ir.sequence.approval')
         invoice = self.browse(cr, uid, ids[0], context=context)
         sequence_app_id = ir_seq_app_obj.search(cr, uid, [(
@@ -245,6 +250,8 @@
         return comprobante
 
     def _get_time_zone(self, cr, uid, invoice_id, context=None):
+        if context is None:
+            context = {}
         res_users_obj = self.pool.get('res.users')
         userstz = res_users_obj.browse(cr, uid, [uid])[0].partner_id.tz
         a = 0
@@ -264,7 +271,9 @@
                 timezone_present + timezone_original)*-1)
         return a
     
-    def _get_file_cancel(self, cr, uid, inv_ids, context={}):
+    def _get_file_cancel(self, cr, uid, inv_ids, context=None):
+        if context is None:
+            context = {}
         inv_ids = inv_ids[0]
         atta_obj = self.pool.get('ir.attachment')
         atta_id = atta_obj.search(cr, uid, [('res_id', '=', inv_ids), (
@@ -279,16 +288,18 @@
                 "This invoice hasn't stamped, so that not possible cancel."))
         return {'file': inv_xml}
 
-    def write_cfd_data(self, cr, uid, ids, cfd_datas, context={}):
+    def write_cfd_data(self, cr, uid, ids, cfd_datas, context=None):
         """
         @param cfd_datas : Dictionary with data that is used in facturae CFDI
         """
+        if context is None:
+            context = {}
         if not cfd_datas:
             cfd_datas = {}
         comprobante = self._get_type_sequence(cr, uid, ids, context=context)
         # obtener cfd_data con varios ids
         # for id in ids:
-        id = ids[0]
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         if True:
             data = {}
             cfd_data = cfd_datas
@@ -303,5 +314,5 @@
                 'sello': sello,
                 'cadena_original': cadena_original,
             }
-            self.write(cr, uid, [id], data, context=context)
+            self.write(cr, uid, ids, data, context=context)
         return True

=== modified file 'l10n_mx_facturae_pac_sf/ir_attachment_facturae.py'
--- l10n_mx_facturae_pac_sf/ir_attachment_facturae.py	2013-12-03 00:41:19 +0000
+++ l10n_mx_facturae_pac_sf/ir_attachment_facturae.py	2013-12-16 18:29:27 +0000
@@ -59,6 +59,8 @@
     _inherit = 'ir.attachment.facturae.mx'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = super(ir_attachment_facturae_mx, self)._get_type(
             cr, uid, ids, context=context)
         types.extend([
@@ -86,6 +88,8 @@
     }
     
     def sf_cancel(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         msg = ''
         certificate_obj = self.pool.get('res.company.facturae.certificate')
         pac_params_obj = self.pool.get('params.pac')
@@ -155,10 +159,12 @@
                 msg = _('Not found information of webservices of PAC, verify that the configuration of PAC is correct')
         return {'message': msg, 'status_uuid': status_uuid, 'status': status}
     
-    def _upload_ws_file(self, cr, uid, ids, fdata=None, context={}):
+    def _upload_ws_file(self, cr, uid, ids, fdata=None, context=None):
         """
         @params fdata : File.xml codification in base64
         """
+        if context is None:
+            context = {}
         invoice_obj = self.pool.get('account.invoice')
         pac_params_obj = invoice_obj.pool.get('params.pac')
         for ir_attachment_facturae_mx_id in self.browse(cr, uid, ids, context=context):

=== modified file 'l10n_mx_facturae_pac_sf/wizard/wizard_cancel_invoice_pac_sf_v6.py'
--- l10n_mx_facturae_pac_sf/wizard/wizard_cancel_invoice_pac_sf_v6.py	2013-05-30 21:28:32 +0000
+++ l10n_mx_facturae_pac_sf/wizard/wizard_cancel_invoice_pac_sf_v6.py	2013-12-16 18:29:27 +0000
@@ -51,16 +51,20 @@
 class wizard_cancel_invoice_pac_sf(osv.TransientModel):
     _name = 'wizard.cancel.invoice.pac.sf'
 
-    def _get_cancel_invoice_id(self, cr, uid, data, context={}):
+    def _get_cancel_invoice_id(self, cr, uid, data, context=None):
         """
         @params data : Dictionary with information of the user, and active ids
         """
+        if context is None:
+            context = {}
         res = {}
         invoice_obj = self.pool.get('account.invoice')
         res = invoice_obj._get_file_cancel(cr, uid, data['active_ids'])
         return res['file']
 
-    def upload_cancel_to_pac(self, cr, uid, ids, context={}):
+    def upload_cancel_to_pac(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         res = {}
         invoice_obj = self.pool.get('account.invoice')
         res = invoice_obj.sf_cancel(cr, uid, context[

=== modified file 'l10n_mx_facturae_pac_sf/wizard/wizard_export_invoice_pac_sf_v6.py'
--- l10n_mx_facturae_pac_sf/wizard/wizard_export_invoice_pac_sf_v6.py	2013-05-15 16:09:43 +0000
+++ l10n_mx_facturae_pac_sf/wizard/wizard_export_invoice_pac_sf_v6.py	2013-12-16 18:29:27 +0000
@@ -47,17 +47,21 @@
 class wizard_export_invoice_pac_sf_v6(osv.TransientModel):
     _name = 'wizard.export.invoice.pac.sf.v6'
 
-    def _get_invoice_id(self, cr, uid, data, context={}):
+    def _get_invoice_id(self, cr, uid, data, context=None):
         """
         @params data : Dictionary with information of the user, and active ids
         """
+        if context is None:
+            context = {}
         res = {}
         invoice_obj = self.pool.get('account.invoice')
         res = invoice_obj._get_file(cr, uid, data['active_ids'])
         file_xml = res['file']
         return file_xml
 
-    def upload_to_pac(self, cr, uid, ids, context={}):
+    def upload_to_pac(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         res = {}
         invoice_obj = self.pool.get('account.invoice')
         res = invoice_obj._upload_ws_file(cr, uid, context['active_ids'])

=== modified file 'l10n_mx_facturae_report/report/invoice_facturae_html.py'
--- l10n_mx_facturae_report/report/invoice_facturae_html.py	2013-12-02 23:02:06 +0000
+++ l10n_mx_facturae_report/report/invoice_facturae_html.py	2013-12-16 18:29:27 +0000
@@ -37,7 +37,9 @@
 import datetime
 
 class invoice_facturae_html(report_sxw.rml_parse):
-    def __init__(self, cr, uid, name, context):
+    def __init__(self, cr, uid, name, context=None):
+        if context is None:
+            context = {}
         super(invoice_facturae_html, self).__init__(
             cr, uid, name, context=context)
         self.localcontext.update({
@@ -46,6 +48,8 @@
             'split_string': self._split_string,
             'company_address': self._company_address,
             'subcompany_address': self._subcompany_address,
+            'get_invoice_sequence': self._get_invoice_sequence,
+            'get_approval': self._get_approval,
             'get_taxes': self._get_taxes,
             'get_taxes_ret': self._get_taxes_ret,
             'float': float,
@@ -53,6 +57,7 @@
             'get_data_partner': self._get_data_partner,
             'get_sum_total': self._get_sum_total,
             'has_disc': self._has_disc,
+            'get_data_certificate': self._get_data_certificate,
             'get_text_promissory' : self._get_text_promissory,
         })
         self.taxes = []
@@ -81,8 +86,35 @@
         except Exception, e:
             print "exception: %s" % (e)
             pass
+        try:
+            self._get_data_certificate(o.id)
+        except Exception, e:
+            print "exception: %s" % (e)
+            pass
         return ""
 
+    def _get_approval(self):
+        return self.approval
+
+    def _get_invoice_sequence(self):
+        return self.sequence
+
+    def _set_invoice_sequence_and_approval(self, invoice_id):
+        context = {}
+        pool = pooler.get_pool(self.cr.dbname)
+        invoice_obj = pool.get('account.invoice')
+        sequence_obj = pool.get('ir.sequence')
+        invoice = invoice_obj.browse(self.cr, self.uid, [
+                                     invoice_id], context=context)[0]
+        context.update({'number_work': invoice.number})
+        sequence = invoice.invoice_sequence_id or False
+        sequence_id = sequence and sequence.id or False
+        self.sequence = sequence
+        approval = sequence and sequence.approval_id or False
+        approval_id = approval and approval.id or False
+        self.approval = approval
+        return sequence, approval
+
     def _get_taxes(self):
         return self.taxes
 
@@ -133,6 +165,7 @@
         return self.invoice_data_dict
 
     def _get_facturae_data_dict(self, invoice):
+        self._set_invoice_sequence_and_approval(invoice.id)
         self.taxes = [
             tax for tax in invoice.tax_line if tax.tax_percent >= 0.0]
         self.taxes_ret = [
@@ -190,6 +223,24 @@
                 break
         return discount
 
+    def _get_data_certificate(self, invoice_id):
+        pool = pooler.get_pool(self.cr.dbname)
+        invoice_obj = pool.get('account.invoice')
+        pac_params_obj = self.pool.get('params.pac')
+        res={}
+        invoice = invoice_obj.browse(self.cr, self.uid, invoice_id)
+        pac_params_ids = pac_params_obj.search(self.cr, self.uid, [
+            ('method_type', '=', 'pac_sf_firmar'),
+            ('company_id', '=', invoice.company_id.id), 
+            ('active', '=', True)], limit=1)
+        pac_params_id = pac_params_ids and pac_params_ids[0] or False
+        if pac_params_id:
+            data_pac = pac_params_obj.browse(self.cr, self.uid, pac_params_id)
+            res.update({
+                'certificate_link' : data_pac.certificate_link or False,
+            })
+        return res
+        
     def _get_text_promissory(self, company, partner, address_emitter, invoice):
         text = ''
         context = {}

=== modified file 'l10n_mx_facturae_seq/invoice.py'
--- l10n_mx_facturae_seq/invoice.py	2013-06-10 10:26:14 +0000
+++ l10n_mx_facturae_seq/invoice.py	2013-12-16 18:29:27 +0000
@@ -33,9 +33,8 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_invoice_sequence(self, cr, uid, ids, field_names=None,
-        arg=False, context={}):
-        if not context:
+    def _get_invoice_sequence(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         for invoice in self.browse(cr, SUPERUSER_ID, ids):

=== modified file 'l10n_mx_facturae_seq/ir_sequence.py'
--- l10n_mx_facturae_seq/ir_sequence.py	2013-09-09 16:39:45 +0000
+++ l10n_mx_facturae_seq/ir_sequence.py	2013-12-16 18:29:27 +0000
@@ -36,6 +36,8 @@
     _rec_name = 'approval_number'
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = []
         return types
 
@@ -76,6 +78,8 @@
     ]
 
     def _check_numbers_range(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         approval = self.browse(cr, uid, ids[0], context=context)
         query = """SELECT approval_1.id AS id1, approval_2.id AS id2--\
             approval_1.number_start, approval_1.number_end, approval_2.\
@@ -108,16 +112,17 @@
 class ir_sequence(osv.Model):
     _inherit = 'ir.sequence'
 
-    def copy(self, cr, uid, id, default={}, context={}, done_list=[], local=False):
+    def copy(self, cr, uid, id, default={}, context=None, done_list=[], local=False):
+        if context is None:
+            context = {}
         if not default:
             default = {}
         default = default.copy()
         default['approval_ids'] = False
         return super(ir_sequence, self).copy(cr, uid, id, default, context=context)
 
-    def _get_current_approval(self, cr, uid, ids, field_names=None, arg=False,
-        context={}):
-        if not context:
+    def _get_current_approval(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         for id in ids:
@@ -146,7 +151,9 @@
         # s'expiring'
     }
 
-    def _check_sequence_number_diff(self, cr, uid, ids, context={}):
+    def _check_sequence_number_diff(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         # ahorita nadie manda a llamar esta funcion, ya que no existen los
         # warnings, como tal en OpenERP.
         sequence_number_diff_rate = 10
@@ -196,7 +203,7 @@
                     """
                     else:
                         _validation_sequence_number_diff(self, cr, uid, ids,
-                            context={}):
+                            context=context):
                         sequence_number_diff = sequence.approval_id.number_end \
                             - sequence.next_number
                         if sequence_number_diff <= sequence_number_diff_rate:

=== modified file 'l10n_mx_import_info/import_info.py'
--- l10n_mx_import_info/import_info.py	2013-05-15 16:56:26 +0000
+++ l10n_mx_import_info/import_info.py	2013-12-16 18:29:27 +0000
@@ -24,7 +24,9 @@
     _description = "Information about customs"
     _order = 'name asc'
 
-    def _get_audit(self, cr, uid, ids, field_name, arg, context):
+    def _get_audit(self, cr, uid, ids, field_name, arg, context=None):
+        if context is None:
+            context = {}
         result = {}
         prod_obj = self.pool.get('product.product')
         for i in ids:

=== modified file 'l10n_mx_import_info/product.py'
--- l10n_mx_import_info/product.py	2013-05-15 16:56:26 +0000
+++ l10n_mx_import_info/product.py	2013-12-16 18:29:27 +0000
@@ -29,10 +29,10 @@
     _name = 'product.import.info'
     _rec_name = 'import_id'
 
-    def _get_qtymoved(self, cr, uid, ids, field_name, arg, context):
+    def _get_qtymoved(self, cr, uid, ids, field_name, arg, context=None):
     # TODO ISAAC: Metodo para calcular el la cantidad de movimientos ya imputados a esta import info. Analizar desde cero, no contemplar lo que este escrito aqui.
     # Recordar quitar los TODO.
-        if not context:
+        if context is None:
             context = {}
         '''
         cr.execute("""
@@ -76,6 +76,8 @@
     }
 
     def _check_uom(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         for import_info in self.browse(cr, uid, ids, context=context):
             if import_info.uom_id and import_info.uom_id.category_id.id != \
                 import_info.product_id.uom_po_id.category_id.id:
@@ -99,6 +101,8 @@
 
         @return: return a dict that contains new values, and context
         """
+        if context is None:
+            context = {}
         res = {}
         if product_id:
             res = {'value': {'uom_id': self.pool.get('product.product').browse(
@@ -112,7 +116,9 @@
     """
     _inherit = 'product.product'
 
-    def _has_import(self, cr, uid, ids, field_name, arg, context):
+    def _has_import(self, cr, uid, ids, field_name, arg, context=None):
+        if context is None:
+            context = {}
         result = {}
         for i in ids:
             if len(self.browse(cr, uid, [i], context)[0].import_info_ids) != 0:

=== modified file 'l10n_mx_import_info/stock.py'
--- l10n_mx_import_info/stock.py	2013-08-05 06:39:25 +0000
+++ l10n_mx_import_info/stock.py	2013-12-16 18:29:27 +0000
@@ -31,6 +31,9 @@
     }
 
     def name_get(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         if not len(ids):
             return []
         # Avoiding use 000 in show name.
@@ -55,6 +58,8 @@
 # product_qty_p=[{'product_id':p.product_id.id,'qty':p.qty,'uom_id':p.uom_id.id}
 # for p in move.tracking_id.import_id.product_info_ids if
 # p.product_id.id==move.product_id.id]
+        if context is None:
+            context = {}
         product_import_info_obj = self.pool.get('product.import.info')
         product_uom_obj = self.pool.get('product.uom')
         for move in self.browse(cr, uid, ids, context=context):
@@ -100,6 +105,8 @@
         # print "move.state",move.state
         # print "import_id",import_id
         # print "move.product_id.pack_control",move.product_id.pack_control
+        if context is None:
+            context = {}
         if move.state != 'done':
             # purchase o sale, generate a stock.move with state confirmed or
             # draft, then not validate with these states.
@@ -132,6 +139,8 @@
         @param context: context arguments, like lang, time zone
         @return: return a dict that contains new values, and context
         """
+        if context is None:
+            context = {}
         return {
             'value': {},
             'context': {},
@@ -141,6 +150,8 @@
         """ Checks track lot with import information is assigned to stock move or not.
         @return: True or False
         """
+        if context is None:
+            context = {}
         for move in self.browse(cr, uid, ids, context=context):
             # Check if i need to verify the track for import info.
             ex = True
@@ -173,6 +184,8 @@
     
     def _prepare_invoice_line(self, cr, uid, group, picking, move_line, invoice_id,
         invoice_vals, context=None):
+        if context is None:
+            context = {}
         invoice_line_data = super(stock_picking, self)._prepare_invoice_line(cr, uid, group, picking, move_line, invoice_id,
             invoice_vals, context=context)
         invoice_line_data.update({

=== modified file 'l10n_mx_invoice_amount_to_text/invoice.py'
--- l10n_mx_invoice_amount_to_text/invoice.py	2013-05-15 17:02:24 +0000
+++ l10n_mx_invoice_amount_to_text/invoice.py	2013-12-16 18:29:27 +0000
@@ -36,9 +36,8 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_amount_to_text(self, cr, uid, ids, field_names=None, arg=False,
-        context={}):
-        if not context:
+    def _get_amount_to_text(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         for invoice in self.browse(cr, uid, ids, context=context):

=== modified file 'l10n_mx_invoice_datetime/invoice.py'
--- l10n_mx_invoice_datetime/invoice.py	2013-10-01 22:22:41 +0000
+++ l10n_mx_invoice_datetime/invoice.py	2013-12-16 18:29:27 +0000
@@ -42,7 +42,9 @@
 class account_payment_term(osv.Model):
     _inherit = "account.payment.term"
 
-    def compute(self, cr, uid, id, value, date_ref=False, context={}):
+    def compute(self, cr, uid, id, value, date_ref=False, context=None):
+        if context is None:
+            context = {}
         if date_ref:
             try:
                 date_ref = time.strftime('%Y-%m-%d', time.strptime(
@@ -57,9 +59,8 @@
     _inherit = 'account.invoice'
     #_order = 'invoice_datetime asc'
 
-    def _get_date_invoice_tz(self, cr, uid, ids, field_names=None, arg=False,
-        context={}):
-        if not context:
+    def _get_date_invoice_tz(self, cr, uid, ids, field_names=None, arg=False, context=None):
+        if context is None:
             context = {}
         res = {}
         if release.version >= '6':
@@ -95,6 +96,8 @@
     }
 
     def create(self, cr, uid, vals, context=None):
+        if context is None:
+            context = {}
         res = self.assigned_datetime(cr, uid, vals, context=context)
         if res:
             vals.update(res)
@@ -116,12 +119,16 @@
                                                             context=context)
         
     def copy(self, cr, uid, id, default=None, context=None):
+        if context is None:
+            context = {}
         if default is None:
             default = {}
         default.update({'invoice_datetime': False, 'date_invoice' : False})
         return super(account_invoice, self).copy(cr, uid, id, default, context)
     
     def _get_time_zone(self, cr, uid, invoice_id, context=None):
+        if context is None:
+            context = {}
         res_users_obj = self.pool.get('res.users')
         userstz = res_users_obj.browse(cr, uid, [uid])[0].partner_id.tz
         a = 0

=== modified file 'l10n_mx_invoice_discount/invoice.py'
--- l10n_mx_invoice_discount/invoice.py	2013-08-25 08:49:13 +0000
+++ l10n_mx_invoice_discount/invoice.py	2013-12-16 18:29:27 +0000
@@ -45,7 +45,9 @@
 class account_invoice(osv.Model):
     _inherit = 'account.invoice'
 
-    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context={}):
+    def _get_facturae_invoice_dict_data(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         invoice_data_parents = super(account_invoice, self).\
         _get_facturae_invoice_dict_data(cr, uid, ids, context)
         invoice = self.browse(cr, uid, ids)[0]
@@ -94,6 +96,8 @@
                 cr, uid, inv, compute_taxes, ait_obj)
 
     def button_reset_taxes(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         invoice = self.browse(cr, uid, ids)[0]
         invoice_line_obj = self.pool.get('account.invoice.line')
         sub_tot = 0

=== modified file 'l10n_mx_invoice_ftp/wizard/wizard_facturae_ftp.py'
--- l10n_mx_invoice_ftp/wizard/wizard_facturae_ftp.py	2013-05-15 17:30:14 +0000
+++ l10n_mx_invoice_ftp/wizard/wizard_facturae_ftp.py	2013-12-16 18:29:27 +0000
@@ -43,13 +43,18 @@
     _name = 'wizard.facturae.ftp'
 
     def invoice_ftp(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         ftp_id = False
         data = self.read(cr, uid, ids)[0]
         atta_obj = self.pool.get('ir.attachment')
         atta_obj.file_ftp(cr, uid, data['files'], context=context)
         return {}
 
-    def _get_files(self, cr, uid, context):
+    def _get_files(self, cr, uid, context=None):
+        if context is None:
+            context = {}
         atta_obj = self.pool.get('ir.attachment')
         atta_ids = atta_obj.search(cr, uid, [('res_id', 'in', context[
             'active_ids']), ('res_model', '=', context['active_model'])],

=== modified file 'l10n_mx_ir_attachment_facturae/invoice.py'
--- l10n_mx_ir_attachment_facturae/invoice.py	2013-09-25 21:19:59 +0000
+++ l10n_mx_ir_attachment_facturae/invoice.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,9 @@
     _inherit = 'account.invoice'
 
     def action_cancel(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         ir_attach_facturae_mx_obj = self.pool.get('ir.attachment.facturae.mx')
         inv_type_facturae = {
             'out_invoice': True,
@@ -50,6 +53,8 @@
         return res
 
     def create_ir_attachment_facturae(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         attach = ''
         ir_attach_obj = self.pool.get('ir.attachment.facturae.mx')
         mod_obj = self.pool.get('ir.model.data')

=== modified file 'l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py'
--- l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py	2013-12-09 19:21:48 +0000
+++ l10n_mx_ir_attachment_facturae/ir_attachment_facturae.py	2013-12-16 18:29:27 +0000
@@ -41,6 +41,8 @@
     _inherit = ['mail.thread', 'ir.needaction_mixin']
 
     def _get_type(self, cr, uid, ids=None, context=None):
+        if context is None:
+            context = {}
         types = []
         return types
 
@@ -124,6 +126,9 @@
         if msj:
             raise osv.except_osv(_('Warning'),_(msj))
         try:
+            if context is None:
+                context = {}
+            ids = isinstance(ids, (int, long)) and [ids] or ids
             invoice_obj = self.pool.get('account.invoice')
             attach = ''
             msj = ''
@@ -186,12 +191,16 @@
             return False
 
     def action_confirm(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
+        ids = isinstance(ids, (int, long)) and [ids] or ids
         return self.write(cr, uid, ids, {'state': 'confirmed'}, context=context)
 
     def signal_sign(self, cr, uid, ids, context=None):
         try:
             if context is None:
                 context = {}
+            ids = isinstance(ids, (int, long)) and [ids] or ids
             invoice_obj = self.pool.get('account.invoice')
             attachment_obj = self.pool.get('ir.attachment')
             attach = ''

=== modified file 'l10n_mx_ir_attachment_facturae/res_config.py'
--- l10n_mx_ir_attachment_facturae/res_config.py	2013-07-31 00:21:50 +0000
+++ l10n_mx_ir_attachment_facturae/res_config.py	2013-12-16 18:29:27 +0000
@@ -35,6 +35,8 @@
     }
 
     def get_default_email_tmp_id(self, cr, uid, fields, context=None):
+        if context is None:
+            context = {}
         email_tmp_id = False
         cr.execute(
             """ select max(id) as email_tmp_id from l10n_mx_email_config_settings """)

=== modified file 'l10n_mx_params_pac/params_pac.py'
--- l10n_mx_params_pac/params_pac.py	2013-06-10 00:32:36 +0000
+++ l10n_mx_params_pac/params_pac.py	2013-12-16 18:29:27 +0000
@@ -35,6 +35,8 @@
     _name = 'params.pac'
 
     def _get_method_type_selection(self, cr, uid, context=None):
+        if context is None:
+            context = {}
         # From module of PAC inherit this function and add new methods
         types = []
         return types

=== modified file 'l10n_mx_partner_address/partner_address.py'
--- l10n_mx_partner_address/partner_address.py	2013-12-07 02:01:16 +0000
+++ l10n_mx_partner_address/partner_address.py	2013-12-16 18:29:27 +0000
@@ -61,7 +61,9 @@
         id = ids and ids[0] or False
         return id
 
-    def fields_view_get_address(self, cr, uid, arch, context={}):
+    def fields_view_get_address(self, cr, uid, arch, context=None):
+        if context is None:
+            context = {}
         locality = _('Locality...')
         street = _('Street...')
         street2 = _('Colony...')

=== modified file 'l10n_mx_partner_address/res_company.py'
--- l10n_mx_partner_address/res_company.py	2013-09-20 18:20:54 +0000
+++ l10n_mx_partner_address/res_company.py	2013-12-16 18:29:27 +0000
@@ -32,6 +32,8 @@
     
     def _get_address_data(self, cr, uid, ids, field_names, arg, context=None):
         """ Read the 'address' functional fields. """
+        if context is None:
+            context = {}
         result = {}
         part_obj = self.pool.get('res.partner')
         for company in self.browse(cr, uid, ids, context=context):
@@ -46,6 +48,8 @@
 
     def _set_address_data(self, cr, uid, company_id, name, value, arg, context=None):
         """ Write the 'address' functional fields. """
+        if context is None:
+            context = {}
         company = self.browse(cr, uid, company_id, context=context)
         if company.partner_id:
             part_obj = self.pool.get('res.partner')

=== modified file 'l10n_mx_purchase_payment_method/purchase.py'
--- l10n_mx_purchase_payment_method/purchase.py	2013-05-17 21:03:54 +0000
+++ l10n_mx_purchase_payment_method/purchase.py	2013-12-16 18:29:27 +0000
@@ -60,6 +60,8 @@
     }
 
     def action_invoice_create(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         invoice_obj = self.pool.get('account.invoice')
         res = super(purchase_order, self).action_invoice_create(
             cr, uid, ids, context=context)

=== modified file 'l10n_mx_res_partner_bank/res_bank.py'
--- l10n_mx_res_partner_bank/res_bank.py	2013-06-05 20:51:07 +0000
+++ l10n_mx_res_partner_bank/res_bank.py	2013-12-16 18:29:27 +0000
@@ -33,6 +33,8 @@
     _inherit = 'res.partner.bank'
 
     def _get_take_digits(self, cr, uid, ids, field, args, context=None):
+        if context is None:
+            context = {}
         result = {}
         res = ''
         n = -1

=== modified file 'l10n_mx_sale_payment_method/model/sale.py'
--- l10n_mx_sale_payment_method/model/sale.py	2013-08-25 08:01:46 +0000
+++ l10n_mx_sale_payment_method/model/sale.py	2013-12-16 18:29:27 +0000
@@ -53,7 +53,8 @@
 
     def _prepare_invoice(self, cr, uid, order, lines, context=None):            
         '''Overwrite this method to send the payment term new to invoice '''
-        
+        if context is None:
+            context = {}
         res = super(inherits_sale,self)._prepare_invoice(cr, uid, order,
                                                          lines, context)
         res.update({'pay_method_id':order.pay_method_id and\
@@ -68,7 +69,8 @@
     def onchange_partner_id(self, cr, uid, ids, part, context=None):                
 
         '''Overwrite this method to set payment term new to sale order '''
-
+        if context is None:
+            context = {}
         res = super(inherits_sale,self).onchange_partner_id(cr, uid, ids,
                                                             part, context)
         part = self.pool.get('res.partner').browse(cr, uid, part, context)

=== modified file 'l10n_mx_sale_payment_method/model/stock.py'
--- l10n_mx_sale_payment_method/model/stock.py	2013-06-06 20:38:17 +0000
+++ l10n_mx_sale_payment_method/model/stock.py	2013-12-16 18:29:27 +0000
@@ -47,6 +47,8 @@
         """ Builds the dict containing the values for the invoice with the new
             payment term fields
         """
+        if context is None:
+            context = {}
         res = super(inherit_picking,self)._prepare_invoice(cr, uid, picking,
                                                            partner, inv_type,
                                                            journal_id,

=== modified file 'l10n_mx_upload_ftp/upload_ftp.py'
--- l10n_mx_upload_ftp/upload_ftp.py	2013-05-17 21:08:06 +0000
+++ l10n_mx_upload_ftp/upload_ftp.py	2013-12-16 18:29:27 +0000
@@ -50,7 +50,9 @@
         os.close(fileno)
         return fname
 
-    def file_ftp(self, cr, uid, ids, context={}):
+    def file_ftp(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         ftp_id = False
         ftp_obj = pooler.get_pool(cr.dbname).get('ftp.server')
         ftp_id = ftp_obj.search(cr, uid, [('name', '!=', False)], context=None)