From bfd5c3a44624604f8d9a58dc011d6b6337f1f998 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=B8=89=E5=92=B2=E6=99=BA=E5=AD=90?= <sxzz@sxzz.moe>
Date: Tue, 3 Jan 2023 10:14:54 +0800
Subject: [PATCH] feat: wip

---
 components/account/AccountFollowButton.vue |  4 ++--
 components/account/AccountHeader.vue       |  2 +-
 components/account/AccountMoreButton.vue   |  2 +-
 components/status/StatusActionsMore.vue    |  4 ++--
 components/status/StatusReplyingTo.vue     |  6 +++++-
 components/user/UserDropdown.vue           |  8 ++------
 components/user/UserPicker.vue             | 14 ++-----------
 components/user/UserSwitcher.vue           | 23 ++++++----------------
 composables/masto.ts                       |  2 +-
 composables/search.ts                      |  2 +-
 composables/users.ts                       | 10 ++++++++++
 layouts/default.vue                        |  5 +++++
 pages/[[server]]/@[account]/[status].vue   |  2 +-
 13 files changed, 39 insertions(+), 45 deletions(-)

diff --git a/components/account/AccountFollowButton.vue b/components/account/AccountFollowButton.vue
index 3b4f5239..6a0c7aa8 100644
--- a/components/account/AccountFollowButton.vue
+++ b/components/account/AccountFollowButton.vue
@@ -7,8 +7,8 @@ const { account, command, ...props } = defineProps<{
   command?: boolean
 }>()
 
-const isSelf = $computed(() => currentUser.value?.account.id === account.id)
-const enable = $computed(() => !isSelf && currentUser.value)
+const isSelf = $computed(() => checkUser(currentUser.value) && currentUser.value.account.id === account.id)
+const enable = $computed(() => !isSelf && !isGuest.value)
 const relationship = $computed(() => props.relationship || useRelationship(account).value)
 
 const masto = useMasto()
diff --git a/components/account/AccountHeader.vue b/components/account/AccountHeader.vue
index 25ab8922..6ec29559 100644
--- a/components/account/AccountHeader.vue
+++ b/components/account/AccountHeader.vue
@@ -60,7 +60,7 @@ watchEffect(() => {
   iconFields.value = icons
 })
 
-const isSelf = $computed(() => currentUser.value?.account.id === account.id)
+const isSelf = $computed(() => checkUser(currentUser.value) && currentUser.value?.account.id === account.id)
 </script>
 
 <template>
diff --git a/components/account/AccountMoreButton.vue b/components/account/AccountMoreButton.vue
index 031270ea..bd85aa16 100644
--- a/components/account/AccountMoreButton.vue
+++ b/components/account/AccountMoreButton.vue
@@ -53,7 +53,7 @@ const toggleBlockDomain = async () => {
         />
       </NuxtLink>
 
-      <template v-if="currentUser">
+      <template v-if="!isGuest">
         <template v-if="!isSelf">
           <CommonDropdownItem
             :text="$t('menu.mention_account', [`@${account.acct}`])"
diff --git a/components/status/StatusActionsMore.vue b/components/status/StatusActionsMore.vue
index 0fcbda85..f25adbf5 100644
--- a/components/status/StatusActionsMore.vue
+++ b/components/status/StatusActionsMore.vue
@@ -23,7 +23,7 @@ const clipboard = useClipboard()
 const router = useRouter()
 const route = useRoute()
 
-const isAuthor = $computed(() => status.account.id === currentUser.value?.account.id)
+const isAuthor = $computed(() => checkUser(currentUser.value) && status.account.id === currentUser.value?.account.id)
 
 const {
   toggle: _toggleTranslation,
@@ -178,7 +178,7 @@ async function editStatus() {
         />
 
         <CommonDropdownItem
-          v-if="currentUser && (status.account.id === currentUser.account.id || status.mentions.some(m => m.id === currentUser!.account.id))"
+          v-if="!isGuest && (status.account.id === currentUser!.account!.id || status.mentions.some(m => m.id === currentUser!.account!.id))"
           :text="status.muted ? $t('menu.unmute_conversation') : $t('menu.mute_conversation')"
           :icon="status.muted ? 'i-ri:eye-line' : 'i-ri:eye-off-line'"
           :command="command"
diff --git a/components/status/StatusReplyingTo.vue b/components/status/StatusReplyingTo.vue
index 6bbcc7b2..b39f1564 100644
--- a/components/status/StatusReplyingTo.vue
+++ b/components/status/StatusReplyingTo.vue
@@ -22,7 +22,11 @@ const account = isSelf ? computed(() => status.account) : useAccountById(status.
       <template v-if="account">
         <div i-ri:reply-fill :class="collapsed ? '' : 'scale-x-[-1]'" text-secondary-light />
         <template v-if="!collapsed">
-          <AccountAvatar v-if="isSelf || simplified || status.inReplyToAccountId === currentUser?.account.id" :account="account" :link="false" w-5 h-5 mx-0.5 />
+          <AccountAvatar
+            v-if="isSelf || simplified
+              || (checkUser(currentUser) && status.inReplyToAccountId === currentUser?.account.id)"
+            :account="account" :link="false" w-5 h-5 mx-0.5
+          />
           <AccountInlineInfo v-else :account="account" :link="false" mx-0.5 />
         </template>
       </template>
diff --git a/components/user/UserDropdown.vue b/components/user/UserDropdown.vue
index 654513ee..4557aee3 100644
--- a/components/user/UserDropdown.vue
+++ b/components/user/UserDropdown.vue
@@ -1,12 +1,8 @@
-<script setup lang="ts">
-// workaround: cannot use TSNonNull `!` in template
-const account = $computed(() => currentUser.value!.account!)
-</script>
-
 <template>
   <VDropdown :distance="0" placement="top-start">
     <button btn-action-icon :aria-label="$t('action.switch_account')">
-      <div :class="{ 'hidden xl:block': currentUser }" i-ri:more-2-line />
+      <!-- TODO -->
+      <div :class="{ 'hidden xl:block': !isGuest }" i-ri:more-2-line />
       <AccountAvatar v-if="!isGuest" xl:hidden :account="account" w-9 h-9 />
       <span v-else>TODO: Guest</span>
     </button>
diff --git a/components/user/UserPicker.vue b/components/user/UserPicker.vue
index 6efd1e5c..9d968e9b 100644
--- a/components/user/UserPicker.vue
+++ b/components/user/UserPicker.vue
@@ -1,16 +1,6 @@
 <script setup lang="ts">
-import type { UserLogin } from '~/types'
-
 const all = useUsers()
-
-const router = useRouter()
 const masto = useMasto()
-const switchUser = (user: UserLogin) => {
-  if (!user.guest && !isGuest.value && user.account.id === currentUser.value!.account!.id)
-    router.push(getAccountRoute(user.account))
-  else
-    masto.loginTo(user)
-}
 </script>
 
 <template>
@@ -23,10 +13,10 @@ const switchUser = (user: UserLogin) => {
           aria-label="Switch user"
           :class="user.account?.id === currentUser?.account?.id ? '' : 'op25 grayscale'"
           hover="filter-none op100"
-          @click="switchUser(user)"
+          @click="switchUser(user, masto)"
         >
           <AccountAvatar v-if="!user.guest" w-13 h-13 :account="user.account" />
-          <span v-else>TODO: Guest</span>
+          <span v-else>TODO: Guest from {{ user.server }}</span>
         </button>
       </template>
     </div>
diff --git a/components/user/UserSwitcher.vue b/components/user/UserSwitcher.vue
index a4ece403..fccd5242 100644
--- a/components/user/UserSwitcher.vue
+++ b/components/user/UserSwitcher.vue
@@ -1,6 +1,4 @@
 <script setup lang="ts">
-import type { UserLogin } from '~/types'
-
 const emit = defineEmits<{
   (event: 'click'): void
 }>()
@@ -8,20 +6,10 @@ const emit = defineEmits<{
 const all = useUsers()
 
 const sorted = computed(() => {
-  return [
-    currentUser.value!,
-    ...all.value.filter(account => account.token !== currentUser.value?.token),
-  ].filter(Boolean)
+  return all.value.sort((a, b) => isSameUser(a, currentUser.value) ? -1 : isSameUser(b, currentUser.value) ? 1 : 0)
 })
 
-const router = useRouter()
 const masto = useMasto()
-const switchUser = (user: UserLogin) => {
-  if (user.account.id === currentUser.value?.account.id)
-    router.push(getAccountRoute(user.account))
-  else
-    masto.loginTo(user)
-}
 </script>
 
 <template>
@@ -31,11 +19,12 @@ const switchUser = (user: UserLogin) => {
         flex rounded px4 py3 text-left
         hover:bg-active cursor-pointer transition-100
         aria-label="Switch user"
-        @click="switchUser(user)"
+        @click="switchUser(user, masto)"
       >
-        <AccountInfo :account="user.account" :hover-card="false" />
+        <AccountInfo v-if="!user.guest" :account="user.account" :hover-card="false" />
+        <span v-else>TODO: Guest from {{ user.server }}</span>
         <div flex-auto />
-        <div v-if="user.token === currentUser?.token" i-ri:check-line text-primary mya text-2xl />
+        <div v-if="isSameUser(user, currentUser)" i-ri:check-line text-primary mya text-2xl />
       </button>
     </template>
     <div border="t base" pt2>
@@ -46,7 +35,7 @@ const switchUser = (user: UserLogin) => {
       />
       <CommonDropdownItem
         v-if="isMastoInitialised && currentUser"
-        :text="$t('user.sign_out_account', [getFullHandle(currentUser.account)])"
+        :text="$t('user.sign_out_account', [getFullHandle(currentUser)])"
         icon="i-ri:logout-box-line rtl-flip"
         @click="signout"
       />
diff --git a/composables/masto.ts b/composables/masto.ts
index 42031e53..21bd903a 100644
--- a/composables/masto.ts
+++ b/composables/masto.ts
@@ -141,7 +141,7 @@ const requestedRelationships = new Map<string, Ref<Relationship | undefined>>()
 let timeoutHandle: NodeJS.Timeout | undefined
 
 export function useRelationship(account: Account): Ref<Relationship | undefined> {
-  if (!currentUser.value)
+  if (isGuest.value)
     return ref()
   let relationship = requestedRelationships.get(account.id)
   if (relationship)
diff --git a/composables/search.ts b/composables/search.ts
index 8eca9016..ad047798 100644
--- a/composables/search.ts
+++ b/composables/search.ts
@@ -25,7 +25,7 @@ export function useSearch(query: MaybeRef<string>, options?: UseSearchOptions) {
      * Based on the source it seems like modifying the params when calling next would result in a new search,
      * but that doesn't seem to be the case. So instead we just create a new paginator with the new params.
      */
-    paginator = masto.search({ q: unref(query), resolve: !!currentUser.value, type: unref(options?.type) })
+    paginator = masto.search({ q: unref(query), resolve: !isGuest.value, type: unref(options?.type) })
     const nextResults = await paginator.next()
 
     done.value = nextResults.done || false
diff --git a/composables/users.ts b/composables/users.ts
index 8ef6dc52..2b077a76 100644
--- a/composables/users.ts
+++ b/composables/users.ts
@@ -65,6 +65,8 @@ export const currentInstance = computed<null | Instance>(() => {
 })
 export const checkUser = (val: UserLogin | undefined): val is UserLogin<true> => !!(val && !val.guest)
 export const isGuest = computed(() => !checkUser(currentUser.value))
+export const isSameUser = (a: UserLogin | undefined, b: UserLogin | undefined) =>
+  a && b && a.server === b.server && a.token === b.token
 
 export const currentUserHandle = computed(() =>
   isGuestId.value ? GUEST_ID : currentUser.value!.account!.acct
@@ -140,6 +142,14 @@ async function loginTo(user?: UserLogin) {
   return masto
 }
 
+export const switchUser = (user: UserLogin, masto: ElkMasto) => {
+  const router = useRouter()
+  if (!user.guest && !isGuest.value && user.account.id === currentUser.value!.account!.id)
+    router.push(getAccountRoute(user.account))
+  else
+    masto.loginTo(user)
+}
+
 export function setAccountInfo(userId: string, account: AccountCredentials) {
   const index = getUsersIndexByUserId(userId)
   if (index === -1)
diff --git a/layouts/default.vue b/layouts/default.vue
index 132a6904..89d62f58 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -18,11 +18,13 @@ const wideLayout = computed(() => route.meta.wideLayout ?? false)
               </div>
               <div v-if="isMastoInitialised" flex flex-col>
                 <UserSignInEntry v-if="!currentUser" sm:hidden />
+                <!-- TODO -->
                 <div v-if="currentUser" p6 pb8 w-full>
                   <div hidden xl-block>
                     <UserPicker v-if="showUserPicker" />
                     <div v-else flex="~" items-center justify-between>
                       <NuxtLink
+                        v-if="checkUser(currentUser)"
                         hidden xl:block
                         rounded-full text-start w-full
                         hover:bg-active cursor-pointer transition-100
@@ -30,6 +32,9 @@ const wideLayout = computed(() => route.meta.wideLayout ?? false)
                       >
                         <AccountInfo :account="currentUser.account" md:break-words />
                       </NuxtLink>
+                      <div v-else>
+                        TODO: guest {{ currentUser.server }} @ default.vue
+                      </div>
                       <UserDropdown />
                     </div>
                   </div>
diff --git a/pages/[[server]]/@[account]/[status].vue b/pages/[[server]]/@[account]/[status].vue
index cfd1be88..0d31a43b 100644
--- a/pages/[[server]]/@[account]/[status].vue
+++ b/pages/[[server]]/@[account]/[status].vue
@@ -73,7 +73,7 @@ onReactivated(() => {
           :actions="status.visibility !== 'direct'"
         />
         <PublishWidget
-          v-if="currentUser"
+          v-if="!isGuest"
           ref="publishWidget"
           border="y base"
           :draft-key="replyDraft!.key"