[FIX] account_payment_pro: include to_pay_move_line_ids in _compute_to_pay_amount dependency

Author: rov-adhoc

account_payment_pro/models/account_payment.py

@@ -150,7 +150,7 @@ def _compute_write_off_available(self):
                 rec.env["account.write_off.type"].search([("company_ids", "=", rec.company_id.id)], limit=1)
             )
 
-    @api.constrains("to_pay_move_line_ids", "state")
+    @api.constrains("to_pay_move_line_ids")
     def _check_to_pay_lines_account(self):
         """TODO ver si esto tmb lo llevamos a la UI y lo mostramos como un warning.
         tmb podemos dar mas info al usuario en el error"""
@@ -549,7 +549,7 @@ def _compute_selected_debt(self):
             #     factor = -1
             # rec.selected_debt = sum(rec.to_pay_move_line_ids._origin.mapped('amount_residual')) * factor
 
-    @api.depends("selected_debt", "unreconciled_amount")
+    @api.depends("selected_debt", "unreconciled_amount", "to_pay_move_line_ids")
     def _compute_to_pay_amount(self):
         for rec in self:
             rec.to_pay_amount = rec.selected_debt + rec.unreconciled_amount
@@ -565,6 +565,13 @@ def _inverse_to_pay_amount(self):
             ):
                 rec.unreconciled_amount = rec.to_pay_amount - rec.selected_debt
 
+    @api.onchange("to_pay_move_line_ids")
+    def _onchange_to_pay_move_line_ids(self):
+        """Evitar que se recompute automáticamente cuando el usuario edita manualmente
+        el campo many2many desde la UI"""
+        # Establecemos un flag para que _compute_to_pay_move_lines no se ejecute
+        self = self.with_context(skip_auto_recompute_to_pay_lines=True)
+
     # We dont set 'is_internal_transfer' as a dependencies as it could leed to recompute to_pay_move_line_ids
     @api.depends("partner_id", "partner_type", "company_id")
     def _compute_to_pay_move_lines(self):
@@ -573,6 +580,10 @@ def _compute_to_pay_move_lines(self):
         # if self._context.get('created_automatically'):
         #     return
 
+        # No recomputar si el usuario está editando manualmente el many2many desde la UI
+        if self._context.get("skip_auto_recompute_to_pay_lines"):
+            return
+
         # Se recomputan las lienas solo si la deuda que esta seleccionada solo si
         # cambio el partner, compania o partner_type
         records = self.filtered(lambda x: x.state == "draft")
@@ -598,6 +609,10 @@ def _get_filter_payments(self, records, extra_fields):
 
     def _get_to_pay_move_lines_domain(self):
         self.ensure_one()
+        # Proteger contra búsquedas si no hay partner definido
+        if not self.partner_id:
+            return [("id", "=", False)]
+        
         domain = [
             ("partner_id", "=", self.partner_id.commercial_partner_id.id),
             ("company_id", "=", self.company_id.id),
@@ -611,7 +626,7 @@ def _get_to_pay_move_lines_domain(self):
                 "asset_receivable" if self.partner_type == "customer" else "liability_payable",
             ),
         ]
-        # TODO revisar bien estos, no debería ser necesario, ver el blame porque se agrego lo del active_ids
+        # Solo agregar active_ids si estamos en contexto de seleccionar líneas específicas
         if self.env.context.get("active_ids") and self.env.context.get("active_model") == "account.move.line":
             domain.append(("move_id.line_ids", "in", self.env.context.get("active_ids")))
         return domain
@@ -628,6 +643,7 @@ def action_add_all(self):
 
     def remove_all(self):
         self.to_pay_move_line_ids = False
+        self.to_pay_amount = 0.0
 
     @api.constrains("partner_id", "to_pay_move_line_ids")
     def check_to_pay_lines(self):
@@ -662,6 +678,7 @@ def _reconcile_after_post(self):
 
     def action_post(self):
         res = super().action_post()
+        self._check_to_pay_lines_account()
         self._reconcile_after_post()
         return res