From 1328114dd9b9e1b4f0c7ae1c36be3d81d0354b22 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20BEAU?= <sebastien.beau@akretion.com>
Date: Fri, 23 Oct 2020 19:23:19 +0200
Subject: [PATCH] [IMP] add smart boutton on form view and action button with
 metric on tree view

---
 .../models/attachment_synchronize_task.py     | 10 ++
 .../views/attachment_queue_views.xml          | 21 ++++
 .../attachment_synchronize_task_views.xml     | 96 +++++++------------
 3 files changed, 65 insertions(+), 62 deletions(-)

diff --git a/attachment_synchronize/models/attachment_synchronize_task.py b/attachment_synchronize/models/attachment_synchronize_task.py
index 25688717e..a7f03edc3 100644
--- a/attachment_synchronize/models/attachment_synchronize_task.py
+++ b/attachment_synchronize/models/attachment_synchronize_task.py
@@ -101,6 +101,16 @@ class AttachmentSynchronizeTask(models.Model):
         "related to this task.\nAn alert will be sent to these emails if any operation "
         "on these Attachment Queue's file type fails.",
     )
+    count_attachment_failed = fields.Integer(compute="_compute_count_state")
+    count_attachment_pending = fields.Integer(compute="_compute_count_state")
+    count_attachment_done = fields.Integer(compute="_compute_count_state")
+
+    def _compute_count_state(self):
+        for record in self:
+            for state in ["failed", "pending", "done"]:
+                record["count_attachment_{}".format(state)] = \
+                    len(record.attachment_ids.filtered(lambda r: r.state == state))
+
 
     def _prepare_attachment_vals(self, data, filename):
         self.ensure_one()
diff --git a/attachment_synchronize/views/attachment_queue_views.xml b/attachment_synchronize/views/attachment_queue_views.xml
index b62863594..16f993585 100644
--- a/attachment_synchronize/views/attachment_queue_views.xml
+++ b/attachment_synchronize/views/attachment_queue_views.xml
@@ -32,4 +32,25 @@
         </field>
     </record>
 
+    <record id="attachment_queue_view_search" model="ir.ui.view">
+        <field name="model">attachment.queue</field>
+        <field name="inherit_id" ref="attachment_queue.view_attachment_queue_search"/>
+        <field name="arch" type="xml">
+            <field name="type" position="after">
+                <field name="task_id" filter_domain="[('task_id.name','ilike',self)]"/>
+            </field>
+        </field>
+    </record>
+
+    <record id="action_attachment_queue_related" model="ir.actions.act_window">
+        <field name="name">Attachments Queue</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">attachment.queue</field>
+        <field name="view_type">form</field>
+        <field name="view_mode">tree,form</field>
+        <field name="view_id" eval="False"/>
+        <field name="domain">[('task_id', '=', active_id)]</field>
+        <field name="search_view_id" ref="attachment_queue.view_attachment_queue_search"/>
+    </record>
+
 </odoo>
diff --git a/attachment_synchronize/views/attachment_synchronize_task_views.xml b/attachment_synchronize/views/attachment_synchronize_task_views.xml
index 18f4a2dbc..e25082bdd 100644
--- a/attachment_synchronize/views/attachment_synchronize_task_views.xml
+++ b/attachment_synchronize/views/attachment_synchronize_task_views.xml
@@ -9,6 +9,30 @@
                 </header>
                 <sheet>
                     <div class="oe_button_box" name="button_box">
+                        <button name="%(action_attachment_queue_related)d" type="action" class="oe_stat_button" icon="fa-thumbs-o-down" context="{'search_default_failed': 1}">
+                            <div class="o_field_widget o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="count_attachment_failed" widget="statinfo" nolabel="1"/>
+                                </span>
+                                <span class="o_stat_text">Fail</span>
+                            </div>
+                        </button>
+                        <button  name="%(action_attachment_queue_related)d" type="action" class="oe_stat_button" icon="fa-spinner" context="{'search_default_pending': 1}">
+                            <div class="o_field_widget o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="count_attachment_pending" widget="statinfo" nolabel="1"/>
+                                </span>
+                                <span class="o_stat_text">Pending</span>
+                            </div>
+                        </button>
+                        <button name="%(action_attachment_queue_related)d" type="action" class="oe_stat_button" icon="fa-thumbs-o-up" context="{'search_default_done': 1}">
+                            <div class="o_field_widget o_stat_info">
+                                <span class="o_stat_value">
+                                    <field name="count_attachment_done" widget="statinfo" nolabel="1"/>
+                                </span>
+                                <span class="o_stat_text">Success</span>
+                            </div>
+                        </button>
                         <button name="toggle_active" type="object" class="oe_stat_button" icon="fa-archive">
                             <field
                               name="active"
@@ -41,61 +65,37 @@
                         <field name="after_import" />
                         <field name="move_path" colspan="4" attrs="{'invisible':[('after_import','!=','move'), ('after_import','!=','move_rename')]}"/>
                         <field name="new_name" colspan="4" attrs="{'invisible': [('after_import','!=','rename'), ('after_import','!=','move_rename')]}"/>
+                        <field name="file_type"/>
                     </group>
-                    <group string="Attachments">
-                        <group>
-                            <field name="file_type" attrs="{'invisible':[('method_type','!=','import')]}" />
-                            <field name="failure_emails" />
-                        </group>
-                        <field name="attachment_ids" domain="[('task_id', '=', id)]" nolabel="1" attrs="{'invisible':[('method_type','!=','export')]}">
-                            <tree>
-                                <field name="name"/>
-                                <field name="datas_fname"/>
-                                <field name="file_type"/>
-                                <field name="storage_backend_id"/>
-                                <field name="type"/>
-                                <field name="create_date"/>
-                                <field name="state"/>
-                            </tree>
-                        </field>
+                    <group string="Notification">
+                        <field name="failure_emails"/>
                     </group>
                 </sheet>
             </form>
         </field>
     </record>
 
-    <record id="view_attachment_import_task_tree" model="ir.ui.view">
+    <record id="view_attachment_task_tree" model="ir.ui.view">
         <field name="model">attachment.synchronize.task</field>
         <field name="arch" type="xml">
             <tree string="Tasks" decoration-muted="active == False">
                 <field name="name" select="1"/>
                 <field name="backend_id"/>
                 <field name="filepath"/>
-                <field name="pattern"/>
-                <field name="after_import"/>
-                <field name="move_path"/>
                 <field name="active" invisible="1"/>
                 <button name="run" type="object" string="Run" icon="fa-play-circle"/>
+                <field name="count_attachment_failed" string=" "/>
+                <button name="%(action_attachment_queue_related)d" type="action" icon="fa-thumbs-o-down" context="{'search_default_failed': 1}"/>
+                <field name="count_attachment_pending" string=" "/>
+                <button  name="%(action_attachment_queue_related)d" type="action" icon="fa-spinner" context="{'search_default_pending': 1}"/>
+                <field name="count_attachment_done" string=" "/>
+                <button name="%(action_attachment_queue_related)d" type="action" icon="fa-thumbs-o-up" context="{'search_default_done': 1}"/>
                 <button name="button_duplicate_record" type="object" string="Copy" icon="fa-clone"/>
                 <button name="toggle_active" type="object" string="Active/Inactive" icon="fa-archive"/>
             </tree>
         </field>
     </record>
 
-    <record id="view_attachment_export_task_tree" model="ir.ui.view">
-        <field name="model">attachment.synchronize.task</field>
-        <field name="arch" type="xml">
-            <tree string="Tasks" decoration-muted="active == False">
-                <field name="name" select="1"/>
-                <field name="backend_id"/>
-                <field name="filepath"/>
-                <field name="active" invisible="1"/>
-                <button name="button_duplicate_record" type="object" string="Copy" icon="fa-clone"/>
-                <button name="toggle_active" type="object" string="Enable" icon="fa-archive"/>
-            </tree>
-        </field>
-    </record>
-
     <record id="view_attachment_task_search" model="ir.ui.view">
         <field name="model">attachment.synchronize.task</field>
         <field name="arch" type="xml">
@@ -121,20 +121,6 @@
         <field name="context">{'default_method_type': 'import', "active_test": False}</field>
     </record>
 
-    <record id="act_open_attachment_import_task_view_tree" model="ir.actions.act_window.view">
-        <field eval="10" name="sequence"/>
-        <field name="view_mode">tree</field>
-        <field name="view_id" ref="view_attachment_import_task_tree"/>
-        <field name="act_window_id" ref="action_attachment_import_task"/>
-    </record>
-
-    <record id="act_open_attachment_import_task_view_form" model="ir.actions.act_window.view">
-        <field eval="10" name="sequence"/>
-        <field name="view_mode">form</field>
-        <field name="view_id" ref="view_attachment_task_form"/>
-        <field name="act_window_id" ref="action_attachment_import_task"/>
-    </record>
-
     <menuitem id="menu_attachment_import_task"
     parent="base.next_id_9"
     sequence="21"
@@ -153,20 +139,6 @@
         <field name="context">{'default_method_type': 'export', "active_test": False}</field>
     </record>
 
-    <record id="act_open_attachment_export_task_view_tree" model="ir.actions.act_window.view">
-        <field eval="10" name="sequence"/>
-        <field name="view_mode">tree</field>
-        <field name="view_id" ref="view_attachment_export_task_tree"/>
-        <field name="act_window_id" ref="action_attachment_export_task"/>
-    </record>
-
-    <record id="act_open_attachment_export_task_view_form" model="ir.actions.act_window.view">
-        <field eval="10" name="sequence"/>
-        <field name="view_mode">form</field>
-        <field name="view_id" ref="view_attachment_task_form"/>
-        <field name="act_window_id" ref="action_attachment_export_task"/>
-    </record>
-
     <menuitem id="menu_attachment_export_task"
     parent="base.next_id_9"
     sequence="22"