[FIX] account_reconcile_oca : multi currency management

account_reconcile_oca/models/account_bank_statement_line.py

@@ -185,23 +185,29 @@ def _onchange_manual_model_id(self):
             )._default_reconcile_data()
         self.can_reconcile = self.reconcile_data_info.get("can_reconcile", False)
 
+    def _get_amount_currency(self, line, dest_curr):
+        if line["line_currency_id"] == dest_curr.id:
+            amount = line["currency_amount"]
+        else:
+            amount = self.company_id.currency_id._convert(
+                line["amount"],
+                dest_curr,
+                self.company_id,
+                self.date,
+            )
+        return amount
+
     @api.onchange("add_account_move_line_id")
     def _onchange_add_account_move_line_id(self):
         if self.add_account_move_line_id:
             data = self.reconcile_data_info["data"]
             new_data = []
             is_new_line = True
             pending_amount = 0.0
-            currency = self._get_reconcile_currency()
             for line in data:
                 if line["kind"] != "suspense":
-                    pending_amount += currency._convert(
-                        line["currency_amount"],
-                        self.env["res.currency"].browse(
-                            line.get("line_currency_id", currency.id)
-                        ),
-                        self.company_id,
-                        self.date,
+                    pending_amount += self._get_amount_currency(
+                        line, self._get_reconcile_currency()
                     )
                 if self.add_account_move_line_id.id in line.get(
                     "counterpart_line_ids", []
@@ -582,6 +588,7 @@ def _default_reconcile_data(self, from_unreconcile=False):
                     self.manual_reference,
                 )
             elif res and res.get("amls"):
+                # TODO should be signed in currency get_reconcile_currency
                 amount = self.amount_total_signed
                 for line in res.get("amls", []):
                     reconcile_auxiliary_id, line_data = self._get_reconcile_line(
@@ -885,7 +892,7 @@ def create(self, mvals):
                     self.manual_reference,
                 )
             elif res.get("amls"):
-                amount = self.amount
+                amount = self.amount_currency or self.amount
                 for line in res.get("amls", []):
                     reconcile_auxiliary_id, line_datas = record._get_reconcile_line(
                         line, "other", is_counterpart=True, max_amount=amount, move=True
@@ -1110,7 +1117,7 @@ def add_statement(self):
 
     def _get_reconcile_currency(self):
         return (
-            self.currency_id
+            self.foreign_currency_id
             or self.journal_id.currency_id
-            or self.company_id._currency_id
+            or self.company_id.currency_id
         )

account_reconcile_oca/models/account_reconcile_abstract.py

@@ -47,18 +47,25 @@ def _get_reconcile_line(
     ):
         date = self.date if "date" in self._fields else line.date
         original_amount = amount = net_amount = line.debit - line.credit
+        line_currency = line.currency_id
         if is_counterpart:
             currency_amount = -line.amount_residual_currency or line.amount_residual
             amount = -line.amount_residual
             currency = line.currency_id or line.company_id.currency_id
             original_amount = net_amount = -line.amount_residual
             if max_amount:
-                real_currency_amount = currency._convert(
-                    currency_amount,
-                    self._get_reconcile_currency(),
-                    self.company_id,
-                    date,
-                )
+                dest_currency = self._get_reconcile_currency()
+                if currency == dest_currency:
+                    real_currency_amount = currency_amount
+                elif self.company_id.currency_id == dest_currency:
+                    real_currency_amount = amount
+                else:
+                    real_currency_amount = self.company_id.currency_id._convert(
+                        amount,
+                        dest_currency,
+                        self.company_id,
+                        date,
+                    )
                 if (
                     -real_currency_amount > max_amount > 0
                     or -real_currency_amount < max_amount < 0
@@ -76,7 +83,8 @@ def _get_reconcile_line(
                         date,
                     )
         else:
-            currency_amount = line.amount_currency
+            currency_amount = self.amount_currency or self.amount
+            line_currency = self._get_reconcile_currency()
         vals = {
             "move_id": move and line.move_id.id,
             "move": move and line.move_id.name,
@@ -91,7 +99,7 @@ def _get_reconcile_line(
             "amount": amount,
             "net_amount": amount - net_amount,
             "currency_id": self.company_id.currency_id.id,
-            "line_currency_id": line.currency_id.id,
+            "line_currency_id": line_currency.id,
             "currency_amount": currency_amount,
             "analytic_distribution": line.analytic_distribution,
             "kind": kind,