From e19c9dbbbe0becfe347e420c0bd924a71efe8db5 Mon Sep 17 00:00:00 2001 From: Joseph Nuthalapati Date: Sat, 4 Dec 2021 12:11:43 +0530 Subject: [PATCH] cockpit: Make 'name' optional in Signal handlers Fixes #2154 Signed-off-by: Joseph Nuthalapati Reviewed-by: James Valleroy --- plinth/modules/cockpit/__init__.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/plinth/modules/cockpit/__init__.py b/plinth/modules/cockpit/__init__.py index 648ec78e8..0a5bc3a4b 100644 --- a/plinth/modules/cockpit/__init__.py +++ b/plinth/modules/cockpit/__init__.py @@ -113,19 +113,19 @@ def setup(helper, old_version=None): helper.call('post', app.enable) -def on_domain_added(sender, domain_type, name, description='', services=None, - **kwargs): +def on_domain_added(sender, domain_type, name='', description='', + services=None, **kwargs): """Handle addition of a new domain.""" - if not app.needs_setup(): + if name and not app.needs_setup(): if name not in utils.get_domains(): actions.superuser_run('cockpit', ['add-domain', name]) actions.superuser_run('service', ['try-restart', CockpitApp.DAEMON]) -def on_domain_removed(sender, domain_type, name, **kwargs): +def on_domain_removed(sender, domain_type, name='', **kwargs): """Handle removal of a domain.""" - if not app.needs_setup(): + if name and not app.needs_setup(): if name in utils.get_domains(): actions.superuser_run('cockpit', ['remove-domain', name]) actions.superuser_run('service',