OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
22 stars 51 forks source link

[FIX] Fixed issues #11 and #12 #14

Closed gabrielo77 closed 8 years ago

sebastiken commented 9 years ago

@gabrielo77 primero que nada, gracias por tu aporte. Te comento un poco el tema que planteás del domain que no te permite buscar configuraciones de subdiario. En un primer momento, se pensó este módulo para permitir hacer el reporte que uno quiera, siempre basado en aeroo. Entonces, un usuario podía crear un reporte de subdiario con el header de tal manera y que se imprima en pdf. Otro quizás lo quería, pero en xls, etc. Por eso, desde la configuración de subdiario, te permite elegir Reporte (de forma poco intuitiva, porque te aparecen todos los reportes del sistema, sean o no de subdiario) y luego elegís los impuestos que querés que formen parte de ese reporte. Hoy por hoy, la experiencia mostró que no tiene mucho sentido poder elegir entre varios reportes diferentes, y siempre terminamos utilizando el mismo reporte que es el que viene en el módulo. Volviendo al domain que está en la vista, el mismo se basa en el campo report_id, que se llena como un default en la función _get_report_id. Si te fijás, en esa función, busca por un nombre de reporte y se queda con ese. O sea, en definitiva, no solo no tiene mucho sentido permitir usar diferentes reportes sino que por código te obliga a usar uno solo. Es algo que nos quedó un tanto colgado y habría que subsanarlo. Si te animás vos, te pediría que quites todo lo relacionado con esa funcionalidad de elegir el reporte. Esto tanto en el wizard, ese campo report_id como en el objeto _tax_subjournal_reportconfig el campo tax_report_id. Eso implica dejar el domain como lo pusiste en este PR, que no tenga en cuenta ese campo report_id. Espero tus novedades, críticas, ideas y gracias.