Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 #3166

Merged
merged 17 commits into from
Feb 22, 2024

Conversation

ACheung-FactorLibre
Copy link
Contributor

No description provided.

@ACheung-FactorLibre ACheung-FactorLibre marked this pull request as draft July 28, 2023 09:51
@ACheung-FactorLibre ACheung-FactorLibre marked this pull request as ready for review August 1, 2023 09:19
@ACheung-FactorLibre ACheung-FactorLibre changed the title [WIP][16.0][MIG] l10n_es_aeat_mod_369: Migration to 16.0 [16.0][MIG] l10n_es_aeat_mod_369: Migration to 16.0 Aug 1, 2023
@RodrigoBM
Copy link
Contributor

@ACheung-FactorLibre cambiar el nombre de la tarea a [16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 gracias

@ACheung-FactorLibre ACheung-FactorLibre changed the title [16.0][MIG] l10n_es_aeat_mod_369: Migration to 16.0 [16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 Aug 22, 2023
@SoniaViciana
Copy link

Aquí realizamos una migración estándar de #3117.

Tras hacer pruebas en Runboat en las versiones 14.0 y 15.0 vemos que el modelo contiene multitud de errores o no lo estamos probando bien. Teniendo en cuenta que sólo contempla la declaración bajo el Régimen de la Unión:

1. Configuración de impuestos

Hay 2 campos de país:

image

Cuando "country_id = Hungría" y "oss_country_id = España". El informe completa la pestaña 4 pero el código de país es erróneo, puesto que debería ser HU:

image

Si invertimos la configuración: "country_id = España" y "oss_country_id = Hungría". El informe completa la pestaña 8 y el código de país es correcto, sin embargo no se rellenan 3 ni 4:

image

Esas pestañas deberían completarse conforme a la configuración del service_type que vemos en el impuesto pero no funciona:

image

2. Readme erróneo

El modelo no crea el asiento de regularización de IVA -> #3194

3. Fichero TXT de carga en AEAT erróneo

Sin entrar al detalle y tras realizar una comprobación en diagonal de las intstrucciones de la AEAT, el fichero que genera Odoo no es correcto:

https://sede.agenciatributaria.gob.es/Sede/ayuda/consultas-informaticas/presentacion-declaraciones-ayuda-tecnica/modelo-369.html

image


¿Este módulo se ha usado en algún entorno de producción en alguna versión? El PR original lleva meses mergeado #2362...

Gracias,

@JuanjoA @victoralmau @hildickethan-S73 @rafaelbn @pedrobaeza

@hildickethan
Copy link
Member

¿Este módulo se ha usado en algún entorno de producción en alguna versión? El PR original lleva meses mergeado #2362...

Nosotros llegamos a usarlo en la 14.0, solo en 1 cliente pero sigue usándolo a día de hoy y no recibido quejas, aunque es posible que la configuración funcione por casualidad
Yo en su día me basé en el fichero ese de diseño de registro de modelo y llegué a sacar ficheros que los acepta la AEAT al declararlo con el cliente mencionado aneriormente

Pero lo que comentas sí que parece un poco raro; country_id en la 14.0 no estaba, parece ser que se puso en base en la 15.0
y debería haberse contemplado en la migración del módulo de OSS o este en esa versión. Aunque en todo caso entiendo que deberían de tener el mismo valor los 2 campos y los casos que presentas no deberían darse

@SoniaViciana
Copy link

Gracias por la respuesta @hildickethan-S73

Revisando en v14, sí existen ambos campos de "country" a nivel de impuestos:

  • Oss country -> oss_country_id (Alemania)
  • Fiscal country -> country_id (España)

image

El resultado en RunBoat es erróneo igual que en v15 y v16, puesto que no se completan los apartados 3 ni 4 a pesar de haber configurado los impuestos:

image

Al contabilizar una factura a la que aplican 2 cuotas de Alemania:

image

El modelo 369 no las diferencia, las unifica en una misma línea y no las tiene en cuenta ni como bienes ni como servicios:

image

@ACheung-FactorLibre ACheung-FactorLibre changed the title [16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 [WIP][16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 Sep 10, 2023
@ACheung-FactorLibre ACheung-FactorLibre changed the title [WIP][16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 [16.0][MIG] l10n_es_aeat_mod369: Migration to 16.0 Sep 25, 2023
@SoniaViciana
Copy link

Con el último FIX aportado por @aritzolea ya se calcula bien el modelo siempre que se establezca esta configuración en los impuestos:

imagen

Esta configuración puede ser algo confusa dado el nombre de los campos, pero vimos que oss_country_id se creó en el módulo l10n_eu_oss_oca en versiones previas, cuando los impuestos no tenían país, en versiones posteriores se incorporó el país en la base de account como country_id. Esta parte del país está bastante confusa actualmente, si alguien más le puede echar un vistazo y aportar ayuda se lo agradecemos.

De todas formas, ahora en la v16 sí se calcula bien el Régimen de la Unión::

imagen

Diferenciando bienes y servicios:

imagen

imagen

Próximos pasos:

  • Incoporar pestaña "7. Correcciones de autoliquidaciones de periodos anteriores".
  • Ajustar el fichero TXT para cargar en la AEAT

Gracias,

@HaraldPanten @ValentinVinagre @pedrobaeza

Copy link

@SoniaViciana SoniaViciana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisión funcional OK

@pedrobaeza
Copy link
Member

Vale, pero quedarían aún pasos antes de poder fusionar esto, ¿no?

/ocabot migration l10n_es_aeat_mod369

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 25, 2023
@ValentinVinagre
Copy link
Contributor

¿ @SoniaViciana el PR ya está listo para revisar?

@SoniaViciana
Copy link

Se puede revisar que los cálculos y desglose de apartados del modelo 369 son correctos.

Lo que creo que no estará bien es el TXT puesto que no nos ha dado tiempo a revisarlo aún y a simple vista parece que ya venía mal de versiones anteriores.

Gracias,

@ValentinVinagre
Copy link
Contributor

Se puede revisar que los cálculos y desglose de apartados del modelo 369 son correctos.

Lo que creo que no estará bien es el TXT puesto que no nos ha dado tiempo a revisarlo aún y a simple vista parece que ya venía mal de versiones anteriores.

Gracias,

Entonces vamos a esperar que lo tengáis terminado completamente para revisarlo.
Un saludo,

@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch 3 times, most recently from 171da4d to bd34674 Compare October 5, 2023 09:15
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch 8 times, most recently from 5f633a0 to f95afac Compare October 17, 2023 11:53
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from f95afac to 1093eea Compare October 18, 2023 06:51
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from 99507b9 to ba26867 Compare November 16, 2023 07:44
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from ba26867 to c2eda12 Compare November 28, 2023 09:50
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from c2eda12 to c98470f Compare November 29, 2023 10:13
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch 2 times, most recently from 4e73d20 to e0effaa Compare December 22, 2023 11:40
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch 2 times, most recently from ab3fa1e to 07a3e60 Compare January 25, 2024 07:35
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from 07a3e60 to eb421c2 Compare February 21, 2024 13:27
@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from eb421c2 to 311252a Compare February 21, 2024 14:37
@SoniaViciana
Copy link

Esta migración ya debe estar correcta y se puede probar (pendiente rehacer el Runboat). El pasado mes de Enero una empresa declaró el 4T de 2023 con el mismo fichero que genera Odoo, este fue el resultado:

imagen

El error que sale sospechamos que es una advertencia solamente, consecuencia del método de redondeo, esto:

imagen

Si comparamos el 369 de Odoo respecto a la imagen que os he pasado de la AEAT, la AEAT directamente ajusta los céntimos:

imagen

Lo he comparado en Excel sólo que no puedo cargarlo, esto:

imagen

La cuestión es que por los céntimos difiere en 0,05€ el resultado que calcula la AEAT respecto a lo que le informamos desde Odoo con el TXT:

imagen

Pero lo dicho, consideramos que es una advertencia y que no tiene solución a no ser que se cambie el método de redondeo (lo cual nosotros en nuestro proyecto no haremos sabiendo cómo funciona). Si surge cualquier duda quedamos a vuestra disposición.

Saludos!

@aritzolea aritzolea force-pushed the 16.0-mig-l10n_es_aeat_mod369 branch from 311252a to dbdeaae Compare February 22, 2024 07:16
@pedrobaeza
Copy link
Member

Efectivamente veo que es un aviso y que además, es demasiado estricto, ya que la suma de redondeos nunca será el redondeo de la suma. En otros modelos lo tienen en cuenta y dejan un margen de tolerancia, pero parece ser que aquí no.

Vamos a fusionar ya después de todo este recorrido, y siempre se pueden hacer PRs extra para cualquier parche encontrado posteriormente. Gracias a todos los que han colaborado.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-3166-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d2aa142 into OCA:16.0 Feb 22, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 4fb94d5. Thanks a lot for contributing to OCA. ❤️

@aritzolea aritzolea deleted the 16.0-mig-l10n_es_aeat_mod369 branch February 27, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.