cockpit: Make 'name' optional in Signal handlers

Fixes #2154

Signed-off-by: Joseph Nuthalapati <njoseph@riseup.net>
Reviewed-by: James Valleroy <jvalleroy@mailbox.org>
This commit is contained in:
Joseph Nuthalapati 2021-12-04 12:11:43 +05:30 committed by James Valleroy
parent 3bf9dac201
commit e19c9dbbbe
No known key found for this signature in database
GPG Key ID: 77C0C75E7B650808

View File

@ -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',