From cfe776be4c40f6e7d5b83b588f1bfdf4d5580dd7 Mon Sep 17 00:00:00 2001
From: gsinghpal <gur_preet.singh@outlook.com>
Date: Mon, 27 Apr 2026 20:27:35 -0400
Subject: [PATCH] =?UTF-8?q?chore:=20session=20housekeeping=20=E2=80=94=20t?=
 =?UTF-8?q?ank=20UX,=20plating=20menu=20defaults,=20WO=20label?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- fusion_plating: tank field labels (Code → Tank Number, Tank → Tank Name)
  + state-control header buttons (Mark Empty/Filled/In Use/Draining/
  Maintenance/Out of Service) with chatter audit logging.
- fusion_plating_configurator: Plating app default landing screen = Sale
  Orders, while keeping menu name as 'Plating'.
- fusion_plating_jobs: SO smart-button label 'Plating Jobs' → 'WO'.

Already deployed and verified on entech earlier in the session.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 fusion_plating/fusion_plating/__manifest__.py |  2 +-
 .../fusion_plating/models/fp_tank.py          | 31 +++++++++++++++++--
 .../fusion_plating/views/fp_tank_views.xml    | 20 +++++++++++-
 .../__manifest__.py                           |  2 +-
 .../views/fp_configurator_menu.xml            |  5 +++
 .../fusion_plating_jobs/__manifest__.py       |  2 +-
 .../views/sale_order_views.xml                |  2 +-
 7 files changed, 57 insertions(+), 7 deletions(-)

diff --git a/fusion_plating/fusion_plating/__manifest__.py b/fusion_plating/fusion_plating/__manifest__.py
index a8b88b1f..996a73c6 100644
--- a/fusion_plating/fusion_plating/__manifest__.py
+++ b/fusion_plating/fusion_plating/__manifest__.py
@@ -5,7 +5,7 @@
 
 {
     'name': 'Fusion Plating',
-    'version': '19.0.9.2.0',
+    'version': '19.0.9.3.0',
     'category': 'Manufacturing/Plating',
     'summary': 'Core plating / metal finishing ERP: facilities, processes, tanks, baths, jobs, operators.',
     'description': """
diff --git a/fusion_plating/fusion_plating/models/fp_tank.py b/fusion_plating/fusion_plating/models/fp_tank.py
index 20d0b321..acd9b0f0 100644
--- a/fusion_plating/fusion_plating/models/fp_tank.py
+++ b/fusion_plating/fusion_plating/models/fp_tank.py
@@ -22,12 +22,12 @@ class FpTank(models.Model):
     _order = 'facility_id, work_center_id, sequence, code'
 
     name = fields.Char(
-        string='Tank',
+        string='Tank Name',
         required=True,
         tracking=True,
     )
     code = fields.Char(
-        string='Code',
+        string='Tank Number',
         required=True,
         tracking=True,
         help='Short unique tank identifier (e.g. "T-01", "EN-A1").',
@@ -174,3 +174,30 @@ class FpTank(models.Model):
             if not vals.get('qr_code') and vals.get('code'):
                 vals['qr_code'] = f"FP-TANK:{vals['code']}"
         return super().create(vals_list)
+
+    # ----- State transition actions ---------------------------------------
+    def _set_state(self, new_state, message):
+        for rec in self:
+            old = dict(rec._fields['state'].selection).get(rec.state, rec.state)
+            new = dict(rec._fields['state'].selection).get(new_state, new_state)
+            rec.state = new_state
+            rec.message_post(body=f"{message} ({old} → {new}) by {self.env.user.name}")
+        return True
+
+    def action_set_empty(self):
+        return self._set_state('empty', 'Tank marked Empty')
+
+    def action_set_filled(self):
+        return self._set_state('filled', 'Tank marked Filled')
+
+    def action_set_in_use(self):
+        return self._set_state('in_use', 'Tank marked In Use')
+
+    def action_set_draining(self):
+        return self._set_state('draining', 'Tank marked Draining')
+
+    def action_set_maintenance(self):
+        return self._set_state('maintenance', 'Tank marked for Maintenance')
+
+    def action_set_out_of_service(self):
+        return self._set_state('out_of_service', 'Tank marked Out of Service')
diff --git a/fusion_plating/fusion_plating/views/fp_tank_views.xml b/fusion_plating/fusion_plating/views/fp_tank_views.xml
index d0a163d4..02f18d8c 100644
--- a/fusion_plating/fusion_plating/views/fp_tank_views.xml
+++ b/fusion_plating/fusion_plating/views/fp_tank_views.xml
@@ -35,6 +35,24 @@
         <field name="arch" type="xml">
             <form string="Tank">
                 <header>
+                    <button name="action_set_empty" type="object"
+                            string="Mark Empty" class="btn-secondary"
+                            invisible="state == 'empty'"/>
+                    <button name="action_set_filled" type="object"
+                            string="Mark Filled" class="btn-primary"
+                            invisible="state == 'filled'"/>
+                    <button name="action_set_in_use" type="object"
+                            string="Mark In Use" class="btn-success"
+                            invisible="state == 'in_use'"/>
+                    <button name="action_set_draining" type="object"
+                            string="Mark Draining" class="btn-warning"
+                            invisible="state == 'draining'"/>
+                    <button name="action_set_maintenance" type="object"
+                            string="Mark for Maintenance" class="btn-warning"
+                            invisible="state == 'maintenance'"/>
+                    <button name="action_set_out_of_service" type="object"
+                            string="Mark Out of Service" class="btn-danger"
+                            invisible="state == 'out_of_service'"/>
                     <field name="state" widget="statusbar"
                            statusbar_visible="empty,filled,in_use,draining,maintenance"/>
                 </header>
@@ -42,7 +60,7 @@
                     <widget name="web_ribbon" title="Out of Service" bg_color="text-bg-danger"
                             invisible="state != 'out_of_service'"/>
                     <div class="oe_title">
-                        <label for="name"/>
+                        <label for="name" string="Tank Name"/>
                         <h1><field name="name" placeholder="e.g. EN Plating Tank A1"/></h1>
                         <div class="text-muted">
                             <field name="code" placeholder="T-01"/>
diff --git a/fusion_plating/fusion_plating_configurator/__manifest__.py b/fusion_plating/fusion_plating_configurator/__manifest__.py
index ead9a7f1..329bbf31 100644
--- a/fusion_plating/fusion_plating_configurator/__manifest__.py
+++ b/fusion_plating/fusion_plating_configurator/__manifest__.py
@@ -5,7 +5,7 @@
 
 {
     'name': 'Fusion Plating — Configurator',
-    'version': '19.0.17.13.0',
+    'version': '19.0.17.15.0',
     'category': 'Manufacturing/Plating',
     'summary': 'Quotation configurator with part catalog, coating configs, and formula-based pricing engine.',
     'description': """
diff --git a/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml b/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
index 3e9b7675..0b9db7d1 100644
--- a/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
+++ b/fusion_plating/fusion_plating_configurator/views/fp_configurator_menu.xml
@@ -15,6 +15,11 @@
         <field name="context">{'default_customer_rank': 1}</field>
     </record>
 
+    <!-- Default landing screen for the Plating app = Sale Orders -->
+    <menuitem id="fusion_plating.menu_fp_root"
+              name="Plating"
+              action="action_fp_sale_orders"/>
+
     <!-- ===== SALES submenu under Fusion Plating root ===== -->
     <menuitem id="menu_fp_sales"
               name="Sales"
diff --git a/fusion_plating/fusion_plating_jobs/__manifest__.py b/fusion_plating/fusion_plating_jobs/__manifest__.py
index 03977918..b4aa422f 100644
--- a/fusion_plating/fusion_plating_jobs/__manifest__.py
+++ b/fusion_plating/fusion_plating_jobs/__manifest__.py
@@ -3,7 +3,7 @@
 # License OPL-1 (Odoo Proprietary License v1.0)
 {
     'name': 'Fusion Plating — Native Jobs',
-    'version': '19.0.6.22.0',
+    'version': '19.0.6.23.0',
     'category': 'Manufacturing/Plating',
     'summary': 'Native plating job model — replaces mrp.production / mrp.workorder bridge.',
     'author': 'Nexa Systems Inc.',
diff --git a/fusion_plating/fusion_plating_jobs/views/sale_order_views.xml b/fusion_plating/fusion_plating_jobs/views/sale_order_views.xml
index 4e5c94f9..64baee05 100644
--- a/fusion_plating/fusion_plating_jobs/views/sale_order_views.xml
+++ b/fusion_plating/fusion_plating_jobs/views/sale_order_views.xml
@@ -21,7 +21,7 @@
                 <button name="action_view_fp_jobs" type="object"
                         class="oe_stat_button" icon="fa-cogs">
                     <field name="x_fc_fp_job_count" widget="statinfo"
-                           string="Plating Jobs"/>
+                           string="WO"/>
                 </button>
                 <!-- Sarah/Tom path: SO → Certificates (one click instead -->
                 <!-- of two via the job). Hidden until a cert exists.     -->