Skip to content

[FIX] account_invoice_commission: adjust visibility conditions for commission pages#254

Closed
rov-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-107732-rov
Closed

[FIX] account_invoice_commission: adjust visibility conditions for commission pages#254
rov-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-107732-rov

Conversation

@rov-adhoc
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 18, 2026 11:57
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR ajusta la visibilidad de las pestañas de comisiones en el formulario de account.move, y endurece la edición/selección de facturas relacionadas según el tipo de asiento y el estado.

Changes:

  • Se ocultan las páginas “Commissions” y “Commission” también para move_type = 'entry'.
  • Se añade contexto (refs de vistas) y readonly condicionado por estado en los one2many de comisiones.
  • Se incrementa la versión del módulo en el manifest.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
account_invoice_commission/views/account_move_view.xml Ajusta condiciones invisible y modifica atributos/contexto de campos de comisiones en el form de account.move.
account_invoice_commission/manifest.py Bump de versión del módulo a 18.0.1.4.0.

Comment on lines +14 to 18
<page string="Commissions" invisible="move_type in ('in_invoice', 'in_refund', 'in_receipt','entry')">
<field name="commission_invoice_ids" context="{'list_view_ref':'account_invoice_commission.view_invoice_tree', 'form_view_ref':'account.move_form'}" readonly="state != 'draft'"/>
</page>
<page string="Commission" invisible="move_type in ('out_invoice', 'out_refund', 'out_receipt')">
<page string="Commission" invisible="move_type in ('out_invoice', 'out_refund', 'out_receipt','entry')">
<field name="commissioned_invoice_ids" context="{'commissioned_partner_id': partner_id, 'list_view_ref':'account_invoice_commission.view_invoice_tree', 'form_view_ref':'account.move_form', 'search_default_not_commissioned': True, 'search_default_closed': 1, 'search_default_invoice_user_id': partner_user_id, 'default_move_type': 'out_invoice'}" readonly="state != 'draft'"/>
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En el context de estos campos se usa form_view_ref: 'account.move_form', pero en este módulo no existe ese XML ID (aquí se hereda account.view_move_form y el view propio es account_invoice_commission.view_move_form). Si ese form_view_ref no corresponde a un external id válido, al abrir un registro desde el one2many puede fallar. Ajustar form_view_ref a un XML ID existente (p. ej. account.view_move_form o el view heredado del módulo).

Copilot uses AI. Check for mistakes.
@feg-adhoc
Copy link
Copy Markdown
Contributor

@roboadhoc nobump r+

roboadhoc pushed a commit that referenced this pull request Feb 20, 2026
…mmission pages

closes #254

Signed-off-by: Felipe Garcia Suez <feg@adhoc.com.ar>
roboadhoc pushed a commit that referenced this pull request Feb 20, 2026
…mmission pages

closes #254

Signed-off-by: Felipe Garcia Suez <feg@adhoc.com.ar>
@rov-adhoc
Copy link
Copy Markdown
Contributor Author

@roboadhoc nobump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants