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][IMP] (l10n_es_aeat_mod303, l10n_es_aeat_mod390): Valores por defecto en compañias #3918

Closed

Conversation

EmilioPascual
Copy link
Contributor

@EmilioPascual EmilioPascual commented Jan 16, 2025

Al cumplimentar un informe del modelo 303 del cuarto trimestre si está exonerado del 390 o cuando tienes que hacer un informe del 390 hay campos relativos a la compañía como actividades, epígrafes o representantes que suelen ser los mismos todos los años y todos los años hay que rellenarlos.

Propongo este PR para dejar configurada esa información en la compañia y que automáticamente se asignen.

¿@loida-vm @ArantxaSudon @pedrobaeza podéis revisar por favor? Gracias

@moduon MT-8488 MT-8682

…anies

@moduon MT-8488

Co-authored-by: Loida Vilaplana <[email protected]>
Co-authored-by: Arantxa Sudón <[email protected]>
@EmilioPascual EmilioPascual force-pushed the 16.0-default_values_in_companies branch from 69c2afd to db7ad0d Compare January 16, 2025 16:28
@pedrobaeza
Copy link
Member

Creo que en lugar de todo este código, sería mejor permitir duplicar la declaración 390 (o en general, cualquiera).

@pedrobaeza pedrobaeza added this to the 16.0 milestone Jan 16, 2025
Copy link
Contributor

@Shide Shide left a comment

Choose a reason for hiding this comment

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

Dado que son valores por defecto, y que pueden llegar a utilizarse en varios informes, no lo veo mal.
Algunos clientes duplicarán las declaraciones y otros las crearán desde 0. No obstante es positivo no tener que acordarse constantemente de todos los códigos de las actividades principales (si tienen más de una seguro que lo tienen apuntado por algún lado), y como punto extra, sólo se aplica en ciertas casuísticas.
Además, al ser store=True, se puede cambiar a lo largo del tiempo y no cambiará declaraciones anteriores.

Es bastante código, pero creo que es fácilmente mantenible ya que son valores por defecto y rara vez tendremos que hacer algo con ellos (mas que ampliar si acaso dónde se utilizan)

Revisado solo el código

@pedrobaeza
Copy link
Member

Insisto en que debemos reducir el código al máximo, aunque no sea "difícil de mantener". Además, el no poder duplicar una declaración siempre ha sido una lacra.

@EmilioPascual
Copy link
Contributor Author

Insisto en que debemos reducir el código al máximo, aunque no sea "difícil de mantener". Además, el no poder duplicar una declaración siempre ha sido una lacra.

El comportamiento de no poder duplicar una declaración no entra en el ambito de este PR.

Por lo demás, es verdad que es mucho código, pero al final esos datos son relativos a la compañía no al informe del 390. Imagina que utilizamos la solución que indicas, rellenamos el primer informe del 390 con todos los datos y por algún motivo antes de presentarla tenemos que borrarlo; al volver a generar el informe 390 habría que rellenar de nuevo todos los campos, mientras que con este PR configurandolos en la compañía una vez sería suficiente. También puede pasar que cambies la actividad o el epígrafe (es verdad que esta opción es muy rara) en abril, con esta opción vas a la compañia y lo cambias directamente en el momento, de la otra manera hasta Enero que presentes el 390 no generarás el informe y se te puede olvidar cambiarlo ya que han pasado más de 8 meses desde que lo cambiaste.

Incluso creo que es algo que debería ir en el modulo base de AEAT pues son datos que AEAT puede solicitar en otros modelos. No lo he incluido en l10n_es_aeat porque creo que el impacto sería aun mayor.

@pedrobaeza
Copy link
Member

Pero es que pudiendo duplicar la declaración, se quita de un plumazo este problema: duplicas el 303/390 anterior, y marchando. Debe ser el camino a tomar.

@EmilioPascual
Copy link
Contributor Author

Cierro el PR, lo llevaremos a nuestro repo público.

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