[ADD] stock reception error + fix dispute

Author: Michael Michot

dispute/models/dispute.py

@@ -119,7 +119,7 @@ def get_related_models(self):
         compute="_compute_model_ref_id_selected_id"
     )
     partner_id = fields.Many2one(
-        comodel_name="res.partner", compute="_compute_partner_id"
+        comodel_name="res.partner", compute="_compute_partner_id", store="True"
     )
     related_document_ids = fields.One2many(
         comodel_name="dispute.related.document",
@@ -207,6 +207,7 @@ def message_post(self, **kwargs):
 
 class DisputeLine(models.Model):
     _name = "dispute.line"
+    _inherit = ["mail.activity.mixin"]
     _parent_field = "_undefined"
     _description = "Dispute line"
 
@@ -239,6 +240,11 @@ def _compute_product_id(self):
                     product_id = r.model_ref_id[product_field]
             r.product_id = product_id
 
+    @api.depends("model_ref_id")
+    def _compute_price_unit(self):
+        for r in self:
+            r.price_unit = 0
+
     @api.depends("dispute_price", "qty")
     def _compute_total(self):
         for r in self:
@@ -248,6 +254,10 @@ def _compute_total(self):
     def _on_change_product_id(self):
         self.dispute_price = self.standard_price
 
+    @api.depends("model_ref_id")
+    def _compute_line_ref_id(self):
+        self.line_ref_id = self.model_ref_id and self.model_ref_id.id or "toto"
+
     dispute_id = fields.Many2one(comodel_name="dispute")
     dispute_price = fields.Float(required=True)
     company_currency = fields.Many2one(
@@ -256,6 +266,7 @@ def _on_change_product_id(self):
         related="dispute_id.company_id.currency_id",
         readonly=True,
     )
+    line_ref_id = fields.Char(compute="_compute_line_ref_id")
     comment = fields.Text()
     model_ref_id = fields.Reference(
         selection="_selection_model", string="Reference", required=True
@@ -272,7 +283,17 @@ def _on_change_product_id(self):
         ],
         required=True,
     )
+    resolution = fields.Selection(
+        [
+            ("return", "Return"),
+            ("destruction", "Destruction"),
+            ("swap", "Swap"),
+            ("accept", "Accept product"),
+        ],
+        required=True,
+    )
     standard_price = fields.Float(related="product_id.standard_price")
+    price_unit = fields.Float(compute="_compute_price_unit")
     total = fields.Monetary(compute="_compute_total", currency_field="company_currency")
 
     @api.depends("dispute_id.model_ref_id")
@@ -367,15 +388,6 @@ def update_dispute(self, values):
                     r[r._parent_field].dispute_id = dispute_id.id
                 r.dispute_line_id.dispute_id = dispute_id.id
 
-    # @api.model_create_multi
-    # def create(self, vals_list):
-    #     #TODO adapt ....
-    #     for values in vals_list:
-    #         new_line = super().create(values)
-    #         new_line.update_dispute(values)
-
-    #     return new_line
-
     def write(self, values):
         res = super().write(values)
         if "dispute_id" not in values and "dispute_line_id" not in values:

dispute/models/res_partner.py

@@ -8,15 +8,21 @@ class ResPartner(models.Model):
     _inherit = "res.partner"
 
     def _compute_dispute_count(self):
-        # retrieve all children partners and prefetch 'parent_id' on them
-        all_partners = self.search([("id", "child_of", self.ids)])
-        all_partners.read(["parent_id"])
+        # # retrieve all children partners and prefetch 'parent_id' on them
+        # all_partners = self.search([("id", "child_of", self.ids)])
+        # all_partners.read(["parent_id"])
 
         for partner in self:
             partner.dispute_count = len(
-                self.env["dispute"].search([("partner_id", "in", all_partners.ids)])
+                self.env["dispute"].search([("partner_id", "child_of", partner.id)])
             )
 
     dispute_count = fields.Integer(
         compute="_compute_dispute_count", string="Dispute Count"
     )
+
+    def action_show_dispute_list(self):
+        act_window = self.env.ref("dispute.dispute_action")
+        act_window.domain = [("partner_id", "child_of", self.id)]
+
+        return act_window.read()[0]

dispute/views/dispute.xml

@@ -257,16 +257,20 @@
                 <field name="company_currency" invisible="1" />
                 <field name="model_ref_id" />
                 <field name="product_id" />
-                <field name="standard_price" />
+                <field name="standard_price" optional="show" />
                 <field name="reason" />
+                <field name="resolution" />
                 <field name="comment" />
-                <field name="dispute_price" />
+                <field name="dispute_price" optional="show" />
                 <field name="qty" />
                 <field
                     name="total"
                     widget="monetary"
                     options="{'currency_field': 'company_currency'}"
+                    optional="show"
                 />
+                <field name="activity_ids" widget="list_activity" optional="show" />
+
             </tree>
         </field>
     </record>
@@ -280,9 +284,11 @@
                     <field name="model_ref_id" />
                     <field name="product_id" />
                     <field name="standard_price" />
+                    <field name="price_unit" />
                     <field name="dispute_price" />
                     <field name="qty" />
                     <field name="reason" />
+                    <field name="resolution" />
                 </group>
                 <group string="Comment">
                     <field name="comment" nolabel="1" />

dispute/views/res_partner_views.xml

@@ -1,14 +1,5 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-    <record id="act_res_partner_2_dispute" model="ir.actions.act_window">
-            <field name="name">Disputes</field>
-            <field name="res_model">dispute</field>
-            <field name="view_mode">tree,form,graph</field>
-            <field
-            name="context"
-        >{'search_default_partner_id': active_id, 'default_partner_id': active_id}</field>
-    </record>
-
     <record id="res_partner_view_purchase_buttons" model="ir.ui.view">
         <field name="name">res.partner.view.purchase.buttons</field>
         <field name="model">res.partner</field>
@@ -18,9 +9,8 @@
         <field name="arch" type="xml">
             <div name="button_box" position="inside">
                 <button
-                    class="oe_stat_button"
-                    name="%(dispute.act_res_partner_2_dispute)d"
-                    type="action"
+                    name="action_show_dispute_list"
+                    type="object"
                     icon="fa-exclamation-circle"
                 >
                     <field string="Dispute" name="dispute_count" widget="statinfo" />

dispute_purchase/models/purchase_dispute.py

@@ -32,3 +32,26 @@ def get_related_models(self):
         models += ["purchase.order"]
 
         return models
+
+
+class DisputeLine(models.Model):
+    _inherit = ["dispute.line"]
+
+    def _compute_price_unit(self):
+        if self.model_ref_id and self.model_ref_id._name == "purchase.order.line":
+            for r in self:
+                r.price_unit = r.model_ref_id.price_unit
+        else:
+            super()._compute_price_unit()
+
+    def _on_change_model_ref_id(self):
+        domains = {
+            "purchase.order.line": [
+                ("order_id.id", "=", self.dispute_id.model_ref_id.id)
+            ],
+            "product.product": [],
+        }
+        if self.model_ref_id and self.model_ref_id._name in domains:
+            return {"domain": {"model_ref_id": domains[self.model_ref_id._name]}}
+        else:
+            return super()._on_change_model_ref_id()

dispute_stock/__manifest__.py

@@ -11,6 +11,7 @@
     "depends": [
         "dispute",
         "stock",
+        "stock_reception_error",
     ],
     "data": [],
     "installable": True,

dispute_stock/models/__init__.py

@@ -1 +1,2 @@
 from . import stock_picking_dispute
+from . import stock_reception_error_dispute

dispute_stock/models/stock_reception_error_dispute.py

@@ -0,0 +1,47 @@
+from odoo import _, fields, models
+
+
+class Dispute(models.Model):
+    _inherit = ["dispute"]
+
+    def _selection_model(self):
+        selection_model = super()._selection_model()
+        model_name = "stock.reception.error"
+        selection_model.append((model_name, _(self.env[model_name]._description)))
+
+        return selection_model
+
+    def get_line_model_info(self, model_name):
+        line_model_info = {
+            "model_name": "stock.reception.error.line",
+            "domain": "[('stock_reception_error_id', '=', dispute_model_ref_id.id)]",
+            "product_id_field_name": "product_id",
+        }
+        if model_name == "stock.reception.error":
+            # Force selection to only compatible sub_model of model_name
+            return [line_model_info]
+        else:
+            super_info = super().get_line_model_info(model_name)
+            if not model_name:
+                super_info.append(line_model_info)
+            return super_info
+
+    def get_related_models(self):
+        models = super().get_related_models()
+
+        models += ["stock.reception.error"]
+
+        return models
+
+
+class StockReceptionError(models.Model):
+    _name = "stock.reception.error"
+    _inherit = ["stock.reception.error"]
+
+    dispute_id = fields.Many2one(comodel_name="dispute")
+
+
+class StockReceptionErrorLine(models.Model):
+    _name = "stock.reception.error.line"
+    # _parent_field = "picking_id"
+    _inherit = ["stock.reception.error.line", "dispute.line.mixin"]

setup/stock_reception_error/odoo/addons/stock_reception_error

@@ -0,0 +1 @@
+../../../../stock_reception_error

setup/stock_reception_error/setup.py

@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)

stock_reception_error/__init__.py

@@ -0,0 +1 @@
+from . import models

stock_reception_error/__manifest__.py

@@ -0,0 +1,19 @@
+# Copyright 2020 Pharmasimple (https://www.pharmasimple.be)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+{
+    "name": "Stock Reception Error",
+    "category": "Stock",
+    "summary": "Stock Reception Error",
+    "version": "14.0.1.0.0",
+    "author": "Pharmasimple",
+    "license": "AGPL-3",
+    "website": "https://github.com/akretion/phs-addons",
+    "depends": ["mail", "stock", "purchase"],
+    "data": [
+        "data/stock_reception_error.xml",
+        "security/ir.model.access.csv",
+        "views/stock_reception_error.xml",
+    ],
+    "installable": True,
+    "application": False,
+}

stock_reception_error/data/stock_reception_error.xml

@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<odoo noupdate="1">
+    <record id="stock_reception_error_seq" model="ir.sequence">
+        <field name="name">stock reception error sequence</field>
+        <field name="code">stock_reception_error</field>
+        <field name="prefix">SRE/</field>
+        <field eval="1" name="number_next" />
+        <field eval="1" name="number_increment" />
+        <field eval="5" name="padding" />
+        <field eval="False" name="company_id" />
+    </record>
+</odoo>

stock_reception_error/models/__init__.py

@@ -0,0 +1 @@
+from . import stock_reception_error

stock_reception_error/models/stock_reception_error.py

@@ -0,0 +1,89 @@
+from odoo import fields, models
+
+
+class StockReceptionError(models.Model):
+    _name = "stock.reception.error"
+    _inherit = ["mail.thread", "mail.activity.mixin"]
+    _description = "Stock Reception Error"
+
+    name = fields.Char(
+        copy=False,
+        readonly=True,
+        default=lambda self: self.env["ir.sequence"].next_by_code(
+            "stock_reception_error"
+        ),
+    )
+    state = fields.Selection(
+        default="draft",
+        selection=[
+            ("draft", "Draft"),
+            ("in_progress", "In Progress"),
+            ("done", "Done"),
+        ],
+        tracking=True,
+    )
+
+    color = fields.Integer("Color Index", default=0)
+    reception_date = fields.Date(required=True)
+    description = fields.Html()
+    line_ids = fields.One2many(
+        comodel_name="stock.reception.error.line",
+        inverse_name="stock_reception_error_id",
+    )
+    partner_id = fields.Many2one(comodel_name="res.partner", required=True)
+    purchase_order_id = fields.Many2one(
+        comodel_name="purchase.order",
+        required=True,
+        domain="[('partner_id', '=',partner_id)]",
+    )
+    responsible_id = fields.Many2one(comodel_name="res.users")
+    summary = fields.Text()
+
+
+class StockReceptionErrorLine(models.Model):
+    _name = "stock.reception.error.line"
+    _inherit = ["mail.thread", "mail.activity.mixin"]
+    _description = "Stock Reception Error line"
+
+    def _compute_name(self):
+        for r in self:
+            r.name = r.product_id and r.product_id.name or r.product_description
+
+    name = fields.Char(compute=_compute_name)
+    stock_reception_error_id = fields.Many2one(comodel_name="stock.reception.error")
+    comment = fields.Text()
+    partner_id = fields.Many2one(
+        related="stock_reception_error_id.partner_id", store=True
+    )
+    reception_date = fields.Date(
+        related="stock_reception_error_id.reception_date", store=True
+    )
+
+    product_id = fields.Many2one(comodel_name="product.product")
+    product_description = fields.Char()
+    qty = fields.Float(required=True)
+    reason = fields.Selection(
+        [
+            ("qty", "Quantity"),
+            ("quality", "Quality"),
+            ("other", "Others"),
+        ],
+        required=True,
+    )
+    resolution = fields.Selection(
+        selection=[
+            ("return", "Supplier return"),
+            ("destruction", "Destruction"),
+            ("ok_reception", "To receive"),
+        ],
+        tracking=True,
+    )
+    state = fields.Selection(
+        default="draft",
+        selection=[
+            ("draft", "Draft"),
+            ("in_progress", "In Progress"),
+            ("done", "Done"),
+        ],
+        tracking=True,
+    )

stock_reception_error/security/ir.model.access.csv

@@ -0,0 +1,3 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_sre_user,stock reception error user,model_stock_reception_error,stock.group_stock_user,1,1,1,1
+access_sre_line_user,stock reception error line user,model_stock_reception_error_line,stock.group_stock_user,1,1,1,1