Skip to content

Conversation

@almumu
Copy link
Member

@almumu almumu commented Oct 29, 2025

When a declaration line is computed and the country is Greece, it gets GR country code, but the correct code is EL.
Call the function

def _map_aeat_country_iso_code(self, country_id):
to get the iso code

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 29, 2025
…: EL instead of GR.

When a declaration line is computed and the country is Greece, it gets GR country code, but the correct code is EL.
@almumu almumu force-pushed the 16.0_l10n_es_intrastat_report_fix_iso_country_code_for_greece branch from 2354973 to c841a85 Compare October 29, 2025 14:26
@victoralmau
Copy link
Member

Otra posible opción sería definir esas condiciones en https://github.com/OCA/intrastat-extrastat/blob/16.0/intrastat_product/models/intrastat_product_declaration.py#L1171 y https://github.com/OCA/intrastat-extrastat/blob/16.0/intrastat_product/models/intrastat_product_declaration.py#L1183 igual que ya se hace en intrastat_product y cambiar directamente un código por otro si corresponde a nivel general.

@almumu
Copy link
Member Author

almumu commented Oct 30, 2025

@victoralmau entonces el código "EL" para Grecia se aplicaría a cualquier declaración intrastat, sea del país que sea? es decir, que no sólo es cosa de la declaración de intrastat española? si es así hago el PR sobre el módulo de intrastat_product

@victoralmau
Copy link
Member

@victoralmau entonces el código "EL" para Grecia se aplicaría a cualquier declaración intrastat, sea del país que sea? es decir, que no sólo es cosa de la declaración de intrastat española? si es así hago el PR sobre el módulo de intrastat_product

Si se añadiera a intrastat_product si, se haría ese cambio de código de pais siempre (similiar a lo de Irlanda); no obstante, era solo un comentario/sugerencia, desconozco completamente si la necesidad de ese código específico en el caso de Grecia es algo requerido solo para España o a nivel general para todos.

@pedrobaeza
Copy link
Member

Como es un informe europeo, yo diría que es para todos, al igual que ocurre con XI para Irlanda.

@almumu
Copy link
Member Author

almumu commented Oct 30, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants