From 36eefe07a7d8b163ec2df8d82ad7b8674dbdfc86 Mon Sep 17 00:00:00 2001
From: SlWa99 <walid.slimani99@gmail.com>
Date: Tue, 30 Dec 2025 15:50:54 +0100
Subject: [PATCH 1/6] [T2845] FIX: Change donation suggestions from quantity to
 amount

---
 my_compassion/models/product_template.py | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/my_compassion/models/product_template.py b/my_compassion/models/product_template.py
index 80d0c7e4..0b87b73e 100644
--- a/my_compassion/models/product_template.py
+++ b/my_compassion/models/product_template.py
@@ -101,16 +101,25 @@ class ProductTemplate(models.Model):
     )
 
     # Donation suggestions
-    my_compassion_donation_quantity_low = fields.Integer(
-        default=1, help="Lowest quantity suggestion when making a donation"
+    my_compassion_donation_amount_low = fields.Monetary(
+        string="Lowest Amount",
+        currency_field='currency_id',
+        default=15.0,
+        help="Lowest price suggestion when making a donation"
     )
 
-    my_compassion_donation_quantity_medium = fields.Integer(
-        default=3, help="Medium quantity suggestion when making a donation"
+    my_compassion_donation_amount_medium = fields.Monetary(
+        string="Medium Amount",
+        currency_field='currency_id',
+        default=50.0,
+        help="Medium price suggestion when making a donation"
     )
 
-    my_compassion_donation_quantity_high = fields.Integer(
-        default=5, help="Highest quantity suggestion when making a donation"
+    my_compassion_donation_amount_high = fields.Monetary(
+        string="Highest Amount",
+        currency_field='currency_id',
+        default=100.0,
+        help="Highest price suggestion when making a donation"
     )
 
     def get_donation_limits(self, company, partner, sponsorship_id=None):

From e63f9b45b46ea6ab5c42c1f4cd3f9e51f66e2081 Mon Sep 17 00:00:00 2001
From: SlWa99 <walid.slimani99@gmail.com>
Date: Tue, 30 Dec 2025 15:52:17 +0100
Subject: [PATCH 2/6] [T2845] FIX: Update donation price logic to use direct
 amounts

---
 my_compassion/controllers/my2_donations.py | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/my_compassion/controllers/my2_donations.py b/my_compassion/controllers/my2_donations.py
index 1eb72363..8131091a 100644
--- a/my_compassion/controllers/my2_donations.py
+++ b/my_compassion/controllers/my2_donations.py
@@ -323,8 +323,8 @@ def my2_gifts_thank_you_page(self, **kwargs):
     def _extract_donation_order_line_fields(product_template, post):
         # Compute quantity
         price = 0
-        amount = post.get("suggested_amount")
-        if amount == "custom":
+        amount_type = post.get("suggested_amount")
+        if amount_type == "custom":
             try:
                 price = float(post.get("custom_amount"))
             except (ValueError, TypeError) as e:
@@ -333,15 +333,15 @@ def _extract_donation_order_line_fields(product_template, post):
             if price <= 0:
                 raise BadRequest()
         else:
-            quantities = {
-                "low": product_template.my_compassion_donation_quantity_low,
-                "medium": product_template.my_compassion_donation_quantity_medium,
-                "high": product_template.my_compassion_donation_quantity_high,
+            amounts = {
+                "low": product_template.my_compassion_donation_amount_low,
+                "medium": product_template.my_compassion_donation_amount_medium,
+                "high": product_template.my_compassion_donation_amount_high,
             }
-            quantity = quantities.get(amount)
-            if not quantity:
+            price = amounts.get(amount_type)
+
+            if price is None:
                 raise BadRequest()
-            price = quantity * product_template.list_price
 
         # Get frequency
         frequency = post.get("frequency")
@@ -350,6 +350,7 @@ def _extract_donation_order_line_fields(product_template, post):
         order_line_fields = {
             "product_id": product.id,
             "price_unit": price,
+            "product_uom_qty": 1.0,
             "frequency": frequency,
         }
         if product_template.my_compassion_donation_type == "gift":

From 7bd1b7f2ed0bd76040d7afe8c0b657dea9e4c38b Mon Sep 17 00:00:00 2001
From: SlWa99 <walid.slimani99@gmail.com>
Date: Tue, 30 Dec 2025 15:53:31 +0100
Subject: [PATCH 3/6] Update backecnd fields to display donation amounts

---
 my_compassion/views/product_view.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/my_compassion/views/product_view.xml b/my_compassion/views/product_view.xml
index 17fb26ed..01e0a620 100644
--- a/my_compassion/views/product_view.xml
+++ b/my_compassion/views/product_view.xml
@@ -101,9 +101,9 @@
                         string="Suggested Donations"
                         attrs="{'invisible': [('activate_for_my_compassion', '=', False)]}"
                     >
-                        <field name="my_compassion_donation_quantity_low" string="Lowest suggested quantity" />
-                        <field name="my_compassion_donation_quantity_medium" string="Medium suggested quantity" />
-                        <field name="my_compassion_donation_quantity_high" string="High suggested quantity" />
+                        <field name="my_compassion_donation_amount_low" string="Lowest suggested amount" />
+                        <field name="my_compassion_donation_amount_medium" string="Medium suggested amount" />
+                        <field name="my_compassion_donation_amount_high" string="High suggested amount" />
                     </group>
                 </page>
             </xpath>

From dd792f298add52a9726de826e3cd83a50025280d Mon Sep 17 00:00:00 2001
From: SlWa99 <walid.slimani99@gmail.com>
Date: Tue, 30 Dec 2025 15:54:14 +0100
Subject: [PATCH 4/6] [T2845] FIX: Display direct amounts and remove price
 multipication

---
 my_compassion/templates/components/my2_donation_form.xml    | 6 +++---
 my_compassion/templates/components/my2_donation_product.xml | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/my_compassion/templates/components/my2_donation_form.xml b/my_compassion/templates/components/my2_donation_form.xml
index a9cb7118..b7824b21 100644
--- a/my_compassion/templates/components/my2_donation_form.xml
+++ b/my_compassion/templates/components/my2_donation_form.xml
@@ -120,7 +120,7 @@
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-low">
                                 <t
                                     t-set="price"
-                                    t-value="product.my_compassion_donation_quantity_low * product.list_price"
+                                    t-value="product.my_compassion_donation_amount_low"
                                 />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
@@ -139,7 +139,7 @@
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-medium">
                                 <t
                                     t-set="price"
-                                    t-value="product.my_compassion_donation_quantity_medium * product.list_price"
+                                    t-value="product.my_compassion_donation_amount_medium"
                                 />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
@@ -158,7 +158,7 @@
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-high">
                                 <t
                                     t-set="price"
-                                    t-value="product.my_compassion_donation_quantity_high * product.list_price"
+                                    t-value="product.my_compassion_donation_amount_high"
                                 />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
diff --git a/my_compassion/templates/components/my2_donation_product.xml b/my_compassion/templates/components/my2_donation_product.xml
index b3ab96ff..6b4cc773 100644
--- a/my_compassion/templates/components/my2_donation_product.xml
+++ b/my_compassion/templates/components/my2_donation_product.xml
@@ -77,7 +77,7 @@
                             <t t-esc="currency_name" />
                             <t
                                 t-set="price"
-                                t-value="product.my_compassion_donation_quantity_low * product.list_price"
+                                t-value="product.my_compassion_donation_amount_low"
                             />
                             <t t-if="price % 1 == 0" t-set="price" t-value="'{0}.-'.format(int(price))" />
                             <t t-else="" t-set="price" t-value="'{0:.2f}'.format(price)" />

From 094ec785e796925ae7b9d9a25399942d47616b5a Mon Sep 17 00:00:00 2001
From: SlWa99 <walid.slimani99@gmail.com>
Date: Tue, 30 Dec 2025 15:57:20 +0100
Subject: [PATCH 5/6] [T2845] STYLE: Run pre-commit

---
 my_compassion/models/product_template.py          | 12 ++++++------
 .../templates/components/my2_donation_form.xml    | 15 +++------------
 .../templates/components/my2_donation_product.xml |  5 +----
 3 files changed, 10 insertions(+), 22 deletions(-)

diff --git a/my_compassion/models/product_template.py b/my_compassion/models/product_template.py
index 0b87b73e..11693d53 100644
--- a/my_compassion/models/product_template.py
+++ b/my_compassion/models/product_template.py
@@ -103,23 +103,23 @@ class ProductTemplate(models.Model):
     # Donation suggestions
     my_compassion_donation_amount_low = fields.Monetary(
         string="Lowest Amount",
-        currency_field='currency_id',
+        currency_field="currency_id",
         default=15.0,
-        help="Lowest price suggestion when making a donation"
+        help="Lowest price suggestion when making a donation",
     )
 
     my_compassion_donation_amount_medium = fields.Monetary(
         string="Medium Amount",
-        currency_field='currency_id',
+        currency_field="currency_id",
         default=50.0,
-        help="Medium price suggestion when making a donation"
+        help="Medium price suggestion when making a donation",
     )
 
     my_compassion_donation_amount_high = fields.Monetary(
         string="Highest Amount",
-        currency_field='currency_id',
+        currency_field="currency_id",
         default=100.0,
-        help="Highest price suggestion when making a donation"
+        help="Highest price suggestion when making a donation",
     )
 
     def get_donation_limits(self, company, partner, sponsorship_id=None):
diff --git a/my_compassion/templates/components/my2_donation_form.xml b/my_compassion/templates/components/my2_donation_form.xml
index b7824b21..c8a11bc1 100644
--- a/my_compassion/templates/components/my2_donation_form.xml
+++ b/my_compassion/templates/components/my2_donation_form.xml
@@ -118,10 +118,7 @@
                                 class="suggested-amount"
                             />
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-low">
-                                <t
-                                    t-set="price"
-                                    t-value="product.my_compassion_donation_amount_low"
-                                />
+                                <t t-set="price" t-value="product.my_compassion_donation_amount_low" />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
                                 <t t-else="" t-esc="'{0:.2f}'.format(price)" />
@@ -137,10 +134,7 @@
                                 class="suggested-amount"
                             />
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-medium">
-                                <t
-                                    t-set="price"
-                                    t-value="product.my_compassion_donation_amount_medium"
-                                />
+                                <t t-set="price" t-value="product.my_compassion_donation_amount_medium" />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
                                 <t t-else="" t-esc="'{0:.2f}'.format(price)" />
@@ -156,10 +150,7 @@
                                 class="suggested-amount"
                             />
                             <label class="mb-0 w-100" t-attf-for="#{base_name}-suggested-high">
-                                <t
-                                    t-set="price"
-                                    t-value="product.my_compassion_donation_amount_high"
-                                />
+                                <t t-set="price" t-value="product.my_compassion_donation_amount_high" />
                                 <t t-esc="product.currency_id.name" />
                                 <t t-if="price % 1 == 0" t-esc="int(price)" />
                                 <t t-else="" t-esc="'{0:.2f}'.format(price)" />
diff --git a/my_compassion/templates/components/my2_donation_product.xml b/my_compassion/templates/components/my2_donation_product.xml
index 6b4cc773..14d7417b 100644
--- a/my_compassion/templates/components/my2_donation_product.xml
+++ b/my_compassion/templates/components/my2_donation_product.xml
@@ -75,10 +75,7 @@
                         From
                         <span class="body-small bold">
                             <t t-esc="currency_name" />
-                            <t
-                                t-set="price"
-                                t-value="product.my_compassion_donation_amount_low"
-                            />
+                            <t t-set="price" t-value="product.my_compassion_donation_amount_low" />
                             <t t-if="price % 1 == 0" t-set="price" t-value="'{0}.-'.format(int(price))" />
                             <t t-else="" t-set="price" t-value="'{0:.2f}'.format(price)" />
                             <t t-esc="price" />

From dcb8a3994615ec4ff70c5c028f150f7ad9c81410 Mon Sep 17 00:00:00 2001
From: Walid <94690298+SlWa99@users.noreply.github.com>
Date: Tue, 30 Dec 2025 16:11:34 +0100
Subject: [PATCH 6/6] Update my_compassion/controllers/my2_donations.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
---
 my_compassion/controllers/my2_donations.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/my_compassion/controllers/my2_donations.py b/my_compassion/controllers/my2_donations.py
index 8131091a..b78e6a0a 100644
--- a/my_compassion/controllers/my2_donations.py
+++ b/my_compassion/controllers/my2_donations.py
@@ -340,7 +340,7 @@ def _extract_donation_order_line_fields(product_template, post):
             }
             price = amounts.get(amount_type)
 
-            if price is None:
+            if price is None or price <= 0:
                 raise BadRequest()
 
         # Get frequency