Skip to content

Changes excepciones#1912

Merged
NeoRazorX merged 5 commits intoNeoRazorX:masterfrom
daniel89fg:changes-excepciones
Mar 24, 2026
Merged

Changes excepciones#1912
NeoRazorX merged 5 commits intoNeoRazorX:masterfrom
daniel89fg:changes-excepciones

Conversation

@daniel89fg
Copy link
Contributor

Mejoras al cambio de regímenes y excepciones de iva en base al primer pr enviado.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Daniel Fernández Giménez added 2 commits March 16, 2026 17:17
…ial de cambio de regímenes de iva y excepciones
Se reemplazan las referencias a `RegimenIVA` por `TaxExceptions` en los tests para reflejar los cambios realizados en la estructura del código. Además, se ajusta el conteo esperado de excepciones en `TaxExceptionsTest`.
Copilot AI review requested due to automatic review settings March 16, 2026 16:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Actualiza y amplía el soporte de regímenes de IVA y excepciones (principalmente España), alineando constantes, validaciones, UI y migraciones para compatibilidad con datos existentes.

Changes:

  • Refactor de constantes de excepciones de IVA (TaxExceptions) y adaptación de validaciones/uso en calculadora y tests.
  • Añade selector de régimen de IVA en settings y valor por defecto para España.
  • Mejora filtros en listados de clientes/proveedores y añade migración para normalizar valores legacy de excepcioniva.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
Test/Core/Model/ProductoTest.php Ajusta test de excepción de IVA a nuevas constantes.
Test/Core/Mod/CalculatorModSpainTest.php Actualiza expectativas de excepciones de IVA en escenarios de cálculo España.
Test/Core/Lib/TaxExceptionsTest.php Adapta tests a nuevos códigos/keys de excepciones y reglas.
Core/XMLView/SettingsDefault.xml Añade campo regimeniva (select) en configuración por defecto.
Core/Mod/CalculatorModSpain.php Sustituye excepciones antiguas por nuevas constantes de TaxExceptions.
Core/Migrations.php Añade migración para normalizar valores legacy de excepcioniva y actualiza mapping de operación desde excepción.
Core/Lib/TaxExceptions.php Redefine catálogo de excepciones, defaults, validación y añade método get().
Core/Lib/RegimenIVA.php Mantiene compatibilidad: constantes deprecated mapeadas a TaxExceptions nuevas.
Core/Lib/InvoiceOperation.php Añade método get() para recuperar la etiqueta por clave.
Core/Data/Codpais/ESP/default.json Añade regimeniva: General a la configuración por defecto de ESP.
Core/Controller/Wizard.php Añade opción vacía inicial en el selector de régimen de IVA.
Core/Controller/ListProveedor.php Añade filtros por régimen, operación y excepción de IVA (con traducción).
Core/Controller/ListCliente.php Añade filtros por régimen, operación y excepción de IVA (con traducción).
Core/Controller/EditSettings.php Carga valores traducidos de regímenes de IVA en el select del panel de control.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

daniel89fg and others added 3 commits March 17, 2026 08:41
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Se modifica el código para verificar la existencia de claves en `InvoiceOperation::get` antes de traducir descripciones en las listas de clientes y proveedores. Además, se añaden nuevas claves a las pruebas de `TaxExceptions` y se actualiza un mensaje en un test de producto.
@NeoRazorX NeoRazorX merged commit f91fa20 into NeoRazorX:master Mar 24, 2026
22 checks passed
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