[IMP] account_internal_transfer: sync amounts and rates for cross-cur…

[cav-adhoc]

…rency transfers

• Enhanced write() method to sync counterpart_currency_amount between paired payments
• Added rate synchronization with inverted counterpart_rate for paired payment
• Handle rate fields: user_accounting_rate, user_counterpart_rate, accounting_rate, counterpart_rate
• Support for payment_pro module integration (use_payment_pro check)
• Improved cross-currency conversion accuracy when syncing paired transfers

Change note: Ahora cuando modificas el monto o la tasa de cambio en una transferencia interna entre diarios con diferentes monedas, el sistema sincroniza automáticamente estos valores en el pago emparejado (aplicando la tasa inversa cuando corresponde), asegurando que ambos pagos mantengan montos equivalentes sin errores de redondeo.

[roboadhoc]

[Copilot]

Pull request overview

Este PR mejora la sincronización entre pagos emparejados de transferencias internas cuando intervienen diarios con distintas monedas, incorporando sincronización de montos y tasas (incluyendo la inversión de tasa en el pago espejo) y soporte opcional para integración con payment_pro.

Changes:

• Extiende write() para sincronizar amount y counterpart_currency_amount en pagos emparejados, contemplando escenarios multi-moneda.
• Sincroniza counterpart_rate en el pago emparejado usando la tasa inversa.
• Activa la lógica “avanzada” bajo una bandera use_payment_pro.

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (2)

account_internal_transfer/models/account_payment.py:284

• En la sincronización payment_pro se hace updates["counterpart_currency_amount"] = source_amount sin verificar que el campo exista en paired_payment._fields. Si el módulo/versión que aporta ese campo no está presente (o quedó deshabilitado) write() puede fallar con campo desconocido. Sugerencia: condicionar esa asignación a "counterpart_currency_amount" in paired_payment._fields (similar a lo que ya se hace con amount_exact/counterpart_rate).

                        if "amount_exact" in paired_payment._fields:
                            updates["amount_exact"] = converted_amount
                    if source_amount != getattr(paired_payment, "counterpart_currency_amount", 0):
                        updates["counterpart_currency_amount"] = source_amount
                    # Set rate: use inverted rate if explicitly set, otherwise calculate from amounts
                    if converted_amount and "counterpart_rate" in paired_payment._fields:

account_internal_transfer/models/account_payment.py:276

• counterpart_currency_amount se evalúa con if counterpart_amount:; esto ignora explícitamente el caso 0.0. Si el usuario pone 0 (o se llega por redondeo) no se sincroniza ni el amount ni la tasa calculada, dejando el pago emparejado desalineado. Sugerencia: usar una comprobación que distinga False/None de 0.0 (p. ej. is not None) y permitir sincronizar a 0 cuando corresponda.

            if amount_fields_changed and use_payment_pro:
                counterpart_amount = getattr(payment, "counterpart_currency_amount", False)
                if counterpart_amount:
                    converted_amount = abs(counterpart_amount)
                    source_amount = getattr(payment, "amount_exact", False) or payment.amount

[Copilot]

Pull request overview

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