appview/{pages,state}: simplify LoginPage params

We don't need to pass full `MultiAccountUser` here. just `[]AccountInfo`
should be enough. This way, we can make `MultiAccountUser` to always
hold an active user.

Signed-off-by: Seongmin Lee <[e-mail]>

appview/oauth/accounts.go

···
 }
 
 type MultiAccountUser struct {
-	Active   *User
+	Active   User
 	Accounts []AccountInfo
 }
 
 func (m *MultiAccountUser) Did() string {
-	if m.Active == nil {
-		return ""
-	}
 	return m.Active.Did
 }
 
···
 
 	registry := o.GetAccounts(r)
 	return &MultiAccountUser{
-		Active: &User{
+		Active: User{
 			Did: sess.Data.AccountDID.String(),
 		},
 		Accounts: registry.Accounts,

appview/oauth/accounts_test.go

···
 		}
 	})
 }
-
-func TestMultiAccountUser_Did(t *testing.T) {
-	t.Run("with active user", func(t *testing.T) {
-		user := &MultiAccountUser{
-			Active: &User{Did: "did:plc:test"},
-		}
-		if user.Did() != "did:plc:test" {
-			t.Errorf("Did() = %s, want did:plc:test", user.Did())
-		}
-	})
-
-	t.Run("with nil active", func(t *testing.T) {
-		user := &MultiAccountUser{Active: nil}
-		if user.Did() != "" {
-			t.Errorf("Did() = %s, want empty string", user.Did())
-		}
-	})
-}

appview/pages/pages.go

···
 }
 
 type LoginParams struct {
-	ReturnUrl    string
-	ErrorCode    string
-	AddAccount   bool
-	LoggedInUser *oauth.MultiAccountUser
+	ReturnUrl  string
+	ErrorCode  string
+	AddAccount bool
+	Accounts   []oauth.AccountInfo
 }
 
 func (p *Pages) Login(w io.Writer, params LoginParams) error {

appview/pages/templates/user/login.html

···
   </div>
   {{ end }}
 
-  {{ if and .LoggedInUser .LoggedInUser.Accounts }}
-  {{ $accounts := .LoggedInUser.Accounts }}
+  {{ if .Accounts }}
   <div class="my-4 border border-gray-200 dark:border-gray-700 rounded overflow-hidden">
       <div class="px-3 py-2 bg-gray-50 dark:bg-gray-800 border-b border-gray-200 dark:border-gray-700">
           <span class="text-xs text-gray-500 dark:text-gray-400 uppercase tracking-wide font-medium">Saved accounts</span>
       </div>
       <div class="divide-y divide-gray-200 dark:divide-gray-700">
-          {{ range $accounts }}
+          {{ range .Accounts }}
           <div class="flex items-center justify-between px-3 py-2 hover:bg-gray-100 dark:hover:bg-gray-700">
               <button
                   type="button"

appview/pulls/pulls.go

···
 	mergeCheckResponse := s.mergeCheck(r, f, pull, stack)
 	branchDeleteStatus := s.branchDeleteStatus(r, f, pull)
 	resubmitResult := pages.Unknown
-	if user != nil && user.Active != nil && user.Active.Did == pull.OwnerDid {
+	if user != nil && user.Active.Did == pull.OwnerDid {
 		resubmitResult = s.resubmitCheck(r, f, pull, stack)
 	}
 

appview/reporesolver/resolver.go

···
 	repoAt := repo.RepoAt()
 	isStarred := false
 	roles := repoinfo.RolesInRepo{}
-	if user != nil && user.Active != nil {
+	if user != nil {
 		isStarred = db.GetStarStatus(rr.execer, user.Active.Did, repoAt)
 		roles.Roles = rr.enforcer.GetPermissionsInRepo(user.Active.Did, repo.Knot, repo.DidSlashRepo())
 	}

appview/state/login.go

···
 	"net/http"
 	"strings"
 
-	"tangled.org/core/appview/oauth"
 	"tangled.org/core/appview/pages"
 )
 
···
 		errorCode := r.URL.Query().Get("error")
 		addAccount := r.URL.Query().Get("mode") == "add_account"
 
-		user := s.oauth.GetMultiAccountUser(r)
-		if user == nil {
-			registry := s.oauth.GetAccounts(r)
-			if len(registry.Accounts) > 0 {
-				user = &oauth.MultiAccountUser{
-					Active:   nil,
-					Accounts: registry.Accounts,
-				}
-			}
-		}
+		registry := s.oauth.GetAccounts(r)
 		s.pages.Login(w, pages.LoginParams{
-			ReturnUrl:    returnURL,
-			ErrorCode:    errorCode,
-			AddAccount:   addAccount,
-			LoggedInUser: user,
+			ReturnUrl:  returnURL,
+			ErrorCode:  errorCode,
+			AddAccount: addAccount,
+			Accounts:   registry.Accounts,
 		})
 	case http.MethodPost:
 		handle := r.FormValue("handle")
···
 	l := s.logger.With("handler", "Logout")
 
 	currentUser := s.oauth.GetMultiAccountUser(r)
-	if currentUser == nil || currentUser.Active == nil {
+	if currentUser == nil {
 		s.pages.HxRedirect(w, "/login")
 		return
 	}

appview/state/timeline.go

···
 	filtered := false
 
 	var userDid string
-	if user != nil && user.Active != nil {
+	if user != nil {
 		userDid = user.Active.Did
 	}
 	timeline, err := db.MakeTimeline(s.db, 50, userDid, filtered)