Merge lp:~rafael-valle-z/openerp-spain/openerp-spain into lp:~openerp-spain-team/openerp-spain/6.1

[Rafael Valle]

Creación de cuentas nuevas:

    472.0 Hacienda Pública, IVA soportado 0%
    477.0 Hacienda Pública. IVA repercutido 0%

Para la vinculación de estas cuentas con los impuestos:

    IVA Soportado exento (operaciones corrientes) con la 4720.0
    IVA Exento con la 4770.0

Creación de nuevos impuestos:

    IVA 21% Intracomunitario Adquisición de servicios
    IVA 21% Adquisición servicios extracomunitarios

Modificaciones de impuestos existentes:

    IVA 21% Inversión del sujeto pasivo
    IVA Soportado exento (operaciones corrientes)
    IVA 0% Intracomunitario

[lp:~rafael-valle-z/openerp-spain/openerp-spain]

351. By Rafael Valle on 2013-05-16

modificaciones en busqueda de periodos para permitir que no sean en un año natural

352. By Rafael Valle on 2013-05-16

modificaciones en busqueda de periodos para permitir que no sean en un año natural para modelo 340

[Pedro Manuel Baeza (pedro.baeza)]

Buenas, Rafael:

Muchas gracias por vuestro aporte. Revisando el diff, hay varias cuestiones que no me quedan claras, que paso a preguntarte:

- Ya existían dos impuestos "IVA Exento" e "IVA Soportado exento", ¿por qué eliminarlos y volverlos a incluir?
- Si el importe del IVA siempre es 0, ¿para qué son necesarias las cuentas 470.0 y 477.0?
- En este MP se incluye también una modificación en el módulo del modelo 340. Aunque Ignacio aún no la ha portado a la 6.1, es cuestión de poco tiempo y cambia totalmente la lógica de tratamiento de los periodos, por lo que no tiene mucho sentido, aunque bueno, tampoco molesta.
- Por último, sobre la inversión del sujeto pasivo, ¿por qué se le cambia el signo al impuesto del primero de los hijos? Realmente esa definición no vuelca nada en los impuestos, ya que no tiene establecido código de cuenta para el impuesto. ¿Hay algún otro cambio que no haya visto yo?

Un saludo.

review: Needs Information

[Rafael Valle (rafael-valle-z)]

Buenos Días!
Contesto sobre el correo.
Gracias de antemano.
Un saludo.

El 17/05/13 16:10, Pedro Manuel Baeza escribió:
> Review: Needs Information
>
> Buenas, Rafael:
>
> Muchas gracias por vuestro aporte. Revisando el diff, hay varias cuestiones que no me quedan claras, que paso a preguntarte:
>
> - Ya existían dos impuestos "IVA Exento" e "IVA Soportado exento", ¿por qué eliminarlos y volverlos a incluir?

No los hemos eliminado, sólo los modificamos para que tuvieran las
cuentas contables asociadas. Así como también hemos modificado el IVA 0%
intracomunitario.
> - Si el importe del IVA siempre es 0, ¿para qué son necesarias las cuentas 470.0 y 477.0?
No son necesarias, pero lo hemos añadido para estandarizar la definición
de los impuestos.
> - En este MP se incluye también una modificación en el módulo del modelo 340. Aunque Ignacio aún no la ha portado a la 6.1, es cuestión de poco tiempo y cambia totalmente la lógica de tratamiento de los periodos, por lo que no tiene mucho sentido, aunque bueno, tampoco molesta.
Ok
> - Por último, sobre la inversión del sujeto pasivo, ¿por qué se le cambia el signo al impuesto del primero de los hijos? Realmente esa definición no vuelca nada en los impuestos, ya que no tiene establecido código de cuenta para el impuesto. ¿Hay algún otro cambio que no haya visto yo?
Los cambios de la inversión del sujeto pasivo son los siguientes:

  *

    *IVA 21% Inversión del sujeto pasivo*: Cambio del código base cuenta
    y cambio del código base reintegro (1), así como cambio de los
    signos en todos.

      o

        IVA 21% Inversión del sujeto pasivo (1):

          +

            Código base cuenta y código base reintegro:

              #

                Nombre código de impuesto: ISPSOPORTADO - Inversión del
                sujeto pasivo SOPORTADO

              #

                Código: ISPSOPORTADO

              #

                Código padre: La propia compañía.

          +

            Signo código base: -1 (rectificativa)

      o

        IVA 21% Inversión del sujeto pasivo (2):

          +

            Signo código impuesto: -1

          +

            Signo código base: -1 (rectificativa)

      o

        IVA 21% Inversión del sujeto pasivo (3):

          +

            Signo código base: -1 (rectificativa)

          +

            Signo código impuesto -1 (rectificativa)

Tuvimos que modificarlos de esta forma, ya que anteriormente no nos
funcionaba de manera correcta.
> Un saludo.

[Ignacio Ibeas (www.acysos.com) (ignacio-acysos)]

Este merge tiene que ser revisado, porque modifica el Modelo 340 antiguo no el nuevo. Si se quita el la modificación sobre el 340 se podrá hacer el merge.

Saludos

review: Needs Fixing

[Pedro Manuel Baeza (pedro.baeza)]

Los cambios contemplados aquí que no están ya incluidos, se han plasmado en la refactorización del l10n_es:

https://code.launchpad.net/~pedro.baeza/openerp-spain/7.0-l10n_es-refactorized/+merge/196807

Por favor, revisad dicho MP en su lugar.

Un saludo.

review: Disapprove

Unmerged revisions

352. By Rafael Valle on 2013-05-16

modificaciones en busqueda de periodos para permitir que no sean en un año natural para modelo 340

351. By Rafael Valle on 2013-05-16

modificaciones en busqueda de periodos para permitir que no sean en un año natural

350. By Rafael Valle on 2013-04-30

creacion y modificacion de nuevas cuentas

=== modified file 'l10n_es/fiscal_templates.xml'
--- l10n_es/fiscal_templates.xml	2012-09-14 22:11:17 +0000
+++ l10n_es/fiscal_templates.xml	2013-05-16 11:38:23 +0000
@@ -335,6 +335,31 @@
             <field name="tax_dest_id" ref="iva_IO" />
         </record>
 
+
+        <record id="fptt_intra_ventas_22" model="account.fiscal.position.tax.template">
+            <field name="position_id" ref="fp_intra" />
+            <field name="tax_src_id" ref="iva_rep_21" />
+            <field name="tax_dest_id" ref="iva_rep_21" />
+        </record>
+
+        <record id="fptt_intra_ventas_23" model="account.fiscal.position.tax.template">
+            <field name="position_id" ref="fp_intra" />
+            <field name="tax_src_id" ref="iva_rep_21" />
+            <field name="tax_dest_id" ref="iva_sop_21" />
+        </record>
+
+        <record id="fptt_intra_ventas_24" model="account.fiscal.position.tax.template">
+            <field name="position_id" ref="fp_intra" />
+            <field name="tax_src_id" ref="iva_sop_21" />
+            <field name="tax_dest_id" ref="iva_sop_21" />
+        </record>
+
+        <record id="fptt_intra_ventas_25" model="account.fiscal.position.tax.template">
+            <field name="position_id" ref="fp_intra" />
+            <field name="tax_src_id" ref="iva_sop_21" />
+            <field name="tax_dest_id" ref="iva_rep_21" />
+        </record>
+
     <!-- Recargo de equivalencia fiscal.position.tax.template-->
 
         <record id="fptt_recargo_4" model="account.fiscal.position.tax.template">

=== modified file 'l10n_es/taxes_data.xml'
--- l10n_es/taxes_data.xml	2013-04-22 09:05:50 +0000
+++ l10n_es/taxes_data.xml	2013-05-16 11:38:23 +0000
@@ -58,35 +58,7 @@
             <field name="sign">1.0</field>
         </record>         	
 
-        <!-- IVA Soportado -->
-        <record id="iva_sop_ex" model="account.tax.template">
-            <field name="chart_template_id" ref="l10nES_chart_template"/>
-            <field name="name">IVA Soportado exento (operaciones corrientes)</field>
-            <field eval="0.0" name="amount"/>
-            <field name="type">percent</field>
-            <field eval="1.0" name="tax_sign"/>
-            <field eval="1.0" name="base_sign"/>
-            <field name="base_code_id" ref="base_sop_ex"/>
-            <field name="ref_base_code_id" ref="base_sop_ex"/>
-            <field eval="-1.0" name="ref_tax_sign"/>
-            <field eval="-1.0" name="ref_base_sign"/>
-            <field name="type_tax_use">purchase</field>
-        </record>
-
-        <!-- IVA REPERCUTIDO-->
-        <record id="iva_rep_ex" model="account.tax.template">
-            <field name="chart_template_id" ref="l10nES_chart_template"/>
-            <field name="name">IVA Exento</field>
-            <field eval="0.0" name="amount"/>
-            <field name="type">percent</field>
-            <field name="base_code_id" ref="base_rep_ex"/>
-            <field name="ref_base_code_id" ref="base_rep_ex"/>
-            <field eval="1.0" name="tax_sign"/>
-            <field eval="1.0" name="base_sign"/>
-            <field eval="-1.0" name="ref_tax_sign"/>
-            <field eval="-1.0" name="ref_base_sign"/>
-            <field name="type_tax_use">sale</field>
-        </record>
+        
 
     <!--Régimen general-->
 
@@ -998,6 +970,13 @@
             <field name="sign">1.0</field>
         </record>
 
+        <record id="base_extra_ipssoportado" model="account.tax.code.template">
+            <field name="name">Inversión del sujeto pasivo SOPORTADO</field>
+            <field name="code">ISPSOPORTADO</field>
+            <field name="parent_id" ref="vat_code_chart_root"/>
+            <field name="sign">1.0</field>
+        </record>
+
         <record id="iva_ded_15" model="account.tax.code.template">
             <field name="name">Recargo Equivalencia Ded. Cuota 1%</field>
             <field name="code">[15]</field>
@@ -1030,6 +1009,25 @@
 <!--Cuentas para IVA-->
 <!-- ********************************* -->
 
+        <record id="pgc_472000" model="account.account.template">
+            <field name="code">472000</field>
+            <field name="reconcile" eval="True"/>
+            <field name="parent_id" ref="pgc_472"/>
+            <field name="type">other</field>
+            <field name="name">Hacienda Pública, IVA soportado 0%</field>
+            <field name="user_type" ref="tax"/>
+        </record>
+
+
+		<record id="pgc_477000" model="account.account.template">
+            <field name="code">477000</field>
+            <field name="reconcile" eval="True"/>
+            <field name="parent_id" ref="pgc_4770"/>
+            <field name="type">other</field>
+            <field name="name">Hacienda Pública. IVA repercutido 0%</field>
+            <field name="user_type" ref="tax"/>
+        </record>
+
         <record id="pgc_472004" model="account.account.template">
             <field name="code">472004</field>
             <field name="reconcile" eval="True"/>
@@ -1268,6 +1266,42 @@
 <!-- Plantillas de Impuestos IVA       -->
 <!-- ********************************* -->
 
+
+
+        <!-- IVA Soportado -->
+        <record id="iva_sop_ex" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA Soportado exento (operaciones corrientes)</field>
+			<field name="account_paid_id" ref="pgc_472000"/>
+			<field name="account_collected_id" ref="pgc_472000"/>
+            <field eval="0.0" name="amount"/>
+            <field name="type">percent</field>
+            <field eval="1.0" name="tax_sign"/>
+            <field eval="1.0" name="base_sign"/>
+            <field name="base_code_id" ref="base_sop_ex"/>
+            <field name="ref_base_code_id" ref="base_sop_ex"/>
+            <field eval="-1.0" name="ref_tax_sign"/>
+            <field eval="-1.0" name="ref_base_sign"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+        <!-- IVA REPERCUTIDO-->
+        <record id="iva_rep_ex" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA Exento</field>
+            <field eval="0.0" name="amount"/>
+            <field name="type">percent</field>
+            <field name="account_paid_id" ref="pgc_477000"/>
+			<field name="account_collected_id" ref="pgc_477000"/>
+            <field name="base_code_id" ref="base_rep_ex"/>
+            <field name="ref_base_code_id" ref="base_rep_ex"/>
+            <field eval="1.0" name="tax_sign"/>
+            <field eval="1.0" name="base_sign"/>
+            <field eval="-1.0" name="ref_tax_sign"/>
+            <field eval="-1.0" name="ref_base_sign"/>
+            <field name="type_tax_use">sale</field>
+        </record>
+
 <!-- IVA Soportado -->
 
         <record id="iva_sop_4" model="account.tax.template">
@@ -1957,6 +1991,133 @@
             <field name="type_tax_use">purchase</field>
         </record>
 
+        <record id="iva_intra_servicios_21" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA 21% Intracomunitario Adquisición de servicios</field>
+            <field name="description">ADSERINTRA</field>
+            <field name="amount" eval="1.00"/>
+            <field name="type">percent</field>
+            <field name="child_depend" eval="1"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+
+		<record id="iva_ISP_soportado" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA Soportado Inversión sujeto pasivo</field>
+            <field name="description">IVA ISPS</field>
+            <field name="parent_id" ref="iva_intra_servicios_21"/>
+            <field name="amount" eval="0.00"/>
+            <field name="type">percent</field>
+            <field name="base_code_id" ref="base_extra_ipssoportado"/>
+            <field name="ref_base_code_id" ref="base_extra_ipssoportado"/>
+            <field name="base_sign" eval="1.0"/>
+            <field name="ref_base_sign" eval="-1.0"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+
+
+		<record id="iva_21_soportado_1" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">21% IVA Soportado (operaciones corrientes)(1)</field>
+            <field name="parent_id" ref="iva_intra_servicios_21"/>
+            <field name="amount" eval="0.21"/>
+			<field name="account_collected_id" ref="pgc_472021"/>
+            <field name="account_paid_id" ref="pgc_472021"/>
+            <field name="type">percent</field>
+            <field name="base_code_id" ref="iva_ded_22_21"/>
+			<field name="tax_code_id" ref="iva_ded_23_21"/>
+            <field name="ref_base_code_id" ref="iva_ded_22_21"/>
+			<field name="ref_tax_code_id" ref="iva_ded_23_21"/>
+            <field name="ref_base_sign" eval="-1.0"/>
+            <field name="ref_tax_sign" eval="-1.0"/>
+			<field name="tax_sign" eval="1.0"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+
+
+		<record id="iva_21_repercutido_1" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA 21% repercutido(1)</field>
+            <field name="parent_id" ref="iva_intra_servicios_21"/>
+            <field name="amount" eval="0.21"/>
+			<field name="account_collected_id" ref="pgc_477021"/>
+            <field name="account_paid_id" ref="pgc_477021"/>
+            <field name="type">percent</field>
+            <field name="base_code_id" ref="iva_dev_07_21"/>
+			<field name="tax_code_id" ref="iva_dev_09_21"/>
+            <field name="ref_base_code_id" ref="iva_dev_07_21"/>
+			<field name="ref_tax_code_id" ref="iva_dev_09_21"/>
+            <field name="ref_base_sign" eval="-1.0"/>
+			<field name="ref_tax_sign" eval="1.0"/>
+			<field name="tax_sign" eval="-1.0"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+
+        <record id="iva_adq_servicios_intra_21" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA 21% Adquisición servicios extracomunitarios</field>
+            <field name="description">ADSEREXTRA</field>
+            <field name="amount" eval="1.00"/>
+            <field name="type">percent</field>
+            <field name="child_depend" eval="1"/>
+            <field name="type_tax_use">purchase</field>
+            <field name="ref_base_sign" eval="-1.0"/>
+			<field name="ref_tax_sign" eval="-1.0"/>
+        </record>
+
+
+		<record id="iva_soportado_isp_2" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA SOPORTADO ISP(1)</field>
+            <field name="parent_id" ref="iva_adq_servicios_intra_21"/>
+            <field name="amount" eval="0.00"/>
+			<field name="account_collected_id" ref="pgc_472021"/>
+            <field name="account_paid_id" ref="pgc_472021"/>
+            <field name="type">percent</field>
+            <field name="base_code_id" ref="base_extra_ipssoportado"/>
+            <field name="ref_base_code_id" ref="base_extra_ipssoportado"/>
+            <field name="ref_base_sign" eval="-1.0"/>
+            <field name="type_tax_use">all</field>
+        </record>
+
+		<record id="iva_repercutido_isp_2" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">IVA 21% repercutido(2)</field>
+            <field name="parent_id" ref="iva_adq_servicios_intra_21"/>
+            <field name="amount" eval="-0.21"/>
+			<field name="base_code_id" ref="iva_dev_07_21"/>
+			<field name="tax_code_id" ref="iva_dev_09_21"/>
+            <field name="ref_base_code_id" ref="iva_dev_07_21"/>
+			<field name="ref_tax_code_id" ref="iva_dev_09_21"/>
+			<field name="account_collected_id" ref="pgc_477021"/>
+            <field name="account_paid_id" ref="pgc_477021"/>
+            <field name="type">percent</field>
+            <field name="tax_sign" eval="-1.0"/>
+			<field name="ref_base_sign" eval="-1.0"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
+
+
+		<record id="iva_repercutido_isp_1" model="account.tax.template">
+            <field name="chart_template_id" ref="l10nES_chart_template"/>
+            <field name="name">21% IVA Soportado (operaciones corrientes)(2)</field>
+            <field name="parent_id" ref="iva_adq_servicios_intra_21"/>
+            <field name="amount" eval="0.21"/>
+			<field name="base_code_id" ref="iva_ded_22_21"/>
+			<field name="tax_code_id" ref="iva_ded_23_21"/>
+            <field name="ref_base_code_id" ref="iva_ded_22_21"/>
+			<field name="ref_tax_code_id" ref="iva_ded_23_21"/>
+			<field name="account_collected_id" ref="pgc_472021"/>
+            <field name="account_paid_id" ref="pgc_472021"/>
+            <field name="type">percent</field>
+            <field name="ref_tax_sign" eval="-1.0"/>
+			<field name="ref_base_sign" eval="-1.0"/>
+            <field name="type_tax_use">purchase</field>
+        </record>
 
 		<record id="iva_ISP_compras_21" model="account.tax.template">
             <field name="chart_template_id" ref="l10nES_chart_template"/>
@@ -1980,7 +2141,7 @@
             <field name="base_code_id" ref="base_extra_44"/>
             <field name="ref_base_code_id" ref="base_extra_44"/>
             <field name="base_sign" eval="1.0"/>
-            <field name="tax_sign" eval="1.0"/>
+            <field name="tax_sign" eval="-1.0"/>
             <field name="ref_base_sign" eval="-1.0"/>
             <field name="ref_tax_sign" eval="-1.0"/>
             <field name="type_tax_use">purchase</field>

=== modified file 'l10n_es_aeat_mod340/wizard/calculate_mod340_records.py'
--- l10n_es_aeat_mod340/wizard/calculate_mod340_records.py	2012-09-26 18:05:45 +0000
+++ l10n_es_aeat_mod340/wizard/calculate_mod340_records.py	2013-05-16 11:38:23 +0000
@@ -72,19 +72,48 @@
             
             mod = mod340.period
 
-            if mod >= '01' and mod <= '12':
-                fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, mod), '%Y-%m-%d')
-                fecha_fin = fecha_ini + relativedelta(months=+1, days=-1)
-
-            if mod in ('1T', '2T', '3T', '4T'):
-                month = ( ( int(mod[0])-1 ) * 3 ) + 1
-                fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, month), '%Y-%m-%d')
-                fecha_fin = fecha_ini + relativedelta(months=+3, days=-1)
-                mod = '%02d' % month
-
-            code = '340'+dec_year+''+mod+'0001'
-            
-            account_period_id = self.pool.get('account.period').search(cr,uid,[('date_start','=',fecha_ini),('date_stop','=',fecha_fin)])
+            dec_year_end=mod340.fiscalyear_id.date_stop.split('-')[0]
+
+            if dec_year != dec_year_end:
+
+                if mod >= '01' and mod <= '12':
+                    fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, mod), '%Y-%m-%d')
+                    fecha_fin = fecha_ini + relativedelta(months=+1, days=-1)
+                    code = '340'+dec_year+''+mod+'0001'
+                    account_period_id = self.pool.get('account.period').search(cr,uid,[('date_start','=',fecha_ini),('date_stop','=',fecha_fin)])
+            
+                    if not account_period_id:
+                        fecha_ini = datetime.strptime('%s-%s-01' % (dec_year_end, mod), '%Y-%m-%d')
+                        fecha_fin = fecha_ini + relativedelta(months=+1, days=-1)
+                        code = '340'+dec_year_end+''+mod+'0001'
+                        account_period_id = self.pool.get('account.period').search(cr,uid,[('date_start','=',fecha_ini),('date_stop','=',fecha_fin)])
+                
+                if mod in ('1T', '2T', '3T', '4T'):
+                    month = ( ( int(mod[0])-1 ) * 3 ) + 1
+                    fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, month), '%Y-%m-%d')
+                    fecha_fin = fecha_ini + relativedelta(months=+3, days=-1)
+                    mod = '%02d' % month
+                    code = '340'+dec_year+''+mod+'0001'
+                    account_period_id = self.pool.get('account.period').search(cr,uid,[('date_start','=',fecha_ini),('date_stop','=',fecha_fin)])
+              
+
+
+            else:
+
+
+                if mod >= '01' and mod <= '12':
+                    fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, mod), '%Y-%m-%d')
+                    fecha_fin = fecha_ini + relativedelta(months=+1, days=-1)
+
+                if mod in ('1T', '2T', '3T', '4T'):
+                    month = ( ( int(mod[0])-1 ) * 3 ) + 1
+                    fecha_ini = datetime.strptime('%s-%s-01' % (dec_year, month), '%Y-%m-%d')
+                    fecha_fin = fecha_ini + relativedelta(months=+3, days=-1)
+                    mod = '%02d' % month
+
+                code = '340'+dec_year+''+mod+'0001'
+            
+                account_period_id = self.pool.get('account.period').search(cr,uid,[('date_start','=',fecha_ini),('date_stop','=',fecha_fin)])
             
             if not account_period_id:
                 raise osv.except_osv(_('El periodo seleccionado no coincide con los periodos del año fiscal:'), dec_year)