[FIX] website_sale_aplicoop: auto-derive home_delivery from delivery_product_id
Make group.order.home_delivery a computed stored field that is True whenever delivery_product_id is set, eliminating the manual checkbox that could be forgotten. This fixes sale.order.home_delivery not being set when the delivery product was configured but the boolean was left unchecked. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
fb1efb0474
commit
9d614ed79e
2 changed files with 9 additions and 3 deletions
|
|
@ -134,7 +134,8 @@ class GroupOrder(models.Model):
|
||||||
|
|
||||||
# === Home delivery ===
|
# === Home delivery ===
|
||||||
home_delivery = fields.Boolean(
|
home_delivery = fields.Boolean(
|
||||||
default=False,
|
compute="_compute_home_delivery",
|
||||||
|
store=True,
|
||||||
tracking=True,
|
tracking=True,
|
||||||
help="Whether this consumer group order includes home delivery service",
|
help="Whether this consumer group order includes home delivery service",
|
||||||
)
|
)
|
||||||
|
|
@ -255,6 +256,11 @@ class GroupOrder(models.Model):
|
||||||
else:
|
else:
|
||||||
record.display_image = False
|
record.display_image = False
|
||||||
|
|
||||||
|
@api.depends("delivery_product_id")
|
||||||
|
def _compute_home_delivery(self):
|
||||||
|
for record in self:
|
||||||
|
record.home_delivery = bool(record.delivery_product_id)
|
||||||
|
|
||||||
available_products_count = fields.Integer(
|
available_products_count = fields.Integer(
|
||||||
compute="_compute_available_products_count",
|
compute="_compute_available_products_count",
|
||||||
store=False,
|
store=False,
|
||||||
|
|
|
||||||
|
|
@ -61,8 +61,8 @@
|
||||||
<field name="period" help="How often this order repeats"/>
|
<field name="period" help="How often this order repeats"/>
|
||||||
<field name="pickup_day" help="Day when members pick up orders"/>
|
<field name="pickup_day" help="Day when members pick up orders"/>
|
||||||
<field name="cutoff_day" help="Day when purchases stop"/>
|
<field name="cutoff_day" help="Day when purchases stop"/>
|
||||||
<field name="home_delivery" help="Enable home delivery option for this order"/>
|
<field name="home_delivery" readonly="1" help="Automatically enabled when a delivery product is configured"/>
|
||||||
<field name="delivery_product_id" invisible="not home_delivery" required="home_delivery" help="Product to use for home delivery"/>
|
<field name="delivery_product_id" help="Product to use for home delivery. Setting this enables home delivery."/>
|
||||||
</group>
|
</group>
|
||||||
</group>
|
</group>
|
||||||
<group string="Calculated Dates" name="calculated_dates">
|
<group string="Calculated Dates" name="calculated_dates">
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue