Code review comment for lp:~txerpa-openerp/openerp-spain/7.0-l10n_es_aeat_modelo303

Buenas, Biel,

Ya he visto que has renombrado el módulo. Muchas gracias. Pero estoy viendo que hay que hacer cambios más profundos por dos cosas:

- Los modelos que has creado son independientes de los antiguos, provocando que se pierda todo el histórico anterior y que además hay tablas en la BD que se queden colgadas. Lo que habría que hacer es utilizar el mismo modelo base (l10n.es.aeat.mod303.report) y crear un script de migración para compatibilizar lo antiguo con lo nuevo.

- Los impuestos los has definido sobre el l10n_es antiguo, y de hecho tu modificación sólo cubre los impuestos de las PYMEs, no el resto. En la refactorización, ahora se comparten todos los impuestos, por lo que sólo es necesario definir una vez las cosas en todos sitios.

Lo que voy a hacer es incluir los nuevos impuestos que has definido tú en la refactorización de l10n_es y que desbloqueemos ya de una vez el MP. Te agradecería si pudieras poner tu revisión en el mismo.

Después de eso, trabajaré en una revisión de tu MP que solucione los problemas que he comentado, y cuando esté listo y revisado por vuestra, incluimos a la vez tu MP y el mío. ¿Te parece bien?

Un saludo.