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

[12.0][FIX] l10n_es_aeat_mod347: Save proper real state fields #2762

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

MiguelPoyatos
Copy link

Relacionado con #2001

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jan 25, 2023
@pedrobaeza
Copy link
Member

Esto requiere más explicación de por qué cambiar esto, y pasos para reproducir el problema. ¿Es alguna ampliación hecha fuera del módulo lo que lo provoca?

@MiguelPoyatos
Copy link
Author

Esto requiere más explicación de por qué cambiar esto, y pasos para reproducir el problema. ¿Es alguna ampliación hecha fuera del módulo lo que lo provoca?

Proviene de la misma casuística que el issue #2001

Resumen:

Error cuando se selecciona una empresa al crear un nuevo registro de inmuebles en el modelo 347

Descripción del problema

Si en un modelo 347 intentamos añadir un registro de inmuebles, al seleccionar una empresa, devuelve un error ya que el campo partner_state_code no pertenece al modelo l10n.es.aeat.mod347.real_estate_record, se resuelve y se mantiene la funcionalidad esperada mapeando el campo con state_code, que si que esta presente en el modelo.

Pasos para reproducir el error

Versiones afectadas: 12.0 y superiores, desde la versión 13.0 en adelante, está actualmente resuelto en: #2001

Pasos para reproducirlo:

  1. Ir al modelo 347 de AEAT
  2. Generar y calcular un nuevo informe del modelo 347
  3. Ir a la pestaña 'Registros de inmuebles'
  4. Añadir una nueva línea
    image
  5. Al seleccionar una empresa el error sucede.
    image
    image

@AlbertCabedo
Copy link

correcto, en una v12 me sucede lo mismo

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-2762-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5b2ce7c into OCA:12.0 Jan 25, 2023
@OCA-git-bot
Copy link
Contributor

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

@MiguelPoyatos
Copy link
Author

Gracias!

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.

4 participants