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

Problema al calcular el total de la factura cuando hay varios impuestos #31

Closed
NeoRazorX opened this issue Aug 24, 2018 · 2 comments
Closed
Labels

Comments

@NeoRazorX
Copy link

Hola, estoy probando tu librería para sustituir a la mía en la generación de facturae en FacturaScripts.

El caso es que yo genero miles de facturas con datos aleatorios para encontrar fallos, y me he encontrado que con las facturas con IRPF falla la validación contable.

El fichero seleccionado no es validable: Error en su lectura: Totalfactura de la factura número 1 no es igual a TotalImporteBrutoAntesImpuestos + TotalImpuestosRepercutidos - TotalImpuestosRetenidos, debería ser : 618.37

Mirando un poco más detenidamente veo que tu librería calcula incorrectamente el total y le baila un céntimo, dando como resultado 618.38

Ta adjunto el xml.
FAC2018IR1_facturae.zip

Mañana seguiré revisando. Y disculpa por si al final fuese problema mio ;-)

@josemmo josemmo added the bug label Sep 4, 2018
josemmo added a commit that referenced this issue Sep 5, 2018
- Corregido cálculo y redondeo de decimales para todos los schemas de FacturaE
- Añadido test de decimales
@josemmo
Copy link
Owner

josemmo commented Sep 5, 2018

He cambiado la forma en la que se redondean los importes de las facturas. En vez de calcular los totales y luego redondear, ahora se redondean los importes de cada línea y luego se suman para obtener los totales.

Es inevitable que con varios decimales se arrestre un pequeño error y el total de la factura difiera en uno o dos céntimos del esperado, pero al menos ahora la factura será válida.

Prueba a ver si te da problemas y me cuentas 😉

@josemmo
Copy link
Owner

josemmo commented Oct 2, 2018

Cierro el issue por falta de actividad y al considerarlo resuelto en el commit 983262b

@josemmo josemmo closed this as completed Oct 2, 2018
@josemmo josemmo mentioned this issue Oct 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants