From df5beb3c8f86bae1f8029e004fb9503945c2ad88 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Fri, 24 Jul 2026 14:39:41 +0200
Subject: [PATCH] Disable buttons to upgrade/update modules if there is no
 action to perform

Closes #4960
---
 trytond/trytond/ir/module.py                  | 27 +++++++++++++------
 .../module_activate_upgrade_start_form.xml    | 11 +-------
 2 files changed, 20 insertions(+), 18 deletions(-)

diff --git a/trytond/trytond/ir/module.py b/trytond/trytond/ir/module.py
index 741d2cc80b..7e3677cae7 100644
--- a/trytond/trytond/ir/module.py
+++ b/trytond/trytond/ir/module.py
@@ -82,7 +82,11 @@ class Module(ModelSQL, ModelView):
                         ['to activate', 'to remove', 'to upgrade']),
                     'depends': ['state'],
                     },
-                'apply': {},
+                'apply': {
+                    'readonly': ~Eval('state').in_(
+                        ['to upgrade', 'to remove', 'to activate']),
+                    'depends': ['state'],
+                    },
                 })
 
     @staticmethod
@@ -240,7 +244,10 @@ class Module(ModelSQL, ModelView):
     @classmethod
     @ModelView.button_action('ir.act_module_activate_upgrade')
     def apply(cls, modules):
-        pass
+        return {
+            # Set keyword to prevent record names to be appended
+            'keyword': 'form_relate',
+            }
 
     @classmethod
     def update_list(cls):
@@ -424,7 +431,9 @@ class ModuleConfigWizard(Wizard):
 
 class ModuleActivateUpgradeStart(ModelView):
     __name__ = 'ir.module.activate_upgrade.start'
-    module_info = fields.Text('Modules to update', readonly=True)
+    modules = fields.Many2Many(
+        'ir.module', None, None,
+        "Modules to Activate / Upgrade", readonly=True)
 
 
 class ModuleActivateUpgradeDone(ModelView):
@@ -437,7 +446,10 @@ class ModuleActivateUpgrade(Wizard):
     start = StateView('ir.module.activate_upgrade.start',
         'ir.module_activate_upgrade_start_view_form', [
             Button('Cancel', 'end', 'tryton-cancel'),
-            Button('Start Upgrade', 'upgrade', 'tryton-ok', default=True),
+            Button("OK", 'upgrade', 'tryton-ok', default=True,
+                states={
+                    'readonly': ~Eval('modules'),
+                    }),
             ])
     upgrade = StateTransition()
     done = StateView('ir.module.activate_upgrade.done',
@@ -453,16 +465,15 @@ class ModuleActivateUpgrade(Wizard):
         with Transaction().new_transaction():
             super().check_access()
 
-    @staticmethod
-    def default_start(fields):
+    @classmethod
+    def default_start(cls, fields):
         pool = Pool()
         Module = pool.get('ir.module')
         modules = Module.search([
                 ('state', 'in', ['to upgrade', 'to remove', 'to activate']),
                 ])
         return {
-            'module_info': '\n'.join(x.name + ': ' + x.state
-                for x in modules),
+            'modules': [m.id for m in modules],
             }
 
     def __init__(self, session_id):
diff --git a/trytond/trytond/ir/view/module_activate_upgrade_start_form.xml b/trytond/trytond/ir/view/module_activate_upgrade_start_form.xml
index 25bfde3353..dc5cbec1d0 100644
--- a/trytond/trytond/ir/view/module_activate_upgrade_start_form.xml
+++ b/trytond/trytond/ir/view/module_activate_upgrade_start_form.xml
@@ -2,14 +2,5 @@
 <!-- This file is part of Tryton.  The COPYRIGHT file at the top level of
 this repository contains the full copyright notices and license terms. -->
 <form col="2">
-    <image name="tryton-info" xexpand="0" xfill="0"/>
-    <group col="1" id="labels" yalign="0.5">
-        <label string="Your system will be upgraded." id="upgraded"
-            xalign="0.0" xexpand="1"/>
-        <label string="Note that this operation may take a few minutes."
-            id="operation"
-            xalign="0.0" xexpand="1"/>
-    </group>
-    <separator name="module_info" colspan="2"/>
-    <field name="module_info" colspan="2"/>
+    <field name="modules" colspan="2"/>
 </form>