Disconnect broken

[andig]

Refs evcc-io/evcc#17446 (comment)

In

// Disconnect the client
func (c *Client) Disconnect() error {
	c.isAuthenticated = false
	c.isConnected = false

        // -> THIS BLOCK CAN NEVER EXECUTE
	if c.isConnected && c.conn != nil {
		if err := c.conn.Close(); err != nil {
			return err
		}
	}

	Log.Info("disconnected")
	return nil
}

[andig]

Instead of isConnected, wouldn't it be simpler to use c.conn == nil for this purpose? Disconnect would then become:

// Disconnect the client
func (c *Client) Disconnect() (err error) {
	c.isAuthenticated = false

	if c.conn != nil {
		err = c.conn.Close()
		c.conn = nil
		Log.Info("disconnected")
	}

	return err
}