From efaf288d945037927cf30de8ac1fbff2dc11c2a6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Niels=20G=C3=B6ttsch?= <ng@ife.de>
Date: Tue, 23 Jul 2024 23:03:22 +0200
Subject: [PATCH] [imp] add batch invoicing, improve max_qty constraint

---
 sale_order_batch/models/sale_order_batch.py   | 38 ++++++++++++++++---
 .../views/sale_order_batch_product_views.xml  |  1 +
 .../views/sale_order_batch_views.xml          | 34 ++++++++++++++++-
 3 files changed, 66 insertions(+), 7 deletions(-)

diff --git a/sale_order_batch/models/sale_order_batch.py b/sale_order_batch/models/sale_order_batch.py
index 59afb0a..1bea44b 100644
--- a/sale_order_batch/models/sale_order_batch.py
+++ b/sale_order_batch/models/sale_order_batch.py
@@ -47,6 +47,8 @@ class SaleOrderBatch(models.Model):
     sale_order_ids = fields.One2many("sale.order", "batch_id")
     sale_order_count = fields.Integer(compute="_compute_sale_order_count")
     sale_order_line_ids = fields.Many2many("sale.order.line", compute="_compute_sale_order_line_ids", store=True)
+    invoice_ids = fields.Many2many("account.move", compute="_compute_invoice_ids")
+    invoice_count = fields.Integer(compute="_compute_invoice_ids")
     amount_total = fields.Float(compute="_compute_amount_total", string="Total")
     product_ids = fields.One2many("sale.order.batch.product", "batch_id")
     product_count = fields.Integer(compute="_compute_product_count")
@@ -60,17 +62,24 @@ class SaleOrderBatch(models.Model):
             else:
                 batch.validity_date = False
 
-    @api.depends("sale_order_ids")
-    def _compute_sale_order_count(self):
-        for batch in self:
-            batch.sale_order_count = len(batch.sale_order_ids)
-
     @api.depends("sale_order_ids.order_line")
     def _compute_sale_order_line_ids(self):
         for batch in self:
             order_lines = self.env["sale.order.line"].search([("order_id", "in", batch.sale_order_ids.ids)])
             batch.sale_order_line_ids = order_lines
 
+    @api.depends("sale_order_ids")
+    def _compute_sale_order_count(self):
+        for batch in self:
+            batch.sale_order_count = len(batch.sale_order_ids)
+
+    @api.depends("sale_order_ids.invoice_ids")
+    def _compute_invoice_ids(self):
+        for batch in self:
+            invoices = batch.sale_order_ids.mapped("invoice_ids")
+            batch.invoice_ids = invoices
+            batch.invoice_count = len(invoices)
+
     @api.depends("sale_order_ids.amount_total")
     def _compute_amount_total(self):
         for batch in self:
@@ -113,6 +122,25 @@ class SaleOrderBatch(models.Model):
             result = {"type": "ir.actions.act_window_close"}
         return result
 
+    def action_view_invoice(self):
+        invoices = self.mapped("invoice_ids")
+        action = self.env["ir.actions.actions"]._for_xml_id("account.action_move_out_invoice_type")
+        if len(invoices) > 1:
+            action["domain"] = [("id", "in", invoices.ids)]
+        elif len(invoices) == 1:
+            form_view = [(self.env.ref("account.view_move_form").id, "form")]
+            if "views" in action:
+                action["views"] = form_view + [(state, view) for state, view in action["views"] if view != "form"]
+            else:
+                action["views"] = form_view
+            action["res_id"] = invoices.id
+        else:
+            action = {"type": "ir.actions.act_window_close"}
+
+        context = {"default_move_type": "out_invoice"}
+        action["context"] = context
+        return action
+
     def action_confirm(self):
         for batch in self:
             orders = batch.with_context(bypass_batch=True).sale_order_ids
diff --git a/sale_order_batch/views/sale_order_batch_product_views.xml b/sale_order_batch/views/sale_order_batch_product_views.xml
index a1661e6..b6e6a46 100644
--- a/sale_order_batch/views/sale_order_batch_product_views.xml
+++ b/sale_order_batch/views/sale_order_batch_product_views.xml
@@ -26,6 +26,7 @@
                             <field name="sale_order_line_ids" widget="section_and_note_one2many">
                                 <tree editable="bottom" create="0">
                                     <field name="sequence" widget="handle"/>
+                                    <field name="name"/>
                                     <field name="order_partner_id"/>
                                     <field name="product_uom_qty"/>
                                     <field name="price_unit"/>
diff --git a/sale_order_batch/views/sale_order_batch_views.xml b/sale_order_batch/views/sale_order_batch_views.xml
index 9370be8..88147e8 100644
--- a/sale_order_batch/views/sale_order_batch_views.xml
+++ b/sale_order_batch/views/sale_order_batch_views.xml
@@ -29,6 +29,19 @@
                         type="object"
                         attrs="{'invisible': [('state', '!=','open')]}"
                     />
+                    <button
+                        id="create_invoices"
+                        name="%(sale.action_view_sale_advance_payment_inv)d"
+                        string="Create Invoices"
+                        type="action"
+                        class="btn-primary"
+                        context="{
+                        'default_sale_order_ids': sale_order_ids,
+                        }"
+                        data-hotkey="q"
+                        attrs="{'invisible': [('state', '!=', 'closed')]}"
+                    />
+
                     <field name="state" widget="statusbar" options="{'clickable': 'true'}"/>
                 </header>
                 <div
@@ -58,6 +71,15 @@
                         >
                             <field string="Products" name="product_count" widget="statinfo"/>
                         </button>
+                        <button
+                            name="action_view_invoice"
+                            type="object"
+                            class="oe_stat_button"
+                            icon="fa-pencil-square-o"
+                            attrs="{'invisible': [('invoice_count', '=', 0)]}"
+                        >
+                            <field name="invoice_count" widget="statinfo" string="Invoices"/>
+                        </button>
                     </div>
                     <h1>
                         <field name="name"/>
@@ -76,7 +98,6 @@
                             <field
                                 name="sale_order_ids"
                                 domain="[('state','in',['draft','sent']),('batch_id','=',False)]"
-                                attrs="{'readonly': [('state', 'in', ('closed'))]}"
                             >
                                 <tree>
                                     <field name="name"/>
@@ -84,6 +105,14 @@
                                     <field name="date_order"/>
                                     <field name="amount_untaxed"/>
                                     <field name="amount_total"/>
+                                    <field
+                                        name="invoice_status"
+                                        decoration-success="invoice_status == 'invoiced'"
+                                        decoration-info="invoice_status == 'to invoice'"
+                                        decoration-warning="invoice_status == 'upselling'"
+                                        widget="badge"
+                                        optional="show"
+                                    />
                                 </tree>
                             </field>
                         </page>
@@ -95,8 +124,9 @@
                             >
                                 <tree editable="bottom">
                                     <field name="sequence" widget="handle"/>
+                                    <field name="order_id"/>
                                     <field name="order_partner_id"/>
-                                    <field name="product_template_id"/>
+                                    <field name="product_template_id" optional="hide"/>
                                     <field name="name"/>
                                     <field name="product_uom_qty"/>
                                     <field name="price_unit"/>