From 30f6a4c9991b42ebade54221dbf7f9e0f20e94f4 Mon Sep 17 00:00:00 2001 From: jwijenbergh Date: Mon, 20 Mar 2023 16:31:29 +0100 Subject: Client: Remove servers argument to transition --- client/client.go | 4 ++-- client/fsm.go | 2 +- client/server.go | 12 ++++++------ 3 files changed, 9 insertions(+), 9 deletions(-) (limited to 'client') diff --git a/client/client.go b/client/client.go index 295fa04..31110bd 100644 --- a/client/client.go +++ b/client/client.go @@ -185,8 +185,8 @@ func (c *Client) Register( log.Logger.Infof("Previous configuration not found") } - // Go to the No Server state with the saved servers after we're done - defer c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + // Go to the No Server state after we're done + defer c.FSM.GoTransition(StateNoServer) // Let's Connect! doesn't care about discovery if c.isLetsConnect() { diff --git a/client/fsm.go b/client/fsm.go index d1d2d9e..9f140e3 100644 --- a/client/fsm.go +++ b/client/fsm.go @@ -180,7 +180,7 @@ func (c *Client) GoBack() error { } // FIXME: Arbitrary back transitions don't work because we need the appropriate data - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) return nil } diff --git a/client/server.go b/client/server.go index 684d780..00e55e8 100644 --- a/client/server.go +++ b/client/server.go @@ -193,7 +193,7 @@ func (c *Client) RemoveSecureInternet() error { } // No error because we can only have one secure internet server and if there are no secure internet servers, this is a NO-OP c.Servers.RemoveSecureInternet() - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) // Save the config if err := c.Config.Save(&c); err != nil { // TODO(jwijenbergh): Not sure why INFO level, yet stacktrace... @@ -215,7 +215,7 @@ func (c *Client) RemoveInstituteAccess(url string) error { } // No error because this is a NO-OP if the server doesn't exist c.Servers.RemoveInstituteAccess(url) - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) // Save the config if err := c.Config.Save(&c); err != nil { // TODO(jwijenbergh): Not sure why INFO level, yet stacktrace... @@ -237,7 +237,7 @@ func (c *Client) RemoveCustomServer(url string) error { } // No error because this is a NO-OP if the server doesn't exist c.Servers.RemoveCustomServer(url) - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) // Save the config if err := c.Config.Save(&c); err != nil { // TODO(jwijenbergh): Not sure why INFO level, yet stacktrace... @@ -296,7 +296,7 @@ func (c *Client) AddInstituteServer(url string) (err error) { return err } - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) return nil } @@ -360,7 +360,7 @@ func (c *Client) AddSecureInternetHomeServer(orgID string) (err error) { _ = c.RemoveSecureInternet() return err } - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) return nil } @@ -408,7 +408,7 @@ func (c *Client) AddCustomServer(url string) (err error) { return err } - c.FSM.GoTransitionWithData(StateNoServer, c.Servers) + c.FSM.GoTransition(StateNoServer) return nil } -- cgit v1.2.3