Skip to content

[16.0][FIX]l10n_es_aeat_sii_oca: arreglar error del sii en un corner case de céntimos con impuestos incluídos - #4648

Open
almumu wants to merge 1 commit into
OCA:16.0from
factorlibre:16.0_l10n_es_aeat_sii_oca_fix_price_included_cents_cornercase
Open

[16.0][FIX]l10n_es_aeat_sii_oca: arreglar error del sii en un corner case de céntimos con impuestos incluídos#4648
almumu wants to merge 1 commit into
OCA:16.0from
factorlibre:16.0_l10n_es_aeat_sii_oca_fix_price_included_cents_cornercase

Conversation

@almumu

@almumu almumu commented Dec 4, 2025

Copy link
Copy Markdown
Member

Arregla error del sii en un corner case de céntimos que sucede en algunas facturas con impuestos incluídos en el precio.

Para reproducir el caso, se reproduce tanto en una factura de venta o compra como en un pos_order. Lo que sucede es que se envía en el desglose de impuestos una base con un céntimo positivo y una cuota con un céntimo negativo, o viceversa.
Por lo que el sii devuelve el error 1231: 'El campo CuotaRepercutida y BaseImponible deben tener el mismo signo.',

Ejemplo: con un iva 21% impuestos incluídos en el precio
image

image

mismo caso pero inverso, base negativa, impuesto positivo
image

image

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@almumu
almumu force-pushed the 16.0_l10n_es_aeat_sii_oca_fix_price_included_cents_cornercase branch 2 times, most recently from 8cc2d48 to d943783 Compare December 5, 2025 06:51
…ase de céntimos que sucede en algunas facturas con impuestos incluídos

Para reproducir el caso, se reproduce tanto en una factura de venta o compra como en un pos_order.
Lo que sucede es que se envía en el desglose de impuestos una base con un céntimo positivo y una cuota con un céntimo negativo, o viceversa.

Por lo que el sii devuelve el error 1231:  'El campo CuotaRepercutida y BaseImponible deben tener el mismo signo.',
@almumu
almumu force-pushed the 16.0_l10n_es_aeat_sii_oca_fix_price_included_cents_cornercase branch from d943783 to a0f919b Compare December 5, 2025 06:56
@almumu
almumu marked this pull request as ready for review December 5, 2025 09:02
@pedrobaeza pedrobaeza added this to the 16.0 milestone Dec 31, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza

Copy link
Copy Markdown
Member

Es un caso demasiado específico como bien decís... pero no debería crearse un método específico para ello, si no ver por qué fallan los mecanismos de limpieza estándar y mejorarlos.

@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ready to merge stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants