Delegate model update on event callback

Keep a TODO to maybe remove it by using the list selection item index.

pam/authmodeselection.go

@@ -34,8 +34,7 @@ type authModeSelected struct {
 }
 
 // selectAuthMode selects current authentication mode.
-func (m *authModeSelectionModel) selectAuthMode(id string) tea.Cmd {
-	m.currentAuthModeSelectedID = id
+func selectAuthMode(id string) tea.Cmd {
 	return func() tea.Msg {
 		return authModeSelected{
 			id: id,
@@ -87,6 +86,8 @@ func (m authModeSelectionModel) Update(msg tea.Msg) (authModeSelectionModel, tea
 			log.Infof(context.TODO(), "authentication mode %q is not part of currently available authentication mode", msg.id)
 			return m, nil
 		}
+		m.currentAuthModeSelectedID = msg.id // TODO: remove by getting current item on the list?
+
 		// Select correct line to ensure model is synchronised
 		for i, a := range m.Items() {
 			if a.(authModeItem).id != msg.id {
@@ -114,7 +115,7 @@ func (m authModeSelectionModel) Update(msg tea.Msg) (authModeSelectionModel, tea
 				return m, nil
 			}
 			authMode := item.(authModeItem)
-			cmd := m.selectAuthMode(authMode.id)
+			cmd := selectAuthMode(authMode.id)
 			return m, cmd
 		case "1", "2", "3", "4", "5", "6", "7", "8", "9":
 			// This is necessarily an integer, so above
@@ -125,7 +126,7 @@ func (m authModeSelectionModel) Update(msg tea.Msg) (authModeSelectionModel, tea
 			}
 			item := items[choice-1]
 			authMode := item.(authModeItem)
-			cmd := m.selectAuthMode(authMode.id)
+			cmd := selectAuthMode(authMode.id)
 			return m, cmd
 		}
 	}