From 323408f8d981bd3c75727a16ed046f2260aa258e Mon Sep 17 00:00:00 2001
From: Pan <panfree23@gmail.com>
Date: Wed, 10 Oct 2018 17:43:33 +0800
Subject: [PATCH] fix[ExternalLink]: fixed bug when url include chinese #1182

---
 src/utils/index.js                                  | 4 ++++
 src/views/layout/components/Sidebar/Link.vue        | 4 ++--
 src/views/layout/components/Sidebar/SidebarItem.vue | 4 ++--
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/utils/index.js b/src/utils/index.js
index f607910c4ac..3af8b29b8ba 100644
--- a/src/utils/index.js
+++ b/src/utils/index.js
@@ -296,3 +296,7 @@ export function deepClone(source) {
 export function uniqueArr(arr) {
   return Array.from(new Set(arr))
 }
+
+export function isExternal(path) {
+  return /^(https?:|mailto:|tel:)/.test(path)
+}
diff --git a/src/views/layout/components/Sidebar/Link.vue b/src/views/layout/components/Sidebar/Link.vue
index 07793b9a29b..5d366f246c3 100644
--- a/src/views/layout/components/Sidebar/Link.vue
+++ b/src/views/layout/components/Sidebar/Link.vue
@@ -7,7 +7,7 @@
 </template>
 
 <script>
-import { validateURL } from '@/utils/validate'
+import { isExternal } from '@/utils'
 
 export default {
   props: {
@@ -18,7 +18,7 @@ export default {
   },
   methods: {
     isExternalLink(routePath) {
-      return validateURL(routePath)
+      return isExternal(routePath)
     },
     linkProps(url) {
       if (this.isExternalLink(url)) {
diff --git a/src/views/layout/components/Sidebar/SidebarItem.vue b/src/views/layout/components/Sidebar/SidebarItem.vue
index 9398af871f1..8a9b439687b 100644
--- a/src/views/layout/components/Sidebar/SidebarItem.vue
+++ b/src/views/layout/components/Sidebar/SidebarItem.vue
@@ -37,7 +37,7 @@
 <script>
 import path from 'path'
 import { generateTitle } from '@/utils/i18n'
-import { validateURL } from '@/utils/validate'
+import { isExternal } from '@/utils'
 import Item from './Item'
 import AppLink from './Link'
 import FixiOSBug from './FixiOSBug'
@@ -98,7 +98,7 @@ export default {
       return path.resolve(this.basePath, routePath)
     },
     isExternalLink(routePath) {
-      return validateURL(routePath)
+      return isExternal(routePath)
     },
     generateTitle
   }