Skip to content

corregido el bug de las casillas 12 - 13 y 28 - 29#21

Closed
Pabloferrandezroca wants to merge 3 commits into
FacturaScripts:masterfrom
Pabloferrandezroca:tarea4542
Closed

corregido el bug de las casillas 12 - 13 y 28 - 29#21
Pabloferrandezroca wants to merge 3 commits into
FacturaScripts:masterfrom
Pabloferrandezroca:tarea4542

Conversation

@Pabloferrandezroca

Copy link
Copy Markdown

corregido el bug que hacia que las casillas 12 - 13 y 28 - 29 no mostraran ningún valor cuando debían mostrarlo

@javiermartingonzalez

Copy link
Copy Markdown
Contributor

He copiado el código a mi instalación para probar, ya que las intracomunitarias no lo declaraba bien antes, y ahora la información tampoco es correcta:

  • Si el tipo de factura de compra es "Intracomunitaria" o "Intracomunitaria de servicios" debe agregar la información a la 10 y 11
  • Si el tipo de factura de compra es "Inversión del sujeto pasivo" debe hacer lo mismo pero en la casilla 12 y 13
image

Aquí en la documentación lo explican muy bien: https://facturascripts.com/publicaciones/como-registrar-una-compra-intracomunitaria

Por otro lado, la deducción que hace de esos mismos IVA, también lo hace mal, lo está sumando a la 28 y 29. Esto SI es correcto para el caso "Inversión del sujeto pasivo", pero para "Intracomunitaria" o "Intracomunitaria de servicios" debe agregar la información a 36 y 37

image

@javiermartingonzalez

javiermartingonzalez commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Y ya quieres afinarlo 100%, en las facturas de ventas:

  • Si la factura de venta es como "Inversión del sujeto pasivo", debe añadirse la base imponible a la casilla 122
  • Si la factura de venta es como "Intracomunitaria" o "Intracomunitaria de servicios" , debe añadirse la base imponible a la casilla 59
image

@daniel89fg

Copy link
Copy Markdown
Member

@javiermartingonzalez esta es la tarea https://facturascripts.com/roadmap/4542 ¿sabrías hacerla completa?, ya sea en este mismo pr o en un pr nuevo....

@javiermartingonzalez

javiermartingonzalez commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

@daniel89fg Le echo un ojo estos días, no sé si me dejará tocar esta rama que no he creado yo o no. O si tendré que crear otra partiendo desde esta o simplemente copiándome el código cambiado que no es mucho.

A nivel fiscal, justamente me tuve que pelear hace poco con esto para meter bien las cosas y conseguí entender bastante claro cómo funcionaba cada cosa y en qué casilla. Me quedaría echar un ojo al tema de cómo saber si a la hora de ver dónde sumar las facturas, y qué parámetro / parámetros fijarse para ello.

P.S.: Realmente lo que ha hecho el usuario está bien pero es para tipo "Inv sujeto pasivo", no para intracomunitaria. Para intracomunitaria habría que hacer lo mismo pero en las otras casillas comentadas.

Y luego el segundo mensaje si que sería totalmente nuevo hacer algo similar pero en vez de en Compras en Ventas y en otras casillas. No debe ser muy complejo, una vez sabido cómo filtrar por los tipos comentados, no sé si habrá alguna guía al respecto para saber cómo identificar estos tipos para sumarlos a otras casillas.

@javiermartingonzalez

Copy link
Copy Markdown
Contributor

Vale, he encontrado la documentación de los tipos, y creado la vista en un fork mío con las nuevas casillas. Voy a ver cómo gestionar lo del controlador y cuando lo tenga hago PR

@daniel89fg

Copy link
Copy Markdown
Member

Gracias @Pabloferrandezroca y @javiermartingonzalez. He abierto el PR #24 que parte de este trabajo y, siguiendo las correcciones de @javiermartingonzalez, completa el mapeo de todas las operaciones (ISP → 12/13 + 28/29, intracomunitarias/servicios → 10/11 + 36/37, importación → 32/33, y ventas exentas 59/60/122), además de corregir el desfase de céntimos unificando las consultas y el cálculo de la cuota. Este PR quedaría sustituido por el #24.

@daniel89fg daniel89fg closed this Jun 11, 2026
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