From 60a867375772642935320fb86d7d2adb95f19bb9 Mon Sep 17 00:00:00 2001
From: Daniel Roe <daniel@roe.dev>
Date: Tue, 20 Dec 2022 15:56:54 +0000
Subject: [PATCH] fix: use `masto.loginTo` to update masto api (#474)

---
 app.vue                          | 2 +-
 components/user/UserPicker.vue   | 3 ++-
 components/user/UserSwitcher.vue | 3 ++-
 composables/command.ts           | 3 ++-
 composables/users.ts             | 4 +++-
 error.vue                        | 5 +++--
 middleware/permalink.global.ts   | 7 ++++---
 pages/[...permalink].vue         | 6 ++----
 8 files changed, 19 insertions(+), 14 deletions(-)

diff --git a/app.vue b/app.vue
index b7379cb6..2599eb4f 100644
--- a/app.vue
+++ b/app.vue
@@ -5,7 +5,7 @@ setupPageHeader()
 provideGlobalCommands()
 
 // We want to trigger rerendering the page when account changes
-const key = computed(() => `${currentServer.value}:${currentUser.value?.account.id || ''}`)
+const key = computed(() => `${currentUser.value?.server ?? currentServer.value}:${currentUser.value?.account.id || ''}`)
 </script>
 
 <template>
diff --git a/components/user/UserPicker.vue b/components/user/UserPicker.vue
index 0d9c17f6..9585fd7f 100644
--- a/components/user/UserPicker.vue
+++ b/components/user/UserPicker.vue
@@ -4,11 +4,12 @@ import type { UserLogin } from '~/types'
 const all = useUsers()
 
 const router = useRouter()
+const masto = useMasto()
 const switchUser = (user: UserLogin) => {
   if (user.account.id === currentUser.value?.account.id)
     router.push(getAccountRoute(user.account))
   else
-    loginTo(user)
+    masto.loginTo(user)
 }
 </script>
 
diff --git a/components/user/UserSwitcher.vue b/components/user/UserSwitcher.vue
index 158cc50a..899eb995 100644
--- a/components/user/UserSwitcher.vue
+++ b/components/user/UserSwitcher.vue
@@ -15,11 +15,12 @@ const sorted = computed(() => {
 })
 
 const router = useRouter()
+const masto = useMasto()
 const switchUser = (user: UserLogin) => {
   if (user.account.id === currentUser.value?.account.id)
     router.push(getAccountRoute(user.account))
   else
-    loginTo(user)
+    masto.loginTo(user)
 }
 </script>
 
diff --git a/composables/command.ts b/composables/command.ts
index 34644ed6..d34ea2f7 100644
--- a/composables/command.ts
+++ b/composables/command.ts
@@ -204,6 +204,7 @@ export const provideGlobalCommands = () => {
   const { locale, t } = useI18n()
   const { locales } = useI18n() as { locales: ComputedRef<LocaleObject[]> }
   const users = useUsers()
+  const masto = useMasto()
 
   useCommand({
     scope: 'Actions',
@@ -299,7 +300,7 @@ export const provideGlobalCommands = () => {
     icon: 'i-ri:user-shared-line',
 
     onActivate() {
-      loginTo(user)
+      masto.loginTo(user)
     },
   })))
   useCommand({
diff --git a/composables/users.ts b/composables/users.ts
index 89c015ea..ef090594 100644
--- a/composables/users.ts
+++ b/composables/users.ts
@@ -133,6 +133,8 @@ export async function signout() {
   if (!currentUser.value)
     return
 
+  const masto = useMasto()
+
   const _currentUserId = currentUser.value.account.id
 
   const index = users.value.findIndex(u => u.account?.id === _currentUserId)
@@ -156,7 +158,7 @@ export async function signout() {
   if (!currentUserId.value)
     await useRouter().push('/')
 
-  await loginTo(currentUser.value)
+  await masto.loginTo(currentUser.value)
 }
 
 const notifications = reactive<Record<string, undefined | [Promise<WsEvents>, number]>>({})
diff --git a/error.vue b/error.vue
index c4b9ee86..1c081a7e 100644
--- a/error.vue
+++ b/error.vue
@@ -18,11 +18,12 @@ const defaultMessage = 'Something went wrong'
 const message = error.message ?? errorCodes[error.statusCode!] ?? defaultMessage
 
 const state = ref<'error' | 'reloading'>('error')
+const masto = useMasto()
 const reload = async () => {
   state.value = 'reloading'
   try {
-    if (!useMasto())
-      await loginTo(currentUser.value)
+    if (!masto.loggedIn.value)
+      await masto.loginTo(currentUser.value)
     clearError({ redirect: currentUser.value ? '/home' : `/${currentServer.value}/public` })
   }
   catch {
diff --git a/middleware/permalink.global.ts b/middleware/permalink.global.ts
index 89e5426d..f636156a 100644
--- a/middleware/permalink.global.ts
+++ b/middleware/permalink.global.ts
@@ -2,8 +2,10 @@ export default defineNuxtRouteMiddleware(async (to, from) => {
   if (process.server)
     return
 
+  const masto = useMasto()
+
   // Skip running middleware before masto has been initialised
-  if (!useNuxtApp().$masto)
+  if (!masto)
     return
 
   if (!('server' in to.params))
@@ -13,7 +15,7 @@ export default defineNuxtRouteMiddleware(async (to, from) => {
 
   if (!user) {
     if (from.params.server !== to.params.server) {
-      await loginTo({
+      await masto.loginTo({
         server: to.params.server as string,
       })
     }
@@ -47,7 +49,6 @@ export default defineNuxtRouteMiddleware(async (to, from) => {
         return getAccountRoute(account)
     }
 
-    const masto = useMasto()
     if (!masto.loggedIn.value)
       await masto.loginTo(currentUser.value)
 
diff --git a/pages/[...permalink].vue b/pages/[...permalink].vue
index 303c1c28..6812826c 100644
--- a/pages/[...permalink].vue
+++ b/pages/[...permalink].vue
@@ -10,10 +10,8 @@ definePageMeta({
     if (hasProtocol(permalink)) {
       const { host, pathname } = parseURL(permalink)
 
-      if (host) {
-        await loginTo({ server: host })
-        return pathname
-      }
+      if (host)
+        return `/${host}${pathname}`
     }
 
     // We've reached a page that doesn't exist