Merge lp:~pexego/openerp-spain/6.1_347_fix_refund_amount into lp:~openerp-spain-team/openerp-spain/6.1

[Omar (Pexego)]

El signo de los impuestos para facturas rectificativas está configurado como negativo, lo cual es correcto, pero esto provoca que el campo tax_amount de las lineas de impuestos tenga signo negativo, por lo tanto, para obtener el total abonado, el sistema hace base imponible de la factura + tax_amount y al ser negativo causa este descuadre, es decir: con una base imponible de 100 y un tax_amount de 21 el importe facturado será (100 + 21) 121 y el importe abonado si esta factura se abonara por completo sería (100 + (-21) 79 entonces el 347 nos pondría 121 - 79 = 42

Este merge proposal, arregla este comportamiento, de la misma forma que se hizo en openerp-spain/6.0 en la revisión 338

[Omar (Pexego) (omar7r)]

Esta misma corrección habría que aplicarla en la 7.0, aunque el destino de la corrección es distinto, en el caso de la 7.0 será en el fichero mod347.py y en el método _calc_total_invoice

[Pedro Manuel Baeza (pedro.baeza)]

Omar, gracias por solucionar este problema. He probado su funcionamiento y es correcto, refrendado además por estar ya operativo en la versión 6.0.

Hago ahora el mismo MP para la versión 7.0.

Gracias.

Un saludo.

review: Approve (code review and test)

[Alejandro Santana (alejandrosantana)]

Pregunta tal vez tonta, pero: ¿cuál es la diferencia entre 'account_invoice_tax.amount' y 'account_invoice_tax.tax_amount'?
No acabo de verlo, ya que veo que a veces a 'amount' se le asigna el valor de 'tax_amount'.

review: Needs Information

[Pedro Manuel Baeza (pedro.baeza)]

Te remito a:

https://code.launchpad.net/~pedro.baeza/openerp-spain/l10n_es_aeat_mod347_fix/+merge/211650/comments/500248

=== modified file 'l10n_es_aeat_mod347/wizard/calculate_mod347_records.py'
--- l10n_es_aeat_mod347/wizard/calculate_mod347_records.py	2013-07-18 16:08:37 +0000
+++ l10n_es_aeat_mod347/wizard/calculate_mod347_records.py	2014-02-10 12:59:31 +0000
@@ -50,7 +50,7 @@
         amount = invoice.cc_amount_untaxed
         for tax_line in invoice.tax_line:
             if tax_line.name.find('IRPF') == -1:
-                amount += tax_line.tax_amount
+                amount += tax_line.amount
         return amount
 
     def _calculate_records(self, cr, uid, ids, context=None, recalculate=True):