Merge lp:~vauxoo/openerp-venezuela-localization/ovl70-fr-rev-kty into lp:~vauxoo/openerp-venezuela-localization/ovl70-fr-rev-hbto

[Katherine Zaoral (Vauxoo)]

[INFO] finalizada revision de los test de fiscal requirements

[lp:~vauxoo/openerp-venezuela-localization/ovl70-fr-rev-kty]

807. By Katherine Zaoral (Vauxoo) on 2012-12-10

[DEL] borrando mensajes de impresion de prueba

808. By Katherine Zaoral (Vauxoo) on 2012-12-10

[MERGE] ovl70-fr-rev-hbto

809. By Katherine Zaoral (Vauxoo) on 2012-12-10

[MERGE] localization 7.0 @ revision 766

810. By Katherine Zaoral (Vauxoo) on 2012-12-10

[INFO] revision de las pruebas con los test yml

811. By Katherine Zaoral (Vauxoo) on 2012-12-14

[FIX] 'res.partner.address': Removed the record for being an deprecated model. Test was adapted to integrate the info of the addresses in the 'res.partner' records.
---- ES_es
[FIX] 'res.partner.address': eliminado record por ser un modelo obsoleto, y adaptacion de la prueba para integrar la info de las direcciones en los records 'res.partner'.

812. By Katherine Zaoral (Vauxoo) on 2012-12-14

 [FIX] adaptacion de las vistas con openerp 7 (deprecated type)

813. By Katherine Zaoral (Vauxoo) on 2012-12-17

[FIX] campo 'tax_id' referenciado en test/fr_damaged.yml no existe, se referia a 'tax_code_id'

814. By Katherine Zaoral (Vauxoo) on 2012-12-17

[FIX] Adaptation for compatibility with OpenERP 7.0 wizard. Erased the line import wizard.
---- ES_es
[FIX] Adaptación para compatibilidad de los wizard con Openerp 7.0. Se borro la linea import wizard.

815. By Katherine Zaoral (Vauxoo) on 2012-12-17

[FIX] misuse of context in browse orm call that generate an error in mids.move_id.id (because this call overwrite the context with only ['move_id'] - OMG a list ).

816. By Katherine Zaoral (Vauxoo) on 2012-12-19

[IMP] se quito el metodo obsoleto _get_period y se uso period.find()

817. By Katherine Zaoral (Vauxoo) on 2013-01-08

[FIX] error in the _refund_cleanup_lines() function (impossible delet a browse record type)

818. By Katherine Zaoral (Vauxoo) on 2013-01-08

[FIX] yml: quitado referencias a registros que ya no existen de 'product' y redirigidos a otros que si.
[FIX] yml: se quitaron campos que ya no existen 'address_contact_id' y 'address_invoice_id'

819. By Katherine Zaoral (Vauxoo) on 2013-01-08

[MERGE] localization revno 770

820. By Katherine Zaoral (Vauxoo) on 2013-01-08

[FIX] eliminado metodo _refund_cleanup_lines(). sin razon de existir. Resulta que funciona perfectamente pero en 'wizard/account.invoice.refund.py:linea 255' se utiliza para limpiar tax_lines, entonces produce error ya que los campos en el metodo estan especificados para invoice_lines y no para tax_lines y producia keyErrror. Dejando el metood que viene en el core no hay problema porque el revisa todos los campos propios del modelo y apartir de ahi hace su accion

821. By Katherine Zaoral (Vauxoo) on 2013-01-08

[FIX] error al correr las pruebas porque read() devolvia una tupla en lugar de un browseable

822. By Katherine Zaoral (Vauxoo) on 2013-01-08

[FIX] record roto a 'product', enlazado a un record que si existe

823. By Katherine Zaoral (Vauxoo) on 2013-01-09

[FIX] cambio del mal uso de campo 'description' (no existe) por el campo 'name' del modelo 'account.inovice'.

824. By Katherine Zaoral (Vauxoo) on 2013-01-09

[MERGE] localization revno 771

825. By Katherine Zaoral (Vauxoo) on 2013-01-10

[FIX] solucionado error de indexacion en el test case 8.12 (IndexError: list index out of range)

826. By Katherine Zaoral (Vauxoo) on 2013-01-10

[FIX] eliminadas lineas relativas al campo 'sale_ids', no esta definido. este campo fue borrado de 'account_invoice' en fiscal requirements en la revion 499 de la localizacion

=== modified file 'l10n_ve_fiscal_requirements/model/invoice.py'
--- l10n_ve_fiscal_requirements/model/invoice.py	2012-12-04 21:56:01 +0000
+++ l10n_ve_fiscal_requirements/model/invoice.py	2013-01-11 18:35:30 +0000
@@ -79,22 +79,6 @@
     _constraints = [
           (_unique_invoice_per_partner, _('The Document you have been entering for this Partner has already been recorded'),['Control Number (nro_ctrl)','Reference (reference)']),
          ]
-     
-    def _refund_cleanup_lines(self, cr, uid, lines):
-        """
-        Method created to clean invoice lines
-        """
-        for line in lines:
-            del line['id']
-            del line['invoice_id']
-            #TODO Verify one more elegant way to do this
-            for field in ('company_id', 'partner_id', 'account_id', 'product_id',
-                          'uos_id', 'account_analytic_id', 'tax_code_id', 'base_code_id',
-                          "concept_id","tax_id"):
-                line[field] = line.get(field, False) and line[field][0]
-            if 'invoice_line_tax_id' in line:
-                line['invoice_line_tax_id'] = [(6,0, line.get('invoice_line_tax_id', [])) ]
-        return map(lambda x: (0,0,x), lines)
 
     def copy(self, cr, uid, id, default={}, context=None):
         if context is None:
@@ -108,4 +92,13 @@
 account_invoice()
 
 
+class account_invoice_tax(osv.osv):
+    _inherit = 'account.invoice.tax'
+    _columns = {
+        'tax_id': fields.many2one('account.tax', 'Tax', required=False, ondelete='set null', 
+        help="Tax relation to original tax, to be able to take off all data from invoices."),
+    }
+
+account_invoice_tax()
+
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'l10n_ve_fiscal_requirements/test/fr_damaged.yml'
--- l10n_ve_fiscal_requirements/test/fr_damaged.yml	2012-09-26 16:20:44 +0000
+++ l10n_ve_fiscal_requirements/test/fr_damaged.yml	2013-01-11 18:35:30 +0000
@@ -19,11 +19,9 @@
     company_id: base.main_company
     currency_id: base.EUR
     date_invoice: !eval time.strftime('%Y-%m-%d')
-    address_contact_id: base.res_partner_address_3
-    address_invoice_id: base.res_partner_address_3
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_3
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 20

=== modified file 'l10n_ve_fiscal_requirements/test/fr_purchase_test.yml'
--- l10n_ve_fiscal_requirements/test/fr_purchase_test.yml	2012-09-26 16:20:44 +0000
+++ l10n_ve_fiscal_requirements/test/fr_purchase_test.yml	2013-01-11 18:35:30 +0000
@@ -10,7 +10,7 @@
     date_invoice: !eval time.strftime('%Y-%m-%d')
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc1
+      - product_id: product.product_product_4
         quantity: 3
         uos_id: product.product_uom_unit
         price_unit: 3
@@ -18,7 +18,7 @@
         account_id: account.a_sale
         invoice_line_tax_id:
         - iva_purchase1
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_3
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 2

=== modified file 'l10n_ve_fiscal_requirements/test/fr_refund_note.yml'
--- l10n_ve_fiscal_requirements/test/fr_refund_note.yml	2012-09-26 16:20:44 +0000
+++ l10n_ve_fiscal_requirements/test/fr_refund_note.yml	2013-01-11 18:35:30 +0000
@@ -8,11 +8,9 @@
     company_id: base.main_company
     currency_id: base.EUR
     date_invoice: !eval time.strftime('%Y-%m-%d')
-    address_contact_id: base.res_partner_address_3
-    address_invoice_id: base.res_partner_address_3
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_3
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 20
@@ -105,10 +103,10 @@
     for line in rf_brw.invoice_line:
       for tax in line.invoice_line_tax_id:
         tax1.append(tax.id)
-    rf_brw = self.browse(cr, uid, ai_id[0])
+    rf_brw = ai_id and self.browse(cr, uid, ai_id[0])
     tax2 = []
     whc2 = []
-    for line in rf_brw.invoice_line:
+    for line in rf_brw and rf_brw.invoice_line:
       for tax in line.invoice_line_tax_id:
         tax2.append(tax.id)
     tax1 = set(tax1)
@@ -126,11 +124,9 @@
     company_id: base.main_company
     currency_id: base.EUR
     date_invoice: !eval time.strftime('%Y-%m-%d')
-    address_contact_id: base.res_partner_address_3
-    address_invoice_id: base.res_partner_address_3
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_3
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 20
@@ -232,11 +228,9 @@
     company_id: base.main_company
     currency_id: base.EUR
     date_invoice: !eval time.strftime('%Y-%m-%d')
-    address_contact_id: base.res_partner_address_3
-    address_invoice_id: base.res_partner_address_3
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_4
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 20

=== modified file 'l10n_ve_fiscal_requirements/test/fr_sale_test.yml'
--- l10n_ve_fiscal_requirements/test/fr_sale_test.yml	2012-09-26 16:20:44 +0000
+++ l10n_ve_fiscal_requirements/test/fr_sale_test.yml	2013-01-11 18:35:30 +0000
@@ -8,11 +8,9 @@
     company_id: base.main_company
     currency_id: base.EUR
     date_invoice: !eval time.strftime('%Y-%m-%d')
-    address_contact_id: base.res_partner_address_3
-    address_invoice_id: base.res_partner_address_3
     account_id: account.a_recv
     invoice_line:
-      - product_id: product.product_product_pc2
+      - product_id: product.product_product_3
         quantity: 5
         uos_id: product.product_uom_unit
         price_unit: 20

=== modified file 'l10n_ve_fiscal_requirements/test/fr_vat_test.yml'
--- l10n_ve_fiscal_requirements/test/fr_vat_test.yml	2012-09-26 21:08:18 +0000
+++ l10n_ve_fiscal_requirements/test/fr_vat_test.yml	2013-01-11 18:35:30 +0000
@@ -1,15 +1,6 @@
 -
   5. Fiscal requirements Vat tests
 -
-  5.1 Add address
--
-  !record {model: res.partner.address, id: test_fr_add01}:
-    type: invoice
-    street: Av Siempre Viva
-    phone: "(555) 5555555"
-    fax: "(555) 1111111"
-    email: [e-mail]
--
   5.2 I try to create a partner with wrong vat
 -
   !python {model: res.partner}: |
@@ -19,7 +10,11 @@
       'supplier': True,
       'customer': True,
       'vat': 'VEV333',
-      'address':[(6, 0, [ref("test_fr_add01")])]
+      'type': 'invoice',
+      'street': 'Av Siempre Viva',
+      'phone': "(555) 5555555",
+      'fax': "(555) 1111111",
+      'email': '[e-mail]',
       }
     part_id = False
     try:
@@ -46,7 +41,11 @@
       'supplier': True,
       'customer': True,
       'vat': 'VEJ112223330',
-      'address': [(6, 0, [ref("test_fr_add01")])],
+      'type': 'invoice',
+      'street': 'Av Siempre Viva',
+      'phone': "(555) 5555555",
+      'fax': "(555) 1111111",
+      'email': '[e-mail]',
       }
     part_id = self.create(cr, uid, part)
     assert part_id, 'Partner should be created'
@@ -60,7 +59,11 @@
       'supplier': True,
       'customer': True,
       'vat': 'VEJ112223330',
-      'address': [(6, 0, [ref("test_fr_add01")])],
+      'type': 'invoice',
+      'street': 'Av Siempre Viva',
+      'phone': "(555) 5555555",
+      'fax': "(555) 1111111",
+      'email': '[e-mail]',
       }
     try:
       rollback = cr.rollback
@@ -86,7 +89,11 @@
       'supplier': True,
       'customer': True,
       'vat': 'VEJ223334445',
-      'address': [(6, 0, [ref("test_fr_add01")])],
+      'type': 'invoice',
+      'street': 'Av Siempre Viva',
+      'phone': "(555) 5555555",
+      'fax': "(555) 1111111",
+      'email': '[e-mail]',
       }
     try:
       rollback = cr.rollback

=== modified file 'l10n_ve_fiscal_requirements/view/account_inv_refund_nctrl_view.xml'
--- l10n_ve_fiscal_requirements/view/account_inv_refund_nctrl_view.xml	2012-11-06 22:01:07 +0000
+++ l10n_ve_fiscal_requirements/view/account_inv_refund_nctrl_view.xml	2013-01-11 18:35:30 +0000
@@ -5,7 +5,6 @@
         <record id="view_account_inv_nctrl_refund" model="ir.ui.view">
             <field name="name">account.inv.nctrl.refund.form</field>
             <field name="model">account.invoice.refund</field>
-            <field name="type">form</field>
             <field name="inherit_id" ref="account.view_account_invoice_refund"/>
             <field name="arch" type="xml">
                 <xpath expr="//field[@name='period']" position="after">

=== modified file 'l10n_ve_fiscal_requirements/view/account_invoice_view.xml'
--- l10n_ve_fiscal_requirements/view/account_invoice_view.xml	2012-12-04 22:37:47 +0000
+++ l10n_ve_fiscal_requirements/view/account_invoice_view.xml	2013-01-11 18:35:30 +0000
@@ -10,7 +10,6 @@
         <record id="account_invoice_ext_view_form_loc_ve" model="ir.ui.view">
             <field name="name">invoice_extended.account.invoice.form.loc_ve</field>
             <field name="model">account.invoice</field>
-            <field name="type">form</field>
             <field name="inherit_id" ref="account.invoice_form"/>
             <field name="arch" type="xml">
                 <xpath expr="//field[@name='date_invoice']" position="replace">
@@ -91,7 +90,6 @@
         <record id="account_inv_ext_view_tree_crdr" model="ir.ui.view">
             <field name="name">invoice_extended.tree.crdr</field>
             <field name="model">account.invoice</field>
-            <field name="type">tree</field>
             <field name="inherit_id" ref="account.invoice_tree"/>
             <field name="arch" type="xml">
                 <xpath expr="//field[@name='origin']" position="after">
@@ -109,7 +107,6 @@
         <record id="account_invoice_ext_view_supp_form_loc_ve" model="ir.ui.view">
             <field name="name">invoice_extended.account.invoice.supplier.form.loc_ve</field>
             <field name="model">account.invoice</field>
-            <field name="type">form</field>
             <field name="inherit_id" ref="account.invoice_supplier_form"/>
             <field name="arch" type="xml">
                 <xpath expr="//field[@name='date_invoice']" position="replace">

=== modified file 'l10n_ve_fiscal_requirements/view/account_tax_view.xml'
--- l10n_ve_fiscal_requirements/view/account_tax_view.xml	2012-11-07 21:33:58 +0000
+++ l10n_ve_fiscal_requirements/view/account_tax_view.xml	2013-01-11 18:35:30 +0000
@@ -13,7 +13,6 @@
         <record id="account_tax_applicability_2" model="ir.ui.view">
             <field name="name">account.tax.applicability</field>
             <field name="model">account.tax</field>
-            <field name="type">form</field>
             <field name="inherit_id" ref="account.view_tax_form"/>
             <field name="arch" type="xml">
                 <xpath expr="//field[@name='active']" position="after">

=== modified file 'l10n_ve_fiscal_requirements/view/l10n_ut_view.xml'
--- l10n_ve_fiscal_requirements/view/l10n_ut_view.xml	2012-11-06 22:01:07 +0000
+++ l10n_ve_fiscal_requirements/view/l10n_ut_view.xml	2013-01-11 18:35:30 +0000
@@ -5,7 +5,6 @@
         <record id="view_l10n_ut_filter" model="ir.ui.view">
             <field name="name">l10n.ut.search</field>
             <field name="model">l10n.ut</field>
-            <field name="type">search</field>
             <field name="arch" type="xml">
                 <search string="Search Tax Unit" version="7.0">
                    <group col="10" colspan="4">
@@ -24,7 +23,6 @@
         <record model="ir.ui.view" id="view_l10n_ut_form">
             <field name="name">l10n.ut.form</field>
             <field name="model">l10n.ut</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Tax Unit" version="7.0">
                     <group col="6" colspan="4">
@@ -39,7 +37,6 @@
         <record model="ir.ui.view" id="view_l10n_ut_tree">
             <field name="name">l10n.ut.tree</field>
             <field name="model">l10n.ut</field>
-            <field name="type">tree</field>
             <field name="arch" type="xml">
                 <tree string="Tax Unit per date" editable="top">
                     <field name="name"/>

=== modified file 'l10n_ve_fiscal_requirements/view/partner_view.xml'
--- l10n_ve_fiscal_requirements/view/partner_view.xml	2012-12-09 23:00:04 +0000
+++ l10n_ve_fiscal_requirements/view/partner_view.xml	2013-01-11 18:35:30 +0000
@@ -5,7 +5,6 @@
     <record id="view_partner_form2" model="ir.ui.view">
         <field name="name">res.partner.vat.inherit2</field>
         <field name="model">res.partner</field>
-        <field name="type">form</field>
         <field name="inherit_id" ref="base.view_partner_form"/>
         <field name="arch" type="xml">
 

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py	2012-12-04 19:17:53 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_debit.py	2013-01-11 18:35:30 +0000
@@ -81,19 +81,6 @@
                 res['fields'][field]['selection'] = journal_select
         return res
 
-    def _get_period(self, cr, uid, context=None):
-        """
-        Return  default account period value
-        """
-        if context is None:
-            context = {}
-        account_period_obj = self.pool.get('account.period')
-        ids = account_period_obj.find(cr, uid, context=context)
-        period_id = False
-        if ids:
-            period_id = ids[0]
-        return period_id
-
     def _get_orig(self, cr, uid, inv, ref, context=None):
         """
         Return  default origin value
@@ -130,45 +117,39 @@
         date = False
         period = False
         description = False
-        company = res_users_obj.browse(cr, uid, uid, context=context).company_id
+
         journal_id = form.journal_id and  form.journal_id.id or False
         inv = inv_obj.browse(cr, uid, context.get('active_ids')[0], context=context)
+        #~ TODOK: no seria mejor ids=context.get(active_id)
+
+        company_id = inv.company_id.id
+        context.update({'company_id':company_id})
+
         if inv.state in ['draft', 'proforma2', 'cancel']:
             raise osv.except_osv(_('Error !'), _('Can not create a debit note from draft/proforma/cancel invoice.'))
         if inv.reconciled in ('cancel', 'modify'):
             raise osv.except_osv(_('Error !'), _('Can not create a debit note from invoice which is already reconciled, invoice should be unreconciled first. You can only Refund or Debit this invoice'))
         if inv.type not in ['in_invoice', 'out_invoice']:
             raise osv.except_osv(_('Error !'), _('Can not make a debit note on a refund invoice.'))
+
+        #Check for the form fields
+
         if form.period:
             period = form.period.id
         else:
             #Take period from the current date
-            #period = inv.period_id and inv.period_id.id or False
-            period = self._get_period(cr, uid, context)
+            period = self.pool.get('account.period').find(cr, uid, context=context)
+            period = period and period[0] or False
+            if not period:
+                raise osv.except_osv(_('No Pediod Defined'), \
+                                        _('You have been left empty the period field that automatically fill with the current period. However there is not period defined for the current company. Please check in Accounting/Configuration/Periods'))
+            self.write(cr, uid, ids, {'period': period }, context=context) 
 
         if not journal_id:
             journal_id = inv.journal_id.id
 
         if form.date:
             date = form.date
-            if not form.period.id:
-                    cr.execute("select name from ir_model_fields \
-                                    where model = 'account.period' \
-                                    and name = 'company_id'")
-                    result_query = cr.fetchone()
-                    if result_query:
-                        #in multi company mode
-                        cr.execute("""select p.id from account_fiscalyear y, account_period p where y.id=p.fiscalyear_id \
-                            and date(%s) between p.date_start AND p.date_stop and y.company_id = %s limit 1""", (date, company.id,))
-                    else:
-                        #in mono company mode
-                        cr.execute("""SELECT id
-                                from account_period where date(%s)
-                                between date_start AND  date_stop  \
-                                limit 1 """, (date,))
-                    res = cr.fetchone()
-                    if res:
-                        period = res[0]
         else:
             #Take current date
             #date = inv.date_invoice
@@ -178,10 +159,6 @@
         else:
             description = inv.name
 
-        if not period:
-            raise osv.except_osv(_('Data Insufficient !'), \
-                                    _('No Period found on Invoice!'))
-
         #we get original data of invoice to create a new invoice that is the copy of the original
         invoice = inv_obj.read(cr, uid, [inv.id],
                     ['name', 'type', 'number', 'reference',

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_debit_view.xml'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_debit_view.xml	2012-11-06 22:01:07 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_debit_view.xml	2013-01-11 18:35:30 +0000
@@ -5,7 +5,6 @@
         <record id="view_account_invoice_debit" model="ir.ui.view">
             <field name="name">account.invoice.debit.form</field>
             <field name="model">account.invoice.debit</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Debit Note" version="7.0">
                     <separator string="Debit Note Options" colspan="4"/>

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_parent_view.xml'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_parent_view.xml	2011-11-10 15:21:35 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_parent_view.xml	2013-01-11 18:35:30 +0000
@@ -5,7 +5,6 @@
         <record id="view_account_invoice_parent" model="ir.ui.view">
             <field name="name">account.invoice.parent.form</field>
             <field name="model">account.invoice.parent</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Debit Note">
                     <separator string="Debit Note Options" colspan="4"/>

=== modified file 'l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py'
--- l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2012-11-07 15:19:17 +0000
+++ l10n_ve_fiscal_requirements/wizard/account_invoice_refund.py	2013-01-11 18:35:30 +0000
@@ -79,21 +79,6 @@
                 res['fields'][field]['selection'] = journal_select
         return res
 
-    def _get_period(self, cr, uid, context={}):
-        """
-        Return  default account period value
-        """
-        period_id= False
-        if context.get('active_id',False):
-            invo_obj = self.pool.get('account.invoice')
-            invo_brw = invo_obj.browse(cr,uid,context.get('active_id'),{})
-            period_id = invo_brw and invo_brw.period_id and invo_brw.period_id.id
-        return period_id
-        
-        
-        
-        return period_id
-
     def _get_orig(self, cr, uid, inv, ref, context={}):
         """
         Return  default origin value
@@ -165,8 +150,15 @@
                     raise osv.except_osv(_('Error !'), _('Can not %s draft/proforma/cancel invoice.') % (mode))
                 if inv.reconciled and mode in ('cancel', 'modify'):
                     raise osv.except_osv(_('Error !'), _('Can not %s invoice which is already reconciled, invoice should be unreconciled first. You can only Refund this invoice') % (mode))
-                #Take period from the current date
-                period = form.get('period') and form.get('period')[0] or self._get_period(cr, uid, context)
+                period = form.get('period') and form.get('period')[0] or False
+                if not period:
+	                #Take period from the current date
+                    period = self.pool.get('account.period').find(cr, uid, context=context)
+                    period = period and period[0] or False
+                    if not period:
+                        raise osv.except_osv(_('No Pediod Defined'), \
+                                _('You have been left empty the period field that automatically fill with the current period. However there is not period defined for the current company. Please check in Accounting/Configuration/Periods'))
+                    self.write(cr, uid, ids, {'period': period }, context=context)
 
                 if not journal_brw:
                     journal_id = inv.journal_id.id
@@ -253,9 +245,10 @@
                                     'journal_id', 'period_id'], context=context)
                         invoice = invoice[0]
                         del invoice['id']
-                        invoice_lines = inv_line_obj.read(cr, uid, invoice['invoice_line'], context=context)
+                        invoice_lines = inv_line_obj.browse(cr, uid, invoice['invoice_line'], context=context)
+
                         invoice_lines = inv_obj._refund_cleanup_lines(cr, uid, invoice_lines)
-                        tax_lines = inv_tax_obj.read(cr, uid, invoice['tax_line'], context=context)
+                        tax_lines = inv_tax_obj.browse(cr, uid, invoice['tax_line'], context=context)
                         tax_lines = inv_obj._refund_cleanup_lines(cr, uid, tax_lines)
                         #Add origin value
                         orig = self._get_orig(cr, uid, inv, invoice['reference'], context)
@@ -283,12 +276,12 @@
                         
                         new_inv_brw = inv_obj.browse(cr,uid,created_inv[1],context=context)
                         inv_obj.write(cr,uid,created_inv[0],{'name':wzd_brw.description,'origin':new_inv_brw.origin},context=context)
-                        inv_obj.write(cr,uid,created_inv[1],{'origin':inv.origin,'description':''},context=context)
+                        inv_obj.write(cr,uid,created_inv[1],{'origin':inv.origin,'name':wzd_brw.description},context=context)
             if inv.type in ('out_invoice', 'out_refund'):
                 xml_id = 'action_invoice_tree3'
-                if hasattr(inv, 'sale_ids'):
-                    for i in inv.sale_ids:
-                        cr.execute('insert into sale_order_invoice_rel (order_id,invoice_id) values (%s,%s)', (i.id, refund_id[0]))
+                #~ if hasattr(inv, 'sale_ids'):
+                    #~ for i in inv.sale_ids:
+                        #~ cr.execute('insert into sale_order_invoice_rel (order_id,invoice_id) values (%s,%s)', (i.id, refund_id[0]))
             else:
                 xml_id = 'action_invoice_tree4'
             result = mod_obj.get_object_reference(cr, uid, 'account', xml_id)
@@ -300,11 +293,11 @@
             
             if wzd_brw.filter_refund == 'cancel':
                 orig = self._get_orig(cr, uid, inv, inv.reference, context)
-                inv_obj.write(cr,uid,created_inv[0],{'origin':orig,'description':wzd_brw.description},context=context)
+                inv_obj.write(cr,uid,created_inv[0],{'origin':orig,'name':wzd_brw.description},context=context)
             
             if wzd_brw.filter_refund == 'refund':
                 orig = self._get_orig(cr, uid, inv, inv.reference, context)
-                inv_obj.write(cr,uid,created_inv[0],{'origin':inv.origin,'description':wzd_brw.description},context=context)
+                inv_obj.write(cr,uid,created_inv[0],{'origin':inv.origin,'name':wzd_brw.description},context=context)
             return result
 
     def validate_total_payment_inv(self, cr, uid, ids, context=None):
@@ -354,7 +347,7 @@
                     if moveline.reconcile_partial_id:
                         rec += [moveline.reconcile_partial_id.id]
                 movelines = moveline_obj.search(cr, uid, [('|'),('reconcile_id','in',rec),('reconcile_partial_id','in',rec)],context=context)
-                for mids in moveline_obj.browse(cr, uid, movelines, ['move_id']):
+                for mids in moveline_obj.browse(cr, uid, movelines, context=context):
                     mid +=[mids.move_id.id]
                 voucherids = voucher_pool.search(cr, uid,[('move_id','in',mid)])
             if voucherids:

=== modified file 'l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.xml'
--- l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.xml	2012-09-26 21:07:02 +0000
+++ l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.xml	2013-01-11 18:35:30 +0000
@@ -4,7 +4,6 @@
         <record model="ir.ui.view" id="view_vat_search">
             <field name="name">search.info.partner.seniat.form</field>
             <field name="model">search.info.partner.seniat</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Wizard to search partner on SENIAT">
                     <group colspan="4">

=== modified file 'l10n_ve_fiscal_requirements/wizard/update_info_partner.xml'
--- l10n_ve_fiscal_requirements/wizard/update_info_partner.xml	2011-10-25 20:33:16 +0000
+++ l10n_ve_fiscal_requirements/wizard/update_info_partner.xml	2013-01-11 18:35:30 +0000
@@ -7,7 +7,6 @@
         <record id="view_wizard_update_info_partner" model="ir.ui.view">
             <field name="name">view.update.info.partner</field>
             <field name="model">update.info.partner</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="">
                     <separator string="Are you sure you want to do this?" colspan="4"/>

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl_view.xml'
--- l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl_view.xml	2012-04-11 21:48:15 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_invoice_nro_ctrl_view.xml	2013-01-11 18:35:30 +0000
@@ -4,7 +4,6 @@
         <record id="view_wizard_invoice_nro_ctrl" model="ir.ui.view">
             <field name="name">view.wizard.invoice.nro.ctrl</field>
             <field name="model">wizard.invoice.nro.ctrl</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Create an invoice to justify the damaged one">
                     <group colspan="8" col="8">

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl_view.xml'
--- l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl_view.xml	2012-02-09 21:33:03 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_nro_ctrl_view.xml	2013-01-11 18:35:30 +0000
@@ -4,7 +4,6 @@
         <record model="ir.ui.view" id="wizard_nro_ctrl_form">
             <field name="name">wizard.nro.ctrl.form</field>
             <field name="model">wiz.nroctrl</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Changing the Control Number">
                     <field name="name" />

=== modified file 'l10n_ve_fiscal_requirements/wizard/wizard_url_seniat_view.xml'
--- l10n_ve_fiscal_requirements/wizard/wizard_url_seniat_view.xml	2012-07-16 18:55:40 +0000
+++ l10n_ve_fiscal_requirements/wizard/wizard_url_seniat_view.xml	2013-01-11 18:35:30 +0000
@@ -4,7 +4,6 @@
         <record id="view_wizard_url_seniat" model="ir.ui.view">
             <field name="name">view.wizard.seniat.url</field>
             <field name="model">wizard.seniat.url</field>
-            <field name="type">form</field>
             <field name="arch" type="xml">
                 <form string="Update Url From Seniat">
                     <separator string="URL Seniat for Partner Information" colspan="4"/>