[IMP] l10n_ar_tax: limitar auto-detección de posición fiscal AR a NC con reversión

[pablohmontenegro]

1. Agrega el campo “Solo auto-detectar en reversiones (AR)” en posiciones fiscales.
2. En notas de crédito manuales, no autodetecta si el campo está marcado.
3. Mantiene la detección en NC creadas desde la factura (con vínculo a comprobante original).
4. Suma cobertura de tests para factura, NC por reversión y NC manual.
5. Actualiza vista, traducciones y README con la nueva funcionalidad.
   Tarea Adhoc: 57361

[roboadhoc]

[Copilot]

Pull request overview

Este PR mejora el módulo l10n_ar_tax para limitar la auto-detección de posiciones fiscales argentinas marcadas como “solo en reversiones” a notas de crédito creadas como reversión, evitando aplicarlas en NC manuales, y agrega tests/registros de UI/documentación asociados.

Changes:

• Agrega el flag l10n_ar_reversal_only en posiciones fiscales AR y lo expone en la vista.
• Ajusta la lógica de ranking/selección de posición fiscal para excluir las “reversal-only” en NC manuales.
• Incorpora cobertura de tests para factura, NC por reversión y NC manual; actualiza traducciones y README.

Reviewed changes

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

Show a summary per file

File	Description
l10n_ar_tax/models/account_fiscal_position.py	Nuevo booleano l10n_ar_reversal_only + ranking para ignorarlo en NC manuales vía contexto.
l10n_ar_tax/models/account_move.py	Override de _compute_fiscal_position_id para inyectar contexto en NC manuales AR.
l10n_ar_tax/views/account_fiscal_position_view.xml	Campo nuevo en formulario de posición fiscal (visible solo para AR).
l10n_ar_tax/tests/test_payment_withholding_validation.py	Tests de regresión para flujos de factura / reversión / NC manual.
l10n_ar_tax/i18n/es.po	Traducciones del label/help del nuevo campo.
l10n_ar_tax/README.rst	Nota funcional agregada sobre el nuevo comportamiento.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

l10n_ar_tax/tests/test_fiscal_position_validation.py:117

• En _create_invoice_with_caba_perception() se vuelve a crear un registro account.fiscal.position.l10n_ar_tax para la misma posición fiscal, pero _create_fiscal_positions() ya lo crea. Esto dispara el constraint _check_tax_group_overlap (mismo tax_group para la misma posición fiscal) y puede hacer fallar el test/instalación. Sugerencia: eliminar la creación duplicada en _create_invoice_with_caba_perception() o mover esa creación a un único helper (dejando el otro sin side-effects).

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (1)

l10n_ar_tax/tests/test_fiscal_position_validation.py:52

• En esta ayuda de tests, las posiciones fiscales creadas con auto_apply=True no fijan company_id ni sequence. En entornos de test donde ya existan otras posiciones fiscales auto-aplicables, esto puede volver la selección no determinística (flaky) y hacer fallar la aserción. Sugerencia: setear company_id (p.ej. self.company_ri.id) y una sequence explícita para que la CABA/generica queden ordenadas de forma estable (aplica también a la posición CABA creada más abajo).

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

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