fix(spp_programs): force currency symbol on the LEFT of amount on entitlement summary (#941 r3 followup)

QA flagged that the previous round-3 fix (which used odoo.tools.misc.
format_amount) rendered the currency symbol on the RIGHT for currency
records configured with position='after'. The program overview should
always show the symbol on the LEFT for consistency.

Replace format_amount with a small static helper _format_money that:

- groups thousands with Python's :, formatter,
- pads to currency.decimal_places (default 2),
- always prefixes the symbol on the left, regardless of currency.position.

Adds test_items_summary_symbol_appears_left_of_amount which explicitly
sets currency.position='after' before asserting the rendered summary
still puts the symbol left of the amount.

Author: emjay0921

spp_programs/models/program_manager_ui.py

@@ -7,7 +7,6 @@
 """
 
 from odoo import _, api, fields, models
-from odoo.tools.misc import format_amount
 
 
 def _format_recurrence(duration, rrule_type):
@@ -479,6 +478,22 @@ def _manager_detail_cel(self, concrete):
             % target_label
         )
 
+    @staticmethod
+    def _format_money(amount, currency):
+        """Render a Float amount with thousands grouping + 2-decimal precision,
+        prefixed by the currency symbol on the left. We don't use
+        odoo.tools.misc.format_amount here because it honours the currency's
+        `position` field — which puts the symbol after the amount for some
+        currency records — and the program overview should always show the
+        symbol on the left for consistency.
+        """
+        precision = currency.decimal_places if currency else 2
+        formatted = f"{amount:,.{precision}f}"
+        if not currency:
+            return formatted
+        symbol = currency.symbol or currency.name or ""
+        return f"{symbol} {formatted}".strip()
+
     def _manager_detail_entitlement_items(self, concrete):
         """Render each entitlement_item line readably."""
         lines = []
@@ -488,7 +503,7 @@ def _manager_detail_entitlement_items(self, concrete):
             multiplier_field = getattr(item, "multiplier_field", False)
             condition = getattr(item, "condition", "") or ""
             currency = getattr(item, "currency_id", False)
-            amount_with_sym = format_amount(self.env, amount, currency) if currency else f"{amount:,.2f}"
+            amount_with_sym = self._format_money(amount, currency)
             if amount_expr:
                 line = _("Amount per beneficiary: %s") % amount_expr
             elif multiplier_field:
@@ -516,7 +531,7 @@ def _manager_detail_basic_cash(self, concrete):
         currency = getattr(concrete, "currency_id", False)
 
         def _fmt(amt):
-            return format_amount(self.env, amt, currency) if currency else f"{amt:,.2f}"
+            return self._format_money(amt, currency)
 
         lines = []
         if per_cycle:

spp_programs/tests/test_manager_summary_formatting.py

@@ -111,3 +111,33 @@ def test_items_summary_includes_thousands_separator(self):
         self.assertIn("1,000,000", summary, f"summary missing thousands separator: {summary!r}")
         # And of course still no bare "1000000" without a separator.
         self.assertNotRegex(summary, r"\b1000000\b")
+
+    def test_items_summary_symbol_appears_left_of_amount(self):
+        """QA round-4: currency symbol must render on the LEFT of the amount."""
+        cash = self.env["spp.program.entitlement.manager.cash"].create(
+            {
+                "name": "Cash Symbol Position [TEST]",
+                "program_id": self.program.id,
+                "approval_definition_id": self.approval_def.id,
+            }
+        )
+        self.env["spp.program.entitlement.manager.cash.item"].create(
+            {
+                "entitlement_id": cash.id,
+                "amount": 500.0,
+            }
+        )
+        # Force currency.position to 'after' to ensure our render still puts
+        # the symbol on the left regardless of the currency record setting.
+        self.program.journal_id.currency_id.position = "after"
+        self._wrap_manager(cash)
+
+        summary = self.program.entitlement_manager_detail or ""
+        currency = self.program.journal_id.currency_id
+        sym = currency.symbol or currency.name
+        # The symbol must appear before the amount substring in the summary.
+        idx_sym = summary.find(sym)
+        idx_amt = summary.find("500.00")
+        self.assertNotEqual(idx_sym, -1, f"symbol missing from summary: {summary!r}")
+        self.assertNotEqual(idx_amt, -1, f"amount missing from summary: {summary!r}")
+        self.assertLess(idx_sym, idx_amt, f"symbol should be LEFT of amount, got: {summary!r}")