From d7dedf15f179e48d7ed9113a8fa6e66e63246f7f Mon Sep 17 00:00:00 2001
From: ibuler <ibuler@qq.com>
Date: Wed, 21 Sep 2022 19:59:18 +0800
Subject: [PATCH] =?UTF-8?q?perf:=20=E4=BF=AE=E6=94=B9=E5=B9=B3=E5=8F=B0?=
 =?UTF-8?q?=E9=80=89=E6=8B=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../AssetList/components/PlatformDialog.vue   | 26 +++++++++++--------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/views/assets/Asset/AssetList/components/PlatformDialog.vue b/src/views/assets/Asset/AssetList/components/PlatformDialog.vue
index 464207e3b..d23dc6c4d 100644
--- a/src/views/assets/Asset/AssetList/components/PlatformDialog.vue
+++ b/src/views/assets/Asset/AssetList/components/PlatformDialog.vue
@@ -12,7 +12,7 @@
       <el-row :gutter="20">
         <el-collapse v-model="activeType" accordion>
           <el-collapse-item
-            v-for="(ps, categoryName) in sortedPlatforms"
+            v-for="(ps, categoryName) in iPlatforms"
             :key="categoryName"
             :title="categoryName"
             :name="categoryName"
@@ -60,6 +60,7 @@ export default {
       platforms: [],
       recentPlatformIds: [],
       activeType: 'host',
+      recentUsedLabel: this.$t('assets.RecentlyUsed'),
       typeIconMapper: {
         linux: 'fa-linux',
         windows: 'fa-windows',
@@ -79,15 +80,21 @@ export default {
       },
       get() { return this.visible }
     },
-    sortedPlatforms() {
-      const recentPlatforms = {}
+    iPlatforms() {
+      let recentPlatforms = {}
+      let platforms = {}
       if (this.category === 'all') {
-        recentPlatforms[this.$t('assets.RecentlyUsed')] = this.allRecentPlatforms
-        return { ...recentPlatforms, ...this.allSortedPlatforms }
+        recentPlatforms[this.recentUsedLabel] = this.allRecentPlatforms
+        platforms = this.allSortedPlatforms
       } else {
-        recentPlatforms[this.$t('assets.RecentlyUsed')] = this.recentPlatforms
-        return { ...recentPlatforms, ...this.typeSortedPlatforms }
+        recentPlatforms[this.recentUsedLabel] = this.typeRecentPlatforms
+        platforms = this.typeSortedPlatforms
       }
+      // 最近使用为空就不用了
+      if (recentPlatforms[this.recentUsedLabel].length === 0) {
+        recentPlatforms = {}
+      }
+      return { ...recentPlatforms, ...platforms }
     },
     allSortedPlatforms() {
       return _.groupBy(this.platforms, (item) => item.category.label)
@@ -96,9 +103,6 @@ export default {
       const typedPlatforms = this.platforms.filter(item => item.category.value === this.category)
       return _.groupBy(typedPlatforms, (item) => item.type.label)
     },
-    recentPlatforms() {
-      return this.category === 'all' ? this.allRecentPlatforms : this.typeRecentPlatforms
-    },
     allRecentPlatforms() {
       return this.recentPlatformIds
         .map(i => this.platforms.find(p => p.id === i))
@@ -112,7 +116,7 @@ export default {
     this.$axios.get('/api/v1/assets/platforms/').then(data => {
       this.platforms = data
       this.loadRecentPlatformIds()
-      this.activeType = Object.keys(this.sortedPlatforms)[0]
+      this.activeType = Object.keys(this.iPlatforms)[0]
     })
   },
   methods: {