Code review comment for lp:~rafael-valle-z/openerp-spain/openerp-spain
Revision history for this message
| Pedro Manuel Baeza (pedro.baeza) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
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.