[16.0][FIX] l10n_es_aeat: Manage AEAT Real Estates #3086

Shide · 2023-06-12T09:23:11Z

Gestión de los Real Estates.

Es un PR inicial/base para cuando se decida incluirlos en los modelos de la AEAT.

MT-3113 @moduon @ArantxaSudon @loida-vm @rafaelbn

OCA-git-bot · 2023-06-12T09:23:15Z

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

loida-vm

¿Podría sacarse un desplegable de:
Código de provincias (Spanish_states) https://sede.agenciatributaria.gob.es/Sede/ayuda/tablas-auxiliares-domicilios/provincias.html (1a columna)
Código de municipios (Township_code) https://sede.agenciatributaria.gob.es/Sede/ayuda/tablas-auxiliares-domicilios/municipios.html (2a columna)
?

Gracias

Shide · 2023-06-13T09:09:10Z

@loida-vm He creado un nuevo modelo para los municipios para que una vez que se seleccione la provincia, se pueda seleccionar el municipio y se auto rellene la ciudad.
Las provincias tiran directamente de las de Odoo y sacan el código de provincia del mapeo de provincias que está directamente en código.

rafaelbn · 2023-06-13T18:40:07Z

Este PR viene de #2893

¡Gracias!

loida-vm · 2023-06-15T08:09:47Z

@loida-vm He creado un nuevo modelo para los municipios para que una vez que se seleccione la provincia, se pueda seleccionar el municipio y se auto rellene la ciudad. Las provincias tiran directamente de las de Odoo y sacan el código de provincia del mapeo de provincias que está directamente en código.

Perfecto @Shide . En la plantilla ahora solo me aparecen los nombres y no los códigos, pero si hubiera que sacar los datos imagino que sacaría también el código, ¿verdad?

Muchas gracias!

rafaelbn · 2023-07-25T13:25:29Z

PR del 180

pedrobaeza · 2023-10-22T16:08:58Z

l10n_es_aeat/data/aeat_township_data.xml

En lugar de tener otra tabla más muy populada, creo que habría que utilizar l10n_es_toponyms, y extraer el código de provincia de los 2 primeros dígitos del CP, aunque teniendo el res.country.state, no lo veo necesario.

pedrobaeza · 2023-10-22T16:11:37Z

l10n_es_aeat/models/aeat_data.py

+}
+
+# https://sede.agenciatributaria.gob.es/Sede/ayuda/tablas-auxiliares-domicilios/tipos-vias.html
+AEAT_ADDRESS_TYPES = [


Esto se utilizar solo en un sitio, así que no veo la necesidad de mover el otro diccionario y poner esto aquí. Ponlo directamente en el archivo py donde se utiliza, y si es necesario utilizarlo en otro sitio, se puede importar desde ese archivo.

Son 300 y pico líneas directamente en el fichero de l10n_es_aeat_real_estate.py y en la parte superior del fichero. Ensucia un poco la legibilidad al tener que hacer tanto scroll nada más abrir el fichero y además ya existe el fichero aeat_data.py (antes se llamaba spanish_states_mapping.py y lo he renombrado), que creo que es apropiado utilizarlo para estos casos.

Vale, pues no cambies el mapeo de provincias, y crea un nuevo archivo para esto. Lo que no quiero es un diff grande sin necesidad

pedrobaeza · 2023-10-22T16:12:55Z

l10n_es_aeat/models/l10n_es_aeat_real_estate.py

+        compute_sudo=True,
+    )
+    township_domain = fields.Many2many(
+        comodel_name="l10n.es.aeat.township",


Sustituir por res.city.zip.

pedrobaeza · 2023-10-22T16:13:28Z

l10n_es_aeat/models/l10n_es_aeat_report.py

 from odoo.tools import config

-from .spanish_states_mapping import SPANISH_STATES as ss
+from .aeat_data import AEAT_STATES_CODE_MAP as ascm


No es necesario este cambio como se ha comentado.

rafaelbn · 2024-02-05T11:51:55Z

@Shide or @EmilioPascual please, could you rebase? Thank you!

netkia · 2024-05-07T10:18:04Z

@Shide or @EmilioPascual please, could you rebase? Thank you!

rafaelbn · 2024-08-20T18:03:56Z

@Shide or @EmilioPascual will rebase next monday 😄 and review all comments. Thank you! ❤️

mariancuadranetkia · 2024-09-11T05:44:56Z

@rafaelbn Could you review comments and rebase?

rafaelbn · 2024-09-11T18:09:31Z

OK @mariancuadranetkia . Tomorrow we will make the rebase and review 😄 . Please @Shide could dedicate some time? Then we can go ahead to review #3617

Juraría que aquí en localización podemos hablar en castellano 😄

rafaelbn · 2024-09-13T17:59:28Z

Hello, just to inform that we could review this PR in the OCA Sprint Code in October. Until that moment we don't have time for this. Sorry.

mariancuadranetkia · 2024-09-14T16:36:54Z

Hasta donde sé, sí, podemos hablar en castellano jeje
Ok, intentaremos estar atentos para poder retomarlo nosotros también.
Gracias @rafaelbn

github-actions · 2025-01-19T12:38:22Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

rafaelbn · 2025-01-22T14:10:19Z

Please @OCA/local-spain-maintainers , could you remove the Stale label? Thank you!

mariancuadranetkia · 2025-06-03T08:34:04Z

@moduon @rafaelbn @Shide Esto lo acabareis?

Hay conflictos.

github-actions · 2025-10-05T12:45:25Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

pedrobaeza changed the title ~~[FIX] l10n_es_aeat: Manage AEAT Real Estates~~ [16.0][FIX] l10n_es_aeat: Manage AEAT Real Estates Jun 12, 2023

Shide force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch from 4f6fdf1 to a63b883 Compare June 12, 2023 09:25

Shide marked this pull request as draft June 12, 2023 09:25

loida-vm reviewed Jun 12, 2023

View reviewed changes

Shide force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch 2 times, most recently from d88d5da to 0161c79 Compare June 13, 2023 09:07

rafaelbn mentioned this pull request Jun 13, 2023

Informar de la referencia catastral de inmuebles para que se auto rellen en los modelos de AEAT #2893

Closed

rafaelbn mentioned this pull request Jul 25, 2023

[12.0][ADD] l10n_es_aeat_mod180 #3153

Closed

Shide marked this pull request as ready for review October 2, 2023 07:49

Shide force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch from 0161c79 to 43d5c19 Compare October 2, 2023 07:53

pedrobaeza reviewed Oct 22, 2023

View reviewed changes

EmilioPascual force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch 2 times, most recently from 10ed08c to bc2265d Compare January 24, 2024 16:55

rafaelbn mentioned this pull request Jan 31, 2024

[16.0][IMP] l10n_es_aeat: NIF del representante legal por defecto #3389

Merged

EmilioPascual force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch 3 times, most recently from 053b670 to 8255f63 Compare February 2, 2024 16:28

Shide force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch from 8255f63 to dc4bbcf Compare March 26, 2024 11:03

Shide force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch from dc4bbcf to b838f47 Compare May 7, 2024 10:43

hugomartineznetkia mentioned this pull request May 30, 2024

[16.0][ADD]l10n_es_aeat_mod180 #3617

Closed

1 task

pedrobaeza added this to the 16.0 milestone Sep 11, 2024

github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 19, 2025

EmilioPascual force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch 5 times, most recently from 12fe01c to 40113e2 Compare January 20, 2025 11:04

pedrobaeza removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 22, 2025

[FIX] l10n_es_aeat: Manage AEAT Real Estates

c563af5

EmilioPascual force-pushed the 16.0-l10n_es_aeat_real_estates-l10n_es_aeat branch from 40113e2 to c563af5 Compare February 5, 2025 09:03

github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 5, 2025

Uh oh!

[16.0][FIX] l10n_es_aeat: Manage AEAT Real Estates #3086

Are you sure you want to change the base?

[16.0][FIX] l10n_es_aeat: Manage AEAT Real Estates #3086

Conversation

Shide commented Jun 12, 2023

Uh oh!

OCA-git-bot commented Jun 12, 2023

Uh oh!

loida-vm left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Shide commented Jun 13, 2023

Uh oh!

rafaelbn commented Jun 13, 2023

Uh oh!

loida-vm commented Jun 15, 2023

Uh oh!

rafaelbn commented Jul 25, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pedrobaeza Oct 22, 2023

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Oct 22, 2023

Choose a reason for hiding this comment

Uh oh!

Shide Oct 24, 2023

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Oct 24, 2023

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Oct 22, 2023

Choose a reason for hiding this comment

Uh oh!

pedrobaeza Oct 22, 2023

Choose a reason for hiding this comment

Uh oh!

rafaelbn commented Feb 5, 2024

Uh oh!

netkia commented May 7, 2024

Uh oh!

rafaelbn commented Aug 20, 2024

Uh oh!

mariancuadranetkia commented Sep 11, 2024

Uh oh!

rafaelbn commented Sep 11, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rafaelbn commented Sep 13, 2024

Uh oh!

mariancuadranetkia commented Sep 14, 2024

Uh oh!

github-actions bot commented Jan 19, 2025

Uh oh!

rafaelbn commented Jan 22, 2025

Uh oh!

mariancuadranetkia commented Jun 3, 2025

Uh oh!

github-actions bot commented Oct 5, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

loida-vm left a comment •

edited

Loading

rafaelbn commented Jul 25, 2023 •

edited

Loading

rafaelbn commented Sep 11, 2024 •

edited

Loading