[IMP] food_distribution: add demo data and scrap reason by chga-odoo · Pull Request #1984 · odoo/industry

chga-odoo · 2026-04-09T13:07:43Z

Added new products and BOMs
Created demo SOs, Purchase Orders, and Transfers
Added scrap reason 'Non-compliance'
Updated knowledge article as per specifications

Task: 6036099

robodoo · 2026-04-09T13:07:46Z

pgu-odoo · 2026-04-28T06:37:31Z

@chga-odoo Good work. Just a few feed backs.
Don't forget to make the version up.

pgu-odoo · 2026-04-28T04:21:29Z

+mto_route = self.env.ref('stock.route_warehouse0_mto', raise_if_not_found=False)
+for rec in self:
+    routes = rec.order_line.mapped('product_template_id.route_ids')
+    rec['x_is_mto_without_date'] = (mto_route in routes and not rec.commitment_date)


Suggested change

rec['x_is_mto_without_date'] = (mto_route in routes and not rec.commitment_date)

rec['x_is_mto_without_date'] = bool(mto_route in routes and not rec.commitment_date)

Just for better practise.

pgu-odoo · 2026-04-28T04:44:19Z

+    <field name="operation_id" ref="mrp_routing_workcenter_5"/>
+    <field name="picking_type_ids" model="stock.warehouse" eval="[(4, obj().env.ref('stock.warehouse0').manu_type_id.id)]"/>
+    <field name="test_type_id" ref="quality.test_type_instructions"/>
+    <field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for pilling and dicing.</div>]]></field>


Suggested change

<field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for pilling and dicing.</div>]]></field>

<field name="note"><![CDATA[<div>Start the process for pilling and dicing.</div>]]></field>

pgu-odoo · 2026-04-28T05:49:54Z

+      <i class="o_editor_banner_icon mb-3 fst-normal" data-oe-aria-label="Banner Info"
+        aria-label="Banner Info">💡</i>
+      <div class="o_editor_banner_content o-contenteditable-true w-100 px-3" contenteditable="true">
+        <p>This setup relies on Odoo standard MTO capability: Potato Fresh Salad operation routes includes the MTO route.</p>


Suggested change

<p>This setup relies on Odoo standard MTO capability: Potato Fresh Salad operation routes includes the MTO route.</p>

<p>This setup relies on Odoo's standard MTO capability: Potato Fresh Salad operation routes include the MTO route.</p>

pgu-odoo · 2026-04-28T05:51:53Z

+    </ul>
+    <h5>Scraping the non-compliant ingredients</h5>
+    <ul>
+      <li>Then go to Operations, Scrap menu and create a new entry.</li>


Suggested change

<li>Then go to Operations, Scrap menu and create a new entry.</li>

<li>Then go to the Operations > Scrap menu and create a new entry.</li>

looks better.

pgu-odoo · 2026-04-28T06:24:14Z

+    <field name="uom_ids" eval="[(6, 0, [ref('uom_uom_39'), ref('uom_uom_40')])]"/>
+    <field name="barcode">EAN041190012854</field>
+    <field name="is_favorite" eval="True"/>
+    <field name="responsible_id" eval="False"/>


Suggested change

<field name="responsible_id" eval="False"/>

<field name="responsible_id" eval="base.user_admin"/>

better to set it.

pgu-odoo · 2026-04-28T06:26:12Z

+  </record>
+  <record id="quality_point_21" model="quality.point">
+    <field name="name">QCP00002</field>
+    <field name="title">Plling &amp; Dicing</field>


Suggested change

<field name="title">Plling & Dicing</field>

<field name="title">Peeling & Dicing</field>

pgu-odoo · 2026-04-28T06:27:33Z

+    <field name="operation_id" ref="mrp_routing_workcenter_5"/>
+    <field name="picking_type_ids" model="stock.warehouse" eval="[(4, obj().env.ref('stock.warehouse0').manu_type_id.id)]"/>
+    <field name="test_type_id" ref="quality.test_type_instructions"/>
+    <field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for pilling and dicing.</div>]]></field>


Suggested change

<field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for pilling and dicing.</div>]]></field>

<field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for peeling and dicing.</div>]]></field>

pgu-odoo · 2026-04-28T06:29:45Z

@@ -16,4 +16,14 @@
    <field name="reminder_date_before_receipt">1</field>
    <field name="date_planned" eval="(DateTime.today().date() + relativedelta(days=3))" />
  </record>
+  <record id="purchase_order_4" model="purchase.order">
+    <field name="partner_id" ref="base_industry_data.res_partner_31"/>
+    <field name="user_id" eval="False" />


Suggested change

<field name="user_id" eval="False" />

<field name="user_id" eval="base.user_admin" />

pgu-odoo · 2026-04-28T06:33:33Z

@@ -43,11 +45,19 @@
        ('picking_id', '=', obj().env.ref('food_distribution.purchase_order_2').picking_ids.id)], limit=1).id"/>
        <value model="stock.lot" eval="{'lot_id': ref('stock_lot_5')}"/>
    </function>
+    <function name="write" model="stock.move.line">
+        <value model="stock.move.line" eval="obj().search([


You can directly use search like -

Suggested change

<value model="stock.move.line" eval="obj().search([

<value model="stock.move.line" search="[.....]"/>

vava-odoo

@robodoo delegate=pgu-odoo

vava-odoo · 2026-04-29T08:15:05Z

+    <field name="ttype">boolean</field>
+    <field name="field_description">Is MTO without date</field>
+    <field name="model_id" ref="sale.model_sale_order"/>
+    <field name="name">x_is_mto_without_date</field>


readonly not stored?

sorry, we need store=true and readonly=true here.

why store? It's only in an invisible condition in a view 🤔

Because if it is not stored, it is recomputed every time the Sales Order is opened, whereas it should only be recomputed when commitment_date or the route_id of a linked product in the sales order lines changes.

yes, that's the spirit of non-stored field 😄
on the other hand, storing such useless information in the table makes no sense.
I still think I'd unstore this.
@pgu-odoo ? any idea?

I believe performance carries more weight than storage. Any how re computation every time leads to performance issue.

as discussed, let's merge and discuss this later ;-)

- Added new products, BOMs, quality points and work center - Set MTO, relocate delivery and add worning on SO - Created demo SOs, Purchase Orders, and Transfers - Added scrap reason 'Non-compliance' - Updated knowledge article as per specifications Task: 6036099

vava-odoo · 2026-04-29T14:49:45Z

@robodoo override=ci/industry
@robodoo r+

- Added new products, BOMs, quality points and work center - Set MTO, relocate delivery and add worning on SO - Created demo SOs, Purchase Orders, and Transfers - Added scrap reason 'Non-compliance' - Updated knowledge article as per specifications closes #1984 Task: 6036099 Signed-off-by: Vallaeys Valentin (vava) <vava@odoo.com>

chga-odoo force-pushed the saas-19.2-food-distribution-imp-chga branch 3 times, most recently from affe923 to 28f9094 Compare April 17, 2026 05:07

chga-odoo force-pushed the saas-19.2-food-distribution-imp-chga branch from 28f9094 to b5a9869 Compare April 28, 2026 07:18

pgu-odoo reviewed Apr 28, 2026

View reviewed changes

chga-odoo force-pushed the saas-19.2-food-distribution-imp-chga branch from b5a9869 to a5a8489 Compare April 29, 2026 05:45

vava-odoo approved these changes Apr 29, 2026

View reviewed changes

chga-odoo force-pushed the saas-19.2-food-distribution-imp-chga branch from a5a8489 to eedaf02 Compare April 29, 2026 12:34

robodoo temporarily deployed to merge April 29, 2026 14:50 Inactive

robodoo closed this Apr 29, 2026

This was referenced Apr 29, 2026

[IMP] food_distribution: add demo data and scrap reason #2086

Closed

[IMP] food_distribution: add demo data and scrap reason #2087

Closed

	rec['x_is_mto_without_date'] = (mto_route in routes and not rec.commitment_date)
	rec['x_is_mto_without_date'] = bool(mto_route in routes and not rec.commitment_date)

	<field name="note"><![CDATA[<div data-oe-version="2.0">Start the process for pilling and dicing.</div>]]></field>
	<field name="note"><![CDATA[<div>Start the process for pilling and dicing.</div>]]></field>

	<p>This setup relies on Odoo standard MTO capability: Potato Fresh Salad operation routes includes the MTO route.</p>
	<p>This setup relies on Odoo's standard MTO capability: Potato Fresh Salad operation routes include the MTO route.</p>

	<li>Then go to Operations, Scrap menu and create a new entry.</li>
	<li>Then go to the Operations > Scrap menu and create a new entry.</li>

	<field name="responsible_id" eval="False"/>
	<field name="responsible_id" eval="base.user_admin"/>

	<field name="title">Plling & Dicing</field>
	<field name="title">Peeling & Dicing</field>

	<field name="user_id" eval="False" />
	<field name="user_id" eval="base.user_admin" />

	<value model="stock.move.line" eval="obj().search([
	<value model="stock.move.line" search="[.....]"/>

Conversation

chga-odoo commented Apr 9, 2026

Uh oh!

robodoo commented Apr 9, 2026

Uh oh!

pgu-odoo commented Apr 28, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vava-odoo left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chga-odoo Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vava-odoo commented Apr 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

chga-odoo Apr 29, 2026 •

edited

Loading