From 543ac86d03b5f1c9775bf4d54a3606e262f66bf4 Mon Sep 17 00:00:00 2001
From: Zhiyuan Zheng <me@xmflsct.com>
Date: Sun, 21 Mar 2021 23:06:53 +0100
Subject: [PATCH] Fixed #43 #73 #58

---
 src/@types/react-navigation.d.ts              |   1 -
 src/api/instance.ts                           |  11 +-
 src/components/GracefullyImage.tsx            |   6 +-
 src/components/Parse/HTML.tsx                 |   2 +-
 src/components/Timeline/Shared/Attachment.tsx |   1 -
 src/components/Timeline/Shared/Card.tsx       |   4 +-
 src/components/openLink.ts                    | 120 +++++++++++++++++-
 src/screens/ImageViewer/@types/index.ts       |   9 --
 src/screens/ImageViewer/Root.tsx              |  13 +-
 .../components/ImageItem.android.tsx          |   8 +-
 .../ImageViewer/components/ImageItem.ios.tsx  |   8 +-
 .../ImageViewer/hooks/usePanResponder.ts      |   2 +-
 src/screens/ImagesViewer.tsx                  |  26 ++--
 src/utils/queryHooks/search.ts                |   9 +-
 14 files changed, 174 insertions(+), 46 deletions(-)

diff --git a/src/@types/react-navigation.d.ts b/src/@types/react-navigation.d.ts
index 58d7b2f1..33834f60 100644
--- a/src/@types/react-navigation.d.ts
+++ b/src/@types/react-navigation.d.ts
@@ -60,7 +60,6 @@ declare namespace Nav {
         url: Mastodon.AttachmentImage['url']
         width?: number
         height?: number
-        preview_url: Mastodon.AttachmentImage['preview_url']
         remote_url?: Mastodon.AttachmentImage['remote_url']
       }[]
       id: Mastodon.Attachment['id']
diff --git a/src/api/instance.ts b/src/api/instance.ts
index 5bfd48c8..d64457c5 100644
--- a/src/api/instance.ts
+++ b/src/api/instance.ts
@@ -1,5 +1,5 @@
 import { RootState } from '@root/store'
-import axios from 'axios'
+import axios, { AxiosRequestConfig } from 'axios'
 import chalk from 'chalk'
 import li from 'li'
 
@@ -14,7 +14,10 @@ export type Params = {
   }
   headers?: { [key: string]: string }
   body?: FormData
-  onUploadProgress?: (progressEvent: any) => void
+  extras?: Omit<
+    AxiosRequestConfig,
+    'method' | 'url' | 'params' | 'headers' | 'data'
+  >
 }
 
 const apiInstance = async <T = unknown>({
@@ -24,7 +27,7 @@ const apiInstance = async <T = unknown>({
   params,
   headers,
   body,
-  onUploadProgress
+  extras
 }: Params): Promise<{ body: T; links: { prev?: string; next?: string } }> => {
   const { store } = require('@root/store')
   const state = store.getState() as RootState
@@ -70,7 +73,7 @@ const apiInstance = async <T = unknown>({
       })
     },
     ...(body && { data: body }),
-    ...(onUploadProgress && { onUploadProgress: onUploadProgress })
+    ...extras
   })
     .then(response => {
       let prev
diff --git a/src/components/GracefullyImage.tsx b/src/components/GracefullyImage.tsx
index f7da33b2..04ad5270 100644
--- a/src/components/GracefullyImage.tsx
+++ b/src/components/GracefullyImage.tsx
@@ -102,7 +102,11 @@ const GracefullyImage = React.memo(
 
     return (
       <Pressable
-        style={[style, dimension, { backgroundColor: theme.shimmerDefault }]}
+        style={[
+          style,
+          dimension,
+          { backgroundColor: theme.backgroundOverlayDefault }
+        ]}
         {...(onPress
           ? hidden
             ? { disabled: true }
diff --git a/src/components/Parse/HTML.tsx b/src/components/Parse/HTML.tsx
index 3a1f6449..e19e874f 100644
--- a/src/components/Parse/HTML.tsx
+++ b/src/components/Parse/HTML.tsx
@@ -121,7 +121,7 @@ const renderNode = ({
             onPress={async () => {
               analytics('status_link_press')
               !disableDetails && !shouldBeTag
-                ? await openLink(href)
+                ? await openLink(href, navigation)
                 : navigation.push('Tab-Shared-Hashtag', {
                     hashtag: content.substring(1)
                   })
diff --git a/src/components/Timeline/Shared/Attachment.tsx b/src/components/Timeline/Shared/Attachment.tsx
index 7afdeff8..3dd10327 100644
--- a/src/components/Timeline/Shared/Attachment.tsx
+++ b/src/components/Timeline/Shared/Attachment.tsx
@@ -48,7 +48,6 @@ const TimelineAttachment = React.memo(
               imageUrls.push({
                 id: attachment.id,
                 url: attachment.url,
-                preview_url: attachment.preview_url,
                 remote_url: attachment.remote_url,
                 width: attachment.meta?.original?.width,
                 height: attachment.meta?.original?.height
diff --git a/src/components/Timeline/Shared/Card.tsx b/src/components/Timeline/Shared/Card.tsx
index 35f35d27..7f5d2469 100644
--- a/src/components/Timeline/Shared/Card.tsx
+++ b/src/components/Timeline/Shared/Card.tsx
@@ -1,6 +1,7 @@
 import analytics from '@components/analytics'
 import GracefullyImage from '@components/GracefullyImage'
 import openLink from '@components/openLink'
+import { useNavigation } from '@react-navigation/native'
 import { StyleConstants } from '@utils/styles/constants'
 import { useTheme } from '@utils/styles/ThemeManager'
 import React from 'react'
@@ -13,13 +14,14 @@ export interface Props {
 const TimelineCard = React.memo(
   ({ card }: Props) => {
     const { theme } = useTheme()
+    const navigation = useNavigation()
 
     return (
       <Pressable
         style={[styles.card, { borderColor: theme.border }]}
         onPress={async () => {
           analytics('timeline_shared_card_press')
-          await openLink(card.url)
+          await openLink(card.url, navigation)
         }}
         testID='base'
       >
diff --git a/src/components/openLink.ts b/src/components/openLink.ts
index b8073cf1..6327128a 100644
--- a/src/components/openLink.ts
+++ b/src/components/openLink.ts
@@ -1,9 +1,127 @@
+import apiInstance from '@api/instance'
+import { NavigationProp, ParamListBase } from '@react-navigation/native'
+import { navigationRef } from '@root/Screens'
 import { store } from '@root/store'
+import { SearchResult } from '@utils/queryHooks/search'
+import { getInstanceUrl } from '@utils/slices/instancesSlice'
 import { getSettingsBrowser } from '@utils/slices/settingsSlice'
 import * as Linking from 'expo-linking'
 import * as WebBrowser from 'expo-web-browser'
 
-const openLink = async (url: string) => {
+// https://social.xmflsct.com/web/statuses/105590085754428765 <- default
+// https://social.xmflsct.com/@tooot/105590085754428765 <- pretty
+const matcherStatus = new RegExp(
+  /http[s]?:\/\/(.*)\/(web\/statuses|@.*)\/([0-9]*)/
+)
+
+// https://social.xmflsct.com/web/accounts/14195 <- default
+// https://social.xmflsct.com/@tooot <- pretty
+const matcherAccount = new RegExp(
+  /http[s]?:\/\/(.*)\/(web\/accounts\/([0-9]*)|@.*)/
+)
+
+export let loadingLink = false
+
+const openLink = async (
+  url: string,
+  navigation?: NavigationProp<
+    ParamListBase,
+    string,
+    Readonly<{
+      key: string
+      index: number
+      routeNames: string[]
+      history?: unknown[] | undefined
+      routes: any[]
+      type: string
+      stale: false
+    }>,
+    {},
+    {}
+  >
+) => {
+  if (loadingLink) {
+    return
+  }
+
+  const handleNavigation = (
+    page: 'Tab-Shared-Toot' | 'Tab-Shared-Account',
+    options: {}
+  ) => {
+    if (navigation) {
+      // @ts-ignore
+      navigation.push(page, options)
+    } else {
+      navigationRef.current?.navigate(page, options)
+    }
+  }
+
+  // If a tooot can be found
+  const matchedStatus = url.match(matcherStatus)
+  if (matchedStatus) {
+    // If the link in current instance
+    const instanceUrl = getInstanceUrl(store.getState())
+    if (matchedStatus[1] === instanceUrl) {
+      handleNavigation('Tab-Shared-Toot', {
+        toot: { id: matchedStatus[3] }
+      })
+      return
+    }
+
+    loadingLink = true
+    let response
+    try {
+      response = await apiInstance<SearchResult>({
+        version: 'v2',
+        method: 'get',
+        url: 'search',
+        params: { type: 'statuses', q: url, limit: 1, resolve: true }
+      })
+    } catch {}
+    if (response && response.body && response.body.statuses.length) {
+      handleNavigation('Tab-Shared-Toot', {
+        toot: response.body.statuses[0]
+      })
+      loadingLink = false
+      return
+    }
+  }
+
+  // If an account can be found
+  const matchedAccount = url.match(matcherAccount)
+  console.log(matchedAccount)
+  if (matchedAccount) {
+    // If the link in current instance
+    const instanceUrl = getInstanceUrl(store.getState())
+    if (matchedAccount[1] === instanceUrl) {
+      if (matchedAccount[3] && matchedAccount[3].match(/[0-9]*/)) {
+        handleNavigation('Tab-Shared-Account', {
+          account: { id: matchedAccount[3] }
+        })
+        return
+      }
+    }
+
+    loadingLink = true
+    let response
+    try {
+      response = await apiInstance<SearchResult>({
+        version: 'v2',
+        method: 'get',
+        url: 'search',
+        params: { type: 'accounts', q: url, limit: 1, resolve: true }
+      })
+    } catch {}
+    if (response && response.body && response.body.accounts.length) {
+      handleNavigation('Tab-Shared-Account', {
+        account: response.body.accounts[0]
+      })
+      loadingLink = false
+      return
+    }
+  }
+
+  loadingLink = false
   switch (getSettingsBrowser(store.getState())) {
     case 'internal':
       await WebBrowser.openBrowserAsync(url, {
diff --git a/src/screens/ImageViewer/@types/index.ts b/src/screens/ImageViewer/@types/index.ts
index 7150b5df..780313fd 100644
--- a/src/screens/ImageViewer/@types/index.ts
+++ b/src/screens/ImageViewer/@types/index.ts
@@ -15,12 +15,3 @@ export type Position = {
   x: number
   y: number
 }
-
-export type ImageSource = {
-  id: string
-  preview_url: string
-  remote_url?: string
-  url: string
-  width?: number
-  height?: number
-}
diff --git a/src/screens/ImageViewer/Root.tsx b/src/screens/ImageViewer/Root.tsx
index 6b99a934..0e78106f 100644
--- a/src/screens/ImageViewer/Root.tsx
+++ b/src/screens/ImageViewer/Root.tsx
@@ -14,17 +14,18 @@ import {
   View,
   VirtualizedList
 } from 'react-native'
-import { ImageSource } from './@types'
 import ImageItem from './components/ImageItem'
 import useAnimatedComponents from './hooks/useAnimatedComponents'
 import useImageIndexChange from './hooks/useImageIndexChange'
 import useRequestClose from './hooks/useRequestClose'
 
 type Props = {
-  images: ImageSource[]
+  images: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls']
   imageIndex: number
   onRequestClose: () => void
-  onLongPress?: (image: ImageSource) => void
+  onLongPress?: (
+    image: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
+  ) => void
   onImageIndexChange?: (imageIndex: number) => void
   backgroundColor?: string
   swipeToCloseEnabled?: boolean
@@ -48,7 +49,11 @@ function ImageViewer ({
   delayLongPress = DEFAULT_DELAY_LONG_PRESS,
   HeaderComponent
 }: Props) {
-  const imageList = React.createRef<VirtualizedList<ImageSource>>()
+  const imageList = React.createRef<
+    VirtualizedList<
+      Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
+    >
+  >()
   const [opacity, onRequestCloseEnhanced] = useRequestClose(onRequestClose)
   const [currentImageIndex, onScroll] = useImageIndexChange(imageIndex, SCREEN)
   const [headerTransform, toggleBarsVisible] = useAnimatedComponents()
diff --git a/src/screens/ImageViewer/components/ImageItem.android.tsx b/src/screens/ImageViewer/components/ImageItem.android.tsx
index d907ad33..20a611af 100644
--- a/src/screens/ImageViewer/components/ImageItem.android.tsx
+++ b/src/screens/ImageViewer/components/ImageItem.android.tsx
@@ -9,7 +9,6 @@
 import GracefullyImage from '@components/GracefullyImage'
 import React, { useState, useCallback } from 'react'
 import { Animated, Dimensions, StyleSheet } from 'react-native'
-import { ImageSource } from '../@types'
 import usePanResponder from '../hooks/usePanResponder'
 import { getImageStyles, getImageTransform } from '../utils'
 
@@ -18,10 +17,12 @@ const SCREEN_WIDTH = SCREEN.width
 const SCREEN_HEIGHT = SCREEN.height
 
 type Props = {
-  imageSrc: ImageSource
+  imageSrc: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
   onRequestClose: () => void
   onZoom: (isZoomed: boolean) => void
-  onLongPress: (image: ImageSource) => void
+  onLongPress: (
+    image: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
+  ) => void
   delayLongPress: number
   swipeToCloseEnabled?: boolean
   doubleTapToZoomEnabled?: boolean
@@ -89,7 +90,6 @@ const ImageItem = ({
         children={
           <GracefullyImage
             uri={{
-              preview: imageSrc.preview_url,
               original: imageSrc.url,
               remote: imageSrc.remote_url
             }}
diff --git a/src/screens/ImageViewer/components/ImageItem.ios.tsx b/src/screens/ImageViewer/components/ImageItem.ios.tsx
index bac05f0d..b21f7aa2 100644
--- a/src/screens/ImageViewer/components/ImageItem.ios.tsx
+++ b/src/screens/ImageViewer/components/ImageItem.ios.tsx
@@ -21,7 +21,6 @@ import {
   State,
   TapGestureHandler
 } from 'react-native-gesture-handler'
-import { ImageSource } from '../@types'
 import useDoubleTapToZoom from '../hooks/useDoubleTapToZoom'
 import { getImageStyles, getImageTransform } from '../utils'
 
@@ -32,10 +31,12 @@ const SCREEN_WIDTH = SCREEN.width
 const SCREEN_HEIGHT = SCREEN.height
 
 type Props = {
-  imageSrc: ImageSource
+  imageSrc: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
   onRequestClose: () => void
   onZoom: (scaled: boolean) => void
-  onLongPress: (image: ImageSource) => void
+  onLongPress: (
+    image: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
+  ) => void
   swipeToCloseEnabled?: boolean
 }
 
@@ -144,7 +145,6 @@ const ImageItem = ({
               children={
                 <GracefullyImage
                   uri={{
-                    preview: imageSrc.preview_url,
                     original: imageSrc.url,
                     remote: imageSrc.remote_url
                   }}
diff --git a/src/screens/ImageViewer/hooks/usePanResponder.ts b/src/screens/ImageViewer/hooks/usePanResponder.ts
index 2be5b685..d4ed1fe1 100644
--- a/src/screens/ImageViewer/hooks/usePanResponder.ts
+++ b/src/screens/ImageViewer/hooks/usePanResponder.ts
@@ -6,7 +6,7 @@
  *
  */
 
-import { useMemo, useEffect, useCallback } from 'react'
+import { useMemo, useEffect } from 'react'
 import {
   Animated,
   Dimensions,
diff --git a/src/screens/ImagesViewer.tsx b/src/screens/ImagesViewer.tsx
index a2ec53b1..e093ea29 100644
--- a/src/screens/ImagesViewer.tsx
+++ b/src/screens/ImagesViewer.tsx
@@ -20,15 +20,9 @@ import {
 } from 'react-native-safe-area-context'
 import ImageViewer from './ImageViewer/Root'
 
-type ImageUrl = {
-  url: string
-  width?: number | undefined
-  height?: number | undefined
-  preview_url: string
-  remote_url?: string | undefined
-}
-
-const saveImage = async (image: ImageUrl) => {
+const saveImage = async (
+  image: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls'][0]
+) => {
   const hasAndroidPermission = async () => {
     const permission = PermissionsAndroid.PERMISSIONS.WRITE_EXTERNAL_STORAGE
 
@@ -44,9 +38,17 @@ const saveImage = async (image: ImageUrl) => {
   if (Platform.OS === 'android' && !(await hasAndroidPermission())) {
     return
   }
-  CameraRoll.save(image.url || image.remote_url || image.preview_url)
+  CameraRoll.save(image.url)
     .then(() => haptics('Success'))
-    .catch(() => haptics('Error'))
+    .catch(() => {
+      if (image.remote_url) {
+        CameraRoll.save(image.remote_url)
+          .then(() => haptics('Success'))
+          .catch(() => haptics('Error'))
+      } else {
+        haptics('Error')
+      }
+    })
 }
 
 const HeaderComponent = React.memo(
@@ -57,7 +59,7 @@ const HeaderComponent = React.memo(
   }: {
     navigation: ScreenImagesViewerProp['navigation']
     currentIndex: number
-    imageUrls: ImageUrl[]
+    imageUrls: Nav.RootStackParamList['Screen-ImagesViewer']['imageUrls']
   }) => {
     const insets = useSafeAreaInsets()
     const { t } = useTranslation('screenImageViewer')
diff --git a/src/utils/queryHooks/search.ts b/src/utils/queryHooks/search.ts
index 6221a5cf..0ea5d5e7 100644
--- a/src/utils/queryHooks/search.ts
+++ b/src/utils/queryHooks/search.ts
@@ -11,7 +11,7 @@ export type QueryKey = [
   }
 ]
 
-type SearchResult = {
+export type SearchResult = {
   accounts: Mastodon.Account[]
   hashtags: Mastodon.Tag[]
   statuses: Mastodon.Status[]
@@ -23,7 +23,12 @@ const queryFunction = ({ queryKey }: { queryKey: QueryKey }) => {
     version: 'v2',
     method: 'get',
     url: 'search',
-    params: { ...(type && { type }), ...(term && { q: term }), limit }
+    params: {
+      ...(type && { type }),
+      ...(term && { q: term }),
+      limit,
+      resolve: true
+    }
   }).then(res => res.body)
 }