replace next-screen/prev-screen signals with method calls

This commit is contained in:
Michael Hudson-Doyle 2019-12-16 23:02:03 +13:00
parent 7e7ca080f6
commit 2765e6a773
14 changed files with 34 additions and 36 deletions

View File

@ -246,7 +246,7 @@ class IdentityController(BaseController):
def cancel(self): def cancel(self):
# You can only go back if we haven't created a user yet. # You can only go back if we haven't created a user yet.
if self.model.user is None: if self.model.user is None:
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def login(self): def login(self):
title = "Configuration Complete" title = "Configuration Complete"

View File

@ -25,7 +25,7 @@ class WelcomeController(BaseController):
self.ui.set_body(view) self.ui.set_body(view)
def done(self): def done(self):
self.signal.emit_signal('next-screen') self.app.next_screen()
def cancel(self): def cancel(self):
# Can't go back from here! # Can't go back from here!

View File

@ -313,14 +313,14 @@ class FilesystemController(BaseController):
self.manual() self.manual()
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def finish(self): def finish(self):
log.debug("FilesystemController.finish next-screen") log.debug("FilesystemController.finish next_screen")
# start curtin install in background # start curtin install in background
self.signal.emit_signal('installprogress:filesystem-config-done') self.signal.emit_signal('installprogress:filesystem-config-done')
# switch to next screen # switch to next screen
self.signal.emit_signal('next-screen') self.app.next_screen()
def create_mount(self, fs, spec): def create_mount(self, fs, spec):
if spec.get('mount') is None: if spec.get('mount') is None:

View File

@ -41,14 +41,14 @@ class IdentityController(BaseController):
self.done(d) self.done(d)
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def done(self, user_spec): def done(self, user_spec):
safe_spec = user_spec.copy() safe_spec = user_spec.copy()
safe_spec['password'] = '<REDACTED>' safe_spec['password'] = '<REDACTED>'
log.debug( log.debug(
"IdentityController.done next-screen user_spec=%s", "IdentityController.done next_screen user_spec=%s",
safe_spec) safe_spec)
self.model.add_user(user_spec) self.model.add_user(user_spec)
self.signal.emit_signal('installprogress:identity-config-done') self.signal.emit_signal('installprogress:identity-config-done')
self.signal.emit_signal('next-screen') self.app.next_screen()

View File

@ -55,11 +55,11 @@ class KeyboardController(BaseController):
async def apply_settings(self, setting): async def apply_settings(self, setting):
await self.model.set_keyboard(setting) await self.model.set_keyboard(setting)
log.debug("KeyboardController next-screen") log.debug("KeyboardController next_screen")
self.signal.emit_signal('next-screen') self.app.next_screen()
def done(self, setting): def done(self, setting):
schedule_task(self.apply_settings(setting)) schedule_task(self.apply_settings(setting))
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()

View File

@ -93,7 +93,7 @@ class MirrorController(BaseController):
self.done(self.model.mirror) self.done(self.model.mirror)
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def serialize(self): def serialize(self):
return self.model.mirror return self.model.mirror
@ -102,7 +102,7 @@ class MirrorController(BaseController):
self.model.mirror = data self.model.mirror = data
def done(self, mirror): def done(self, mirror):
log.debug("MirrorController.done next-screen mirror=%s", mirror) log.debug("MirrorController.done next_screen mirror=%s", mirror)
if mirror != self.model.mirror: if mirror != self.model.mirror:
self.model.mirror = mirror self.model.mirror = mirror
self.signal.emit_signal('next-screen') self.app.next_screen()

View File

@ -35,7 +35,7 @@ class ProxyController(BaseController):
self.done(self.answers['proxy']) self.done(self.answers['proxy'])
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def serialize(self): def serialize(self):
return self.model.proxy return self.model.proxy
@ -44,9 +44,9 @@ class ProxyController(BaseController):
self.model.proxy = data self.model.proxy = data
def done(self, proxy): def done(self, proxy):
log.debug("ProxyController.done next-screen proxy=%s", proxy) log.debug("ProxyController.done next_screen proxy=%s", proxy)
if proxy != self.model.proxy: if proxy != self.model.proxy:
self.model.proxy = proxy self.model.proxy = proxy
os.environ['http_proxy'] = os.environ['https_proxy'] = proxy os.environ['http_proxy'] = os.environ['https_proxy'] = proxy
self.signal.emit_signal('network-proxy-set') self.signal.emit_signal('network-proxy-set')
self.signal.emit_signal('next-screen') self.app.next_screen()

View File

@ -235,8 +235,8 @@ class RefreshController(BaseController):
raise Skip() raise Skip()
def done(self, sender=None): def done(self, sender=None):
log.debug("RefreshController.done next-screen") log.debug("RefreshController.done next_screen")
self.signal.emit_signal('next-screen') self.app.next_screen()
def cancel(self, sender=None): def cancel(self, sender=None):
self.signal.emit_signal('prev-screen') self.app.prev_screen()

View File

@ -161,11 +161,11 @@ class SnapListController(BaseController):
def done(self, snaps_to_install): def done(self, snaps_to_install):
log.debug( log.debug(
"SnapListController.done next-screen snaps_to_install=%s", "SnapListController.done next_screen snaps_to_install=%s",
snaps_to_install) snaps_to_install)
self.model.set_installed_list(snaps_to_install) self.model.set_installed_list(snaps_to_install)
self.signal.emit_signal("installprogress:snap-config-done") self.signal.emit_signal("installprogress:snap-config-done")
self.signal.emit_signal("next-screen") self.app.next_screen()
def cancel(self, sender=None): def cancel(self, sender=None):
self.signal.emit_signal("prev-screen") self.app.prev_screen()

View File

@ -58,7 +58,7 @@ class SSHController(BaseController):
self.fetch_ssh_keys(d) self.fetch_ssh_keys(d)
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def _fetch_cancel(self): def _fetch_cancel(self):
if self._fetch_task is None: if self._fetch_task is None:
@ -111,10 +111,10 @@ class SSHController(BaseController):
self._fetch_task = schedule_task(self._fetch_ssh_keys(user_spec)) self._fetch_task = schedule_task(self._fetch_ssh_keys(user_spec))
def done(self, result): def done(self, result):
log.debug("SSHController.done next-screen result=%s", result) log.debug("SSHController.done next_screen result=%s", result)
self.model.install_server = result['install_server'] self.model.install_server = result['install_server']
self.model.authorized_keys = result.get('authorized_keys', []) self.model.authorized_keys = result.get('authorized_keys', [])
self.model.pwauth = result.get('pwauth', True) self.model.pwauth = result.get('pwauth', True)
self.model.ssh_import_id = result.get('ssh_import_id', None) self.model.ssh_import_id = result.get('ssh_import_id', None)
self.signal.emit_signal('installprogress:ssh-config-done') self.signal.emit_signal('installprogress:ssh-config-done')
self.signal.emit_signal('next-screen') self.app.next_screen()

View File

@ -45,10 +45,10 @@ class WelcomeController(BaseController):
self.done(self.answers['lang']) self.done(self.answers['lang'])
def done(self, code): def done(self, code):
log.debug("WelcomeController.done %s next-screen", code) log.debug("WelcomeController.done %s next_screen", code)
self.signal.emit_signal('l10n:language-selected', code) self.signal.emit_signal('l10n:language-selected', code)
self.model.switch_language(code) self.model.switch_language(code)
self.signal.emit_signal('next-screen') self.app.next_screen()
def cancel(self): def cancel(self):
# Can't go back from here! # Can't go back from here!

View File

@ -646,11 +646,11 @@ class ZdevController(BaseController):
self.ui.set_body(ZdevView(self)) self.ui.set_body(ZdevView(self))
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def done(self): def done(self):
# switch to next screen # switch to next screen
self.signal.emit_signal('next-screen') self.app.next_screen()
def chzdev(self, action, zdevinfo): def chzdev(self, action, zdevinfo):
if self.opts.dry_run: if self.opts.dry_run:

View File

@ -194,13 +194,13 @@ class NetworkController(BaseController):
self.observer.trigger_scan(dev.ifindex) self.observer.trigger_scan(dev.ifindex)
def done(self): def done(self):
log.debug("NetworkController.done next-screen") log.debug("NetworkController.done next_screen")
self.model.has_network = bool( self.model.has_network = bool(
self.network_event_receiver.default_routes) self.network_event_receiver.default_routes)
self.signal.emit_signal('next-screen') self.app.next_screen()
def cancel(self): def cancel(self):
self.signal.emit_signal('prev-screen') self.app.prev_screen()
def _action_get(self, id): def _action_get(self, id):
dev_spec = id[0].split() dev_spec = id[0].split()

View File

@ -330,7 +330,7 @@ class Application:
# "Identity", # "Identity",
# "InstallProgress", # "InstallProgress",
# ] # ]
# The 'next-screen' and 'prev-screen' signals move through the list of # The 'next_screen' and 'prev-screen' methods move through the list of
# controllers in order, calling the start_ui method on the controller # controllers in order, calling the start_ui method on the controller
# instance. # instance.
@ -415,8 +415,6 @@ class Application:
"""Connect signals used in the core controller.""" """Connect signals used in the core controller."""
signals = [ signals = [
('quit', self.exit), ('quit', self.exit),
('next-screen', self.next_screen),
('prev-screen', self.prev_screen),
] ]
if self.opts.dry_run: if self.opts.dry_run:
signals.append(('control-x-quit', self.exit)) signals.append(('control-x-quit', self.exit))