From 1fa472aac512e63d989abbebde7f671d1cc7b07e Mon Sep 17 00:00:00 2001
From: David Alonso // Solvos <david.alonso@solvos.es>
Date: Thu, 29 Jun 2023 13:54:27 +0200
Subject: [PATCH] [IMP] helpdesk_mgmt: tags - visual and functional adjustments

Included in this commit:
* Tickets tree: added tags, as optional field.
* Tickets search view: added tags grouping functionality.
* Tickets form: usage of tag colors.
* Tags form: replaced old archive/active button with ribbon.
* Tags view: added active field.
---
 .../views/helpdesk_ticket_tag_views.xml       | 22 ++++++++-----------
 helpdesk_mgmt/views/helpdesk_ticket_views.xml | 14 +++++++++++-
 2 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/helpdesk_mgmt/views/helpdesk_ticket_tag_views.xml b/helpdesk_mgmt/views/helpdesk_ticket_tag_views.xml
index 24a614b9cb..88c4ed2f51 100644
--- a/helpdesk_mgmt/views/helpdesk_ticket_tag_views.xml
+++ b/helpdesk_mgmt/views/helpdesk_ticket_tag_views.xml
@@ -23,19 +23,14 @@
                 <header>
                 </header>
                 <sheet>
+                    <field name="active" invisible="1" />
+                    <widget
+                        name="web_ribbon"
+                        text="Archived"
+                        bg_color="bg-danger"
+                        attrs="{'invisible': [('active', '=', True)]}"
+                    />
                     <div class="oe_button_box" name="button_box">
-                        <button
-                            name="toggle_active"
-                            type="object"
-                            class="oe_stat_button"
-                            icon="fa-check"
-                        >
-                            <field
-                                name="active"
-                                widget="boolean_button"
-                                options="{&quot;terminology&quot;: &quot;active&quot;}"
-                            />
-                        </button>
                     </div>
                     <div class="oe_title">
                         <label for="name" class="oe_edit_only" />
@@ -54,8 +49,9 @@
         <field name="name">helpdesk.ticket.tag.tree</field>
         <field name="model">helpdesk.ticket.tag</field>
         <field name="arch" type="xml">
-            <tree>
+            <tree decoration-muted="not active">
                 <field name="name" />
+                <field name="active" widget="boolean_toggle" />
             </tree>
         </field>
     </record>
diff --git a/helpdesk_mgmt/views/helpdesk_ticket_views.xml b/helpdesk_mgmt/views/helpdesk_ticket_views.xml
index 26ec1d481a..9e76a93772 100644
--- a/helpdesk_mgmt/views/helpdesk_ticket_views.xml
+++ b/helpdesk_mgmt/views/helpdesk_ticket_views.xml
@@ -108,6 +108,12 @@
                         domain="[]"
                         context="{'group_by':'stage_id'}"
                     />
+                    <filter
+                        string="Tag"
+                        name="group_tag"
+                        domain="[]"
+                        context="{'group_by':'tag_ids'}"
+                    />
                 </group>
             </search>
         </field>
@@ -170,7 +176,7 @@
                             <field
                                 name="tag_ids"
                                 widget="many2many_tags"
-                                options="{'no_create_edit': True}"
+                                options="{'no_create_edit': True, 'color_field': 'color',}"
                             />
                         </group>
                     </group>
@@ -212,6 +218,12 @@
                 <field name="user_id" />
                 <field name="stage_id" />
                 <field name="priority" widget="priority" />
+                <field
+                    name="tag_ids"
+                    widget="many2many_tags"
+                    options="{'color_field': 'color'}"
+                    optional="show"
+                />
                 <field name="create_date" readonly="1" />
                 <field name="last_stage_update" />
             </tree>