Skip to content

[FIX] l10n_ar_sale: duplicado de órdenes de venta, cómputo de impuestos#282

Merged
jjscarafia merged 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-18.0-h-110078-pam-3710-fw
Mar 3, 2026
Merged

[FIX] l10n_ar_sale: duplicado de órdenes de venta, cómputo de impuestos#282
jjscarafia merged 1 commit intoingadhoc:19.0from
adhoc-dev:19.0-18.0-h-110078-pam-3710-fw

Conversation

@fw-bot-adhoc
Copy link
Copy Markdown
Contributor

Si tengo una Orden de venta con impuesto de iva y percepción de CABA validada, si luego al contacto le agrego impuesto de ARBA, al duplicar la OV se crea solamente con el impuesto de arba y luego al validar se borra el impuesto de arba y se pone el de caba. VIDEO REPLICA RUNBOT ADHOC --> https://drive.google.com/file/d/1RGyX5SuHCgBjABleQQovwuAfm3uaUlK_/view (importante: el video muestra la réplica con facturas pero con órdenes de venta sucede lo mismo).

Este pull request va junto a ingadhoc/odoo-argentina#1295
También necesitamos que se recompute los impuestos al cambiar el contacto --> ver video jgo https://drive.google.com/file/d/1C7oS4EjxmMYzM2N-1MibwbshuShGbSr-/view reportado en ticket 110059 (ver nota jgo 05/02/2026 23:32pm https://www.adhoc.inc/mail/message/15914070 ). Al cambiar un partner, si la FP cambia odoo ofrece un botón para re-calcular. Pero en el caso de que el nuevo partner tenga misma FP, no se recalculan los impuestos. Esto es un problema porque en Argentina, aunque la FP sea la misma, los impuestos pueden variar dependiendo del partner (por ejemplo, por ser monotributista o no). Con este cambio, al cambiar el partner, se recalcularán los impuestos aunque la FP sea la misma.

Ticket Adhoc: 110078

Forward-Port-Of: #279

@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

@fw-bot-adhoc
Copy link
Copy Markdown
Contributor Author

@pablohmontenegro @jjscarafia cherrypicking of pull request #279 failed.

stdout:

Auto-merging l10n_ar_sale/models/sale_order.py
CONFLICT (content): Merge conflict in l10n_ar_sale/models/sale_order.py

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @roboadhoc.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot-adhoc
Copy link
Copy Markdown
Contributor Author

@pablohmontenegro @jjscarafia this forward port of #279 is awaiting action (not merged or closed).

stdout:
Auto-merging l10n_ar_sale/models/sale_order.py
CONFLICT (content): Merge conflict in l10n_ar_sale/models/sale_order.py

stderr:
@pablohmontenegro pablohmontenegro force-pushed the 19.0-18.0-h-110078-pam-3710-fw branch from 54f6ca1 to 145c2ba Compare February 26, 2026 11:27
@jjscarafia
Copy link
Copy Markdown
Member

@roboadhoc r+ nobump override=ci/runbot-modified-modules

@roboadhoc
Copy link
Copy Markdown
Contributor

@jjscarafia you may want to rebuild or fix this PR as it has failed CI.

@roboadhoc
Copy link
Copy Markdown
Contributor

@pablohmontenegro @jjscarafia linked pull request(s) ingadhoc/odoo-argentina#1304 not ready. Linked PRs are not staged until all of them are ready.

@fw-bot-adhoc
Copy link
Copy Markdown
Contributor Author

@pablohmontenegro @jjscarafia this forward port of #279 is awaiting action (not merged or closed).

@jjscarafia jjscarafia merged commit 58060a3 into ingadhoc:19.0 Mar 3, 2026
3 of 4 checks passed
@jjscarafia jjscarafia deleted the 19.0-18.0-h-110078-pam-3710-fw branch March 3, 2026 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants