Skip to content

3619: Añadir nuevas funciones de cálculo y pruebas para el informe de impuestos - #81

Closed
daniel89fg wants to merge 5 commits into
mainfrom
3619-report-taxes-calculator
Closed

3619: Añadir nuevas funciones de cálculo y pruebas para el informe de impuestos#81
daniel89fg wants to merge 5 commits into
mainfrom
3619-report-taxes-calculator

Conversation

@daniel89fg

Copy link
Copy Markdown
Member

Copilot AI review requested due to automatic review settings April 14, 2026 12:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Este PR actualiza el informe de impuestos del plugin Informes para que el desglose de impuestos se base en los subtotales calculados del documento (en vez de recalcular desde SQL por líneas), y añade pruebas para casos fiscales especiales (intracomunitarias, bienes usados y agencias de viaje).

Changes:

  • Refactor de ReportTaxes para obtener subtotales por factura usando Calculator::getSubtotals() y generar filas de informe desde el documento.
  • Añade tests de regresión/cobertura para operaciones intracomunitarias y regímenes especiales (segunda mano y agencias de viaje, incluyendo margen negativo).
  • Sube la versión mínima requerida del core en facturascripts.ini.

Reviewed changes

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

File Description
facturascripts.ini Actualiza la versión mínima requerida del core para soportar los nuevos cálculos.
Controller/ReportTaxes.php Cambia la generación del reporte a un cálculo por documento usando subtotales y añade utilidades de reparto/validación.
Test/main/ReportTaxesTest.php Añade pruebas para validar el desglose de IVA en escenarios fiscales especiales.

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

Comment thread Test/main/ReportTaxesTest.php
Comment thread Controller/ReportTaxes.php
Comment thread Controller/ReportTaxes.php
@NeoRazorX

Copy link
Copy Markdown
Member

¡Gracias por el trabajo, @daniel89fg! 🙏

Cerramos este PR porque el bug de la tarea #3619 (IVA incorrecto en el informe de impuestos con facturas de segunda mano) ya se ha corregido en main con otro enfoque, manteniendo el cálculo por SQL del informe:

  • Segunda mano (REBU): commit 2262752
  • Ampliación a agencias de viajes (mismo régimen de margen): en preparación

Las líneas de venta en régimen de margen (bienes usados y agencias de viajes) se desdoblan ahora en coste al 0% y margen al tipo de IVA, replicando CalculatorModSpain::applyUsedGoods/applyTravel, con tests en ReportTaxesTest. Aun así, tu idea de delegar en el Calculator es interesante para el futuro; si quieres la retomamos por separado. ¡Gracias de nuevo!

@NeoRazorX NeoRazorX closed this Jul 10, 2026
@daniel89fg
daniel89fg deleted the 3619-report-taxes-calculator branch July 28, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants