From dea732e270c213ff88747d118214bdac632dfe3d Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Wed, 27 Sep 2023 23:41:58 +0200
Subject: [PATCH] account_usability_ak: add blocked on account.move and show
 warning banner on invoice form view

---
 .../models/account_move.py                    | 28 ++++++++++++++++++
 .../views/account_move.xml                    | 29 +++++++++++++++++++
 2 files changed, 57 insertions(+)

diff --git a/account_usability_akretion/models/account_move.py b/account_usability_akretion/models/account_move.py
index 38317b8..1a87d0c 100644
--- a/account_usability_akretion/models/account_move.py
+++ b/account_usability_akretion/models/account_move.py
@@ -36,6 +36,34 @@ class AccountMove(models.Model):
         compute="_compute_sales_dates",
         help="This information appear on invoice qweb report "
              "(you may use it for your own report)")
+        # There is a native "blocked" field (bool) on account.move.line
+    # We want to have that field on invoices to improve usability
+    # while keeping compatibility with the standard Odoo datamodel
+    blocked = fields.Boolean(
+        compute="_compute_blocked",
+        inverse="_inverse_blocked",
+        store=True,
+        string="Dispute",
+        tracking=True,
+    )
+
+    @api.depends("line_ids", "line_ids.blocked")
+    def _compute_blocked(self):
+        for move in self:
+            move.blocked = any(
+                [
+                    l.blocked
+                    for l in move.line_ids
+                    if l.account_id.account_type in ("liability_payable", "asset_receivable")
+                ]
+            )
+
+    def _inverse_blocked(self):
+        for move in self:
+            for line in move.line_ids.filtered(
+                lambda l: l.account_id.account_type in ("liability_payable", "asset_receivable")
+            ):
+                line.blocked = move.blocked
 
     def _compute_has_discount(self):
         prec = self.env['decimal.precision'].precision_get('Discount')
diff --git a/account_usability_akretion/views/account_move.xml b/account_usability_akretion/views/account_move.xml
index 3b4576c..4ea1689 100644
--- a/account_usability_akretion/views/account_move.xml
+++ b/account_usability_akretion/views/account_move.xml
@@ -43,6 +43,16 @@
         <xpath expr="//field[@name='invoice_line_ids']/tree/field[@name='product_id']" position="after">
             <field name="product_barcode" optional="hide"/>
         </xpath>
+        <field name="invoice_source_email" position="after">
+            <field name="blocked"/>
+        </field>
+        <div role="alert" position="after">
+            <div id="warn_blocked" groups="account.group_account_invoice,account.group_account_readonly"
+                    class="alert alert-warning" role="alert" style="margin-bottom:0px;"
+                    attrs="{'invisible': ['|', ('move_type', 'not in', ('in_invoice', 'in_refund', 'out_invoice', 'out_refund')), ('blocked', '=', False)]}">
+                    This <field name="move_type"/> is marked as <b>disputed</b>.
+            </div>
+        </div>
         <xpath expr="//button[@name='open_duplicated_ref_bill_view']/.." position="attributes">
             <!-- show duplicate warning not only in draft state, but also in posted state -->
             <attribute name="attrs">{'invisible': ['|', ('state', '=', 'cancel'), ('duplicated_ref_ids', '=', [])]}</attribute>
@@ -77,6 +87,8 @@
              <filter name="sent" string="Sent" domain="[('is_move_sent', '=', True), ('move_type', 'in', ('out_invoice', 'out_refund'))]"/>
             <separator/>
             <filter name="no_attachment" string="Missing Attachment" domain="[('has_attachment', '=', False)]"/>
+            <separator/>
+            <filter name="dispute" string="Dispute" domain="[('blocked', '=', True)]"/>
         </filter>
         <filter name="salesperson" position="before">
             <filter name="commercial_partner_groupby" string="Commercial Partner" context="{'group_by': 'commercial_partner_id'}"/>
@@ -87,6 +99,23 @@
     </field>
 </record>
 
+<record id="view_move_line_form" model="ir.ui.view">
+    <field name="model">account.move.line</field>
+    <field name="inherit_id" ref="account.view_move_line_form"/>
+    <field name="arch" type="xml">
+        <!-- The field 'blocked' is alone in it's block
+        We don't want to display an empty block, so we put the attrs on the group
+        The drawback of this is that, if someone added a field in that group,
+        he won't see the field when internal_type is not payable/receivable -->
+        <xpath expr="//field[@name='blocked']/.." position="attributes">
+            <attribute name="attrs">{'invisible': [('account_type', 'not in', ('liability_payable', 'asset_receivable'))]}</attribute>
+        </xpath>
+        <field name="account_id" position="after">
+            <field name="account_type" invisible="1"/>
+        </field>
+    </field>
+</record>
+
 <record id="view_move_line_tree" model="ir.ui.view">
     <field name="model">account.move.line</field>
     <field name="inherit_id" ref="account.view_move_line_tree"/>