From 62ed11e7ec91fa99ec66768e54d96398ca22eddf Mon Sep 17 00:00:00 2001
From: fit2bot <68588906+fit2bot@users.noreply.github.com>
Date: Mon, 18 Apr 2022 20:20:32 +0800
Subject: [PATCH] =?UTF-8?q?perf:=20=E4=BF=AE=E6=94=B9=E7=94=A8=E6=88=B7?=
 =?UTF-8?q?=E6=89=80=E5=B1=9E=E7=9A=84=20org=20(#1662)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* perf: 修改用户所属的 org

* perf: 修改组织问题

* perf: 修改优化 nav 切换

* perf: 修改 org id

* perf: 修改验证逻辑

* perf: 修改 orgs

Co-authored-by: ibuler <ibuler@qq.com>
---
 src/api/orgs.js                               |  6 ++
 src/api/users.js                              |  5 ++
 src/guards.js                                 | 23 +----
 .../components/NavHeader/AccountDropdown.vue  | 24 ++---
 .../components/NavHeader/Organization.vue     | 41 +++++----
 .../components/NavHeader/ViewSwitcher.vue     | 22 +++--
 src/router/audit/index.js                     |  7 +-
 src/router/console/index.js                   |  7 +-
 src/router/index.js                           |  7 +-
 src/router/profile/index.js                   |  2 +-
 src/router/workbench/index.js                 |  7 +-
 src/store/getters.js                          |  9 +-
 src/store/modules/app.js                      | 12 ++-
 src/store/modules/permission.js               |  3 -
 src/store/modules/users.js                    | 89 ++++++-------------
 src/utils/auth.js                             |  5 +-
 src/utils/jms.js                              | 46 ++++------
 src/utils/org.js                              |  6 +-
 src/utils/startup.js                          | 52 ++++++++---
 src/views/myassets/index.vue                  |  1 -
 src/views/profile/TempPassword.vue            |  4 +-
 .../tickets/TicketFlow/FlowCreateUpdate.vue   |  4 +-
 src/views/tickets/components/Session.vue      |  1 -
 23 files changed, 194 insertions(+), 189 deletions(-)

diff --git a/src/api/orgs.js b/src/api/orgs.js
index 34ec9acd7..823581c42 100644
--- a/src/api/orgs.js
+++ b/src/api/orgs.js
@@ -13,3 +13,9 @@ export function getCurrentOrg() {
     method: 'get'
   })
 }
+
+export default {
+  getCurrentOrg,
+  getOrgDetail
+}
+
diff --git a/src/api/users.js b/src/api/users.js
index 84b302629..3482c9e89 100644
--- a/src/api/users.js
+++ b/src/api/users.js
@@ -69,3 +69,8 @@ export function logout() {
 export function refreshSessionIdAge() {
   return getProfile()
 }
+
+export default {
+  getProfile,
+  getUserList
+}
diff --git a/src/guards.js b/src/guards.js
index ee484ddcc..82c6d8397 100644
--- a/src/guards.js
+++ b/src/guards.js
@@ -4,8 +4,7 @@ import NProgress from 'nprogress' // progress bar
 import 'nprogress/nprogress.css' // progress bar style
 import { startup } from '@/utils/startup'
 import store from '@/store'
-import { getPropView, hasRouteViewPerm, isSameView } from '@/utils/jms'
-import Vue from 'vue'
+import { isSameView } from '@/utils/jms'
 
 NProgress.configure({
   showSpinner: false
@@ -30,20 +29,6 @@ function generateViewRoutesIfChange({ to, from }) {
   }
 }
 
-async function changeCurrentViewIfNeed({ to, from, next }) {
-  if (!to.path || isSameView(to, from)) {
-    return
-  }
-  const hasPerm = hasRouteViewPerm(to)
-  Vue.$log.debug('Change has current view, has perm: ', hasPerm)
-  if (hasPerm) {
-    return
-  }
-  const view = getPropView()
-  Vue.$log.debug('Get prop view and goto: ', view)
-  next(`/${view}`)
-}
-
 function setPageTitle() {
   const currentRoute = router.currentRoute
   const loginTitle = store.getters.publicSettings['LOGIN_TITLE']
@@ -53,12 +38,6 @@ function setPageTitle() {
   }
 }
 
-router.beforeResolve(async(to, from, next) => {
-  /* must call `next` */
-  await changeCurrentViewIfNeed({ to, from, next })
-  next()
-})
-
 router.afterEach(async(to, from) => {
   // finish progress bar
   await setPageTitle()
diff --git a/src/layout/components/NavHeader/AccountDropdown.vue b/src/layout/components/NavHeader/AccountDropdown.vue
index 4f2014ab5..4bc375a72 100644
--- a/src/layout/components/NavHeader/AccountDropdown.vue
+++ b/src/layout/components/NavHeader/AccountDropdown.vue
@@ -7,10 +7,18 @@
         <i class="el-icon-arrow-down el-icon--right" />
       </span>
       <el-dropdown-menu slot="dropdown">
-        <el-dropdown-item icon="el-icon-user" command="profile">{{ $t('common.nav.Profile') }}</el-dropdown-item>
-        <el-dropdown-item v-if="$hasPerm('authentication.view_accesskey')" icon="el-icon-key" command="apiKey">{{ $t('common.nav.APIKey') }}</el-dropdown-item>
-        <el-dropdown-item v-if="$store.getters.publicSettings.AUTH_TEMP_TOKEN" icon="el-icon-magic-stick" command="tempPassword">{{ $t('common.nav.TempPassword') }}</el-dropdown-item>
-        <el-dropdown-item divided command="logout"><svg-icon icon-class="logout" style="margin-right: 4px" />{{ $t('common.nav.Logout') }}</el-dropdown-item>
+        <el-dropdown-item icon="el-icon-user" command="profile">
+          {{ $t('common.nav.Profile') }}
+        </el-dropdown-item>
+        <el-dropdown-item v-if="$hasPerm('authentication.view_accesskey')" icon="el-icon-key" command="apiKey">
+          {{ $t('common.nav.APIKey') }}
+        </el-dropdown-item>
+        <el-dropdown-item v-if="$store.getters.publicSettings.AUTH_TEMP_TOKEN" icon="el-icon-magic-stick" command="tempPassword">
+          {{ $t('common.nav.TempPassword') }}
+        </el-dropdown-item>
+        <el-dropdown-item divided command="logout"><svg-icon icon-class="logout" style="margin-right: 4px" />
+          {{ $t('common.nav.Logout') }}
+        </el-dropdown-item>
       </el-dropdown-menu>
     </el-dropdown>
   </div>
@@ -29,11 +37,7 @@ export default {
   },
   computed: {
     ...mapGetters([
-      'currentUser',
-      'currentRole',
-      'currentOrgRoles',
-      'orgs',
-      'currentOrgPerms'
+      'currentUser'
     ])
   },
   created() {
@@ -52,7 +56,7 @@ export default {
           this.$router.push('/profile/key')
           break
         case 'tempPassword':
-          this.$router.push('/profile/tempPassword')
+          this.$router.push('/profile/temp-password')
       }
     },
     logout() {
diff --git a/src/layout/components/NavHeader/Organization.vue b/src/layout/components/NavHeader/Organization.vue
index 062621c83..87439fb69 100644
--- a/src/layout/components/NavHeader/Organization.vue
+++ b/src/layout/components/NavHeader/Organization.vue
@@ -11,7 +11,7 @@
     </template>
 
     <el-option-group
-      v-for="group in orgOption"
+      v-for="group in orgGroups"
       :key="group.label"
       :label="group.label"
       class="option-group"
@@ -29,7 +29,6 @@
         </span>
         <span>{{ item.name }}</span>
       </el-option>
-
     </el-option-group>
   </el-select>
 </template>
@@ -55,15 +54,10 @@ export default {
   computed: {
     ...mapGetters([
       'currentOrg',
-      'currentRole',
-      'orgs'
-    ])
-  },
-  created() {
-    this.init()
-  },
-  methods: {
-    init() {
+      'usingOrgs',
+      'currentViewRoute'
+    ]),
+    orgActionsGroup() {
       const orgActions = {
         label: this.$t('xpack.Organization.OrganizationList'),
         options: [
@@ -82,14 +76,25 @@ export default {
         ]
       }
       const hasPerms = this.$hasPerm('orgs.view_organization | orgs.add_organization')
-      this.orgOption = [
-        (hasPerms && orgActions),
-        {
-          label: this.$t('xpack.Organization.AllOrganization'),
-          options: this.orgs
-        }
-      ]
+      const isConsole = this.currentViewRoute.name === 'console'
+      return hasPerms && isConsole ? orgActions : {}
     },
+    orgChoicesGroup() {
+      return {
+        label: this.$t('xpack.Organization.AllOrganization'),
+        options: this.usingOrgs
+      }
+    },
+    orgGroups() {
+      return [
+        this.orgActionsGroup,
+        this.orgChoicesGroup
+      ]
+    }
+  },
+  created() {
+  },
+  methods: {
     changeOrg(orgId) {
       if (orgId === 'create') {
         this.$router.push({ name: 'OrganizationCreate' })
diff --git a/src/layout/components/NavHeader/ViewSwitcher.vue b/src/layout/components/NavHeader/ViewSwitcher.vue
index df0533803..817523cdc 100644
--- a/src/layout/components/NavHeader/ViewSwitcher.vue
+++ b/src/layout/components/NavHeader/ViewSwitcher.vue
@@ -19,7 +19,6 @@
       <el-menu-item
         v-for="view of views"
         :key="view.name"
-        v-perms="view.perms"
         :index="view.name"
       >
         <i v-if="mode === 'horizontal'" class="icons" :class="view.meta.icon" />
@@ -31,6 +30,7 @@
 
 <script>
 import { mapGetters } from 'vuex'
+import store from '@/store'
 
 export default {
   name: 'ViewSwitcher',
@@ -49,12 +49,17 @@ export default {
   },
   computed: {
     ...mapGetters([
-      'currentViewRoute'
+      'currentViewRoute',
+      'viewRoutes'
     ]),
     views() {
-      return this.$store.state.permission.addRoutes.filter(
-        item => item.meta?.showNavSwitcher
-      )
+      return this.viewRoutes.filter((item) => {
+        let show = item.meta?.showNavSwitcher
+        if (typeof show === 'function') {
+          show = show()
+        }
+        return show
+      })
     },
     viewsMapper() {
       const mapper = {}
@@ -67,11 +72,12 @@ export default {
       return this.viewsMapper[this.currentViewRoute.name]
     }
   },
-  created() {
-  },
   methods: {
-    handleSelectView(key, keyPath) {
+    async handleSelectView(key, keyPath) {
       const routeName = this.viewsMapper[key] || '/'
+      localStorage.setItem('PreView', key)
+      // Next 之前要重置 init 状态，否则这些路由守卫就不走了
+      await store.dispatch('app/reset')
       this.$router.push(routeName)
     }
   }
diff --git a/src/router/audit/index.js b/src/router/audit/index.js
index 638e69832..135fc9f98 100644
--- a/src/router/audit/index.js
+++ b/src/router/audit/index.js
@@ -4,6 +4,7 @@ import i18n from '@/i18n/i18n'
 import SessionRoutes from './sessions'
 import LogRoutes from './logs'
 import empty from '@/layout/empty'
+import store from '@/store'
 
 export default {
   path: '/audit/',
@@ -13,8 +14,10 @@ export default {
   meta: {
     title: i18n.t('common.nav.Audits'),
     icon: 'el-icon-s-claim',
-    showNavSwitcher: true,
-    permissions: ['rbac.view_audit'],
+    showNavSwitcher: () => {
+      return store.getters.auditOrgs.length > 0
+    },
+    permissions: [],
     view: 'audit'
   },
   children: [
diff --git a/src/router/console/index.js b/src/router/console/index.js
index 6ffc37327..3bdac0827 100644
--- a/src/router/console/index.js
+++ b/src/router/console/index.js
@@ -1,6 +1,7 @@
 import Layout from '@/layout/index'
 import i18n from '@/i18n/i18n'
 import empty from '@/layout/empty'
+import store from '@/store'
 
 import UsersRoute from './users'
 import AssetsRoute from './assets'
@@ -20,8 +21,10 @@ export default {
     icon: 'el-icon-s-operation',
     view: 'console',
     type: 'view',
-    showNavSwitcher: true,
-    permissions: ['rbac.view_console']
+    showNavSwitcher: () => {
+      return store.getters.consoleOrgs.length > 0
+    },
+    permissions: []
   },
   children: [
     {
diff --git a/src/router/index.js b/src/router/index.js
index b655b9168..dc39c1b8e 100644
--- a/src/router/index.js
+++ b/src/router/index.js
@@ -51,14 +51,13 @@ export const constantRoutes = [
           title: i18n.t('route.Overview')
         },
         beforeEnter: (to, from, next) => {
-          console.log('Enter home view')
-          const preferView = getPermedPreferView()
+          const preferView = getFirstPermedView()
           if (preferView) {
-            console.log('Go to preferView: ', preferView)
             next(`/${preferView}/`)
             return false
           }
           next()
+          return false
         }
       }
     ]
@@ -83,7 +82,7 @@ import workbenchViewRoutes from './workbench'
 import ticketsRoutes from './tickets'
 import settingsRoutes from './settings'
 import profileRoutes from './profile'
-import { getPermedPreferView } from '@/utils/jms'
+import { getFirstPermedView } from '@/utils/jms'
 
 /**
  * admin
diff --git a/src/router/profile/index.js b/src/router/profile/index.js
index 30c8899d1..a1652ead7 100644
--- a/src/router/profile/index.js
+++ b/src/router/profile/index.js
@@ -55,7 +55,7 @@ export default {
       }
     },
     {
-      path: '/profile/tempPassword',
+      path: '/profile/temp-password',
       component: () => import('@/views/profile/TempPassword'),
       name: 'TempPassword',
       meta: {
diff --git a/src/router/workbench/index.js b/src/router/workbench/index.js
index 6bf12fd08..8c21a50b2 100644
--- a/src/router/workbench/index.js
+++ b/src/router/workbench/index.js
@@ -2,6 +2,7 @@ import Layout from '@/layout'
 import i18n from '@/i18n/i18n'
 import { BASE_URL } from '@/utils/common'
 import empty from '@/layout/empty'
+import store from '@/store'
 
 export default {
   path: '/workbench/',
@@ -13,9 +14,11 @@ export default {
     type: 'view',
     view: 'workbench',
     icon: 'el-icon-user-solid',
-    showNavSwitcher: true,
+    showNavSwitcher: () => {
+      return store.getters.workbenchOrgs.length > 0
+    },
     showOrganization: true,
-    permissions: ['rbac.view_workbench']
+    permissions: []
   },
   children: [
     // 404 page must be placed at the end !!!
diff --git a/src/store/getters.js b/src/store/getters.js
index f2fdabfef..4c4263c5a 100644
--- a/src/store/getters.js
+++ b/src/store/getters.js
@@ -1,8 +1,13 @@
 const getters = {
   sidebar: state => state.app.sidebar,
   device: state => state.app.device,
+  inited: state => state.app.inited,
   isMobile: state => state.app.device === 'mobile',
   token: state => state.users.token,
+  consoleOrgs: state => state.users.consoleOrgs,
+  auditOrgs: state => state.users.auditOrgs,
+  workbenchOrgs: state => state.users.workbenchOrgs,
+  usingOrgs: state => state.users.usingOrgs,
   currentOrg: state => state.users.currentOrg,
   currentOrgIsDefault: state => state.users.currentOrg['is_default'],
   currentOrgIsRoot: state => {
@@ -11,6 +16,7 @@ const getters = {
   currentRole: state => state.users.currentRole,
   currentUser: state => state.users.profile,
   currentViewRoute: state => state.permission.currentViewRoute,
+  viewRoutes: state => state.permission.addRoutes,
   publicSettings: state => state.settings.publicSettings,
   currentOrgRoles: state => state.users.roles,
   currentOrgPerms: state => state.users.perms,
@@ -19,7 +25,6 @@ const getters = {
   tableConfig: state => state.table.tableConfig,
   currentUserIsSuperAdmin: state => state.users.isSuperAdmin,
   currentUserIsAdmin: state => state.users.isAdmin,
-  hasValidLicense: state => state.settings.hasValidLicense,
-  orgs: state => state.users.orgs
+  hasValidLicense: state => state.settings.hasValidLicense
 }
 export default getters
diff --git a/src/store/modules/app.js b/src/store/modules/app.js
index 7ea7e3322..7e29649af 100644
--- a/src/store/modules/app.js
+++ b/src/store/modules/app.js
@@ -5,7 +5,8 @@ const state = {
     opened: Cookies.get('sidebarStatus') ? !!+Cookies.get('sidebarStatus') : true,
     withoutAnimation: false
   },
-  device: 'desktop'
+  device: 'desktop',
+  inited: false
 }
 
 const mutations = {
@@ -25,6 +26,9 @@ const mutations = {
   },
   TOGGLE_DEVICE: (state, device) => {
     state.device = device
+  },
+  SET_INIT: (state, value) => {
+    state.inited = value
   }
 }
 
@@ -37,6 +41,12 @@ const actions = {
   },
   toggleDevice({ commit }, device) {
     commit('TOGGLE_DEVICE', device)
+  },
+  init({ commit }) {
+    commit('SET_INIT', true)
+  },
+  reset({ commit }) {
+    commit('SET_INIT', false)
   }
 }
 
diff --git a/src/store/modules/permission.js b/src/store/modules/permission.js
index 36565de4c..14f02be22 100644
--- a/src/store/modules/permission.js
+++ b/src/store/modules/permission.js
@@ -193,9 +193,6 @@ const actions = {
           break
         }
       }
-      if (viewRoute.meta?.showNavSwitcher) {
-        localStorage.setItem('PreView', viewName)
-      }
       commit('SET_VIEW_ROUTE', viewRoute)
       resolve(viewRoute)
     })
diff --git a/src/store/modules/users.js b/src/store/modules/users.js
index 96a387c5c..6ccbc75b7 100644
--- a/src/store/modules/users.js
+++ b/src/store/modules/users.js
@@ -1,12 +1,11 @@
-import { logout, getProfile } from '@/api/users'
+import { logout, getProfile as apiGetProfile } from '@/api/users'
 import {
   getCurrentOrgLocal,
-  getCurrentRoleLocal,
   getTokenFromCookie,
-  saveCurrentOrgLocal,
-  saveCurrentRoleLocal
+  saveCurrentOrgLocal
 } from '@/utils/auth'
 import { resetRouter } from '@/router'
+import Vue from 'vue'
 
 const getDefaultState = () => {
   return {
@@ -14,10 +13,14 @@ const getDefaultState = () => {
     currentOrg: '',
     profile: {},
     username: '',
-    orgs: [],
+    auditOrgs: [],
+    consoleOrgs: [],
+    workbenchOrgs: [],
+    usingOrgs: [],
     perms: [],
     MFAVerifyAt: null,
     isSuperAdmin: false,
+    isAdmin: false,
     hasAdminPerm: false,
     hasAuditPerm: false
   }
@@ -34,57 +37,37 @@ const mutations = {
   },
   SET_PROFILE: (state, profile) => {
     state.profile = profile
-    const username = profile.username
-    state.username = username
-    state.currentOrg = getCurrentOrgLocal(username)
-    state.currentRole = getCurrentRoleLocal(username)
+    state.username = profile.username
     state.perms = profile.perms
-    state.orgs = profile.orgs
+    state.consoleOrgs = profile['console_orgs']
+    state.workbenchOrgs = profile['workbench_orgs']
+    state.auditOrgs = profile['audit_orgs']
+    state.currentOrg = getCurrentOrgLocal(profile.username)
   },
-  SET_ORGS: (state, orgs) => {
-    state.orgs = orgs
+  SET_USING_ORGS: (state, orgs) => {
+    state.usingOrgs = orgs
   },
   MODIFY_ORG: (state, org) => {
-    state.orgs = state.orgs.map(oldOrg => {
+    state.consoleOrgs = state.consoleOrgs.map(oldOrg => {
       if (oldOrg.id === org.id) {
         oldOrg.name = org.name
       }
       return oldOrg
-    }
-    )
+    })
   },
   ADD_ORG: (state, org) => {
-    state.orgs.push(org)
+    state.consoleOrgs.push(org)
   },
   SET_CURRENT_ORG(state, org) {
     state.currentOrg = org
     saveCurrentOrgLocal(state.username, org)
   },
-  SET_CURRENT_ROLE(state, role) {
-    state.currentRole = role
-    saveCurrentRoleLocal(state.username, role)
-  },
   SET_MFA_VERIFY(state) {
     state.MFAVerifyAt = (new Date()).valueOf()
   }
 }
 
 const actions = {
-  // user login
-  // login({ commit }, userInfo) {
-  //   const { username, password } = userInfo
-  //   return new Promise((resolve, reject) => {
-  //     login({ username: username.trim(), password: password }).then(response => {
-  //       const { data } = response
-  //       commit('SET_TOKEN', data.token)
-  //       setToken(data.token)
-  //       resolve()
-  //     }).catch(error => {
-  //       reject(error)
-  //     })
-  //   })
-  // },
-
   // get user Profile
   getProfile({ commit, state }, refresh = false) {
     return new Promise((resolve, reject) => {
@@ -92,7 +75,7 @@ const actions = {
         resolve(state.profile)
         return
       }
-      getProfile().then(response => {
+      apiGetProfile().then(response => {
         if (!response) {
           reject('Verification failed, please Login again.')
         }
@@ -104,24 +87,17 @@ const actions = {
       })
     })
   },
-  getInOrgs({ commit, dispatch, state }, refresh) {
-    return new Promise((resolve, reject) => {
-      if (!refresh && state.role && state.role.length > 0) {
-        return resolve(state.roles)
-      }
-      dispatch('getProfile').then(profile => {
-        const { orgs } = profile
-        commit('SET_ORGS', orgs)
-        resolve(orgs)
-      }).catch((e) => reject(e))
-    })
-  },
   addAdminOrg({ commit, state }, org) {
     commit('ADD_ORG', org)
   },
   modifyOrg({ commit, state }, org) {
     commit('MODIFY_ORG', org)
   },
+  setUsingOrgs({ commit, state, rootGetters }) {
+    const viewName = rootGetters.currentViewRoute.name
+    const usingOrgs = state[`${viewName}Orgs`] || []
+    commit('SET_USING_ORGS', usingOrgs)
+  },
   // user logout
   logout({ commit, state }) {
     return new Promise((resolve, reject) => {
@@ -135,23 +111,16 @@ const actions = {
       })
     })
   },
-
-  // remove token
-  resetToken({ commit }) {
-    return new Promise(resolve => {
-      // removeToken() // must remove  token  first
-      commit('RESET_STATE')
-      resolve()
-    })
-  },
   setCurrentOrg({ commit }, data) {
     commit('SET_CURRENT_ORG', data)
   },
-  setCurrentRole({ commit }, role) {
-    commit('SET_CURRENT_ROLE', role)
-  },
   setMFAVerify({ commit }) {
     commit('SET_MFA_VERIFY')
+  },
+  changeToView({ commit }, viewName) {
+    const usingOrgs = state[`${viewName}Orgs`] || []
+    Vue.$log.debug('Set using orgs: ', viewName, usingOrgs)
+    commit('SET_USING_ORGS', usingOrgs)
   }
 }
 
diff --git a/src/utils/auth.js b/src/utils/auth.js
index 3bc96dd72..a9c43f1d9 100644
--- a/src/utils/auth.js
+++ b/src/utils/auth.js
@@ -5,7 +5,10 @@ const CURRENT_ORG_KEY = 'jms_current_org'
 const CURRENT_ROLE_KEY = 'jms_current_role'
 
 export function getTokenFromCookie() {
-  const cookieNamePrefix = VueCookie.get('SESSION_COOKIE_NAME_PREFIX', '')
+  let cookieNamePrefix = VueCookie.get('SESSION_COOKIE_NAME_PREFIX', '')
+  if (cookieNamePrefix === '""') {
+    cookieNamePrefix = ''
+  }
   return VueCookie.get(cookieNamePrefix + TOKEN_KEY)
 }
 
diff --git a/src/utils/jms.js b/src/utils/jms.js
index b789e0b66..16555dd3f 100644
--- a/src/utils/jms.js
+++ b/src/utils/jms.js
@@ -1,5 +1,4 @@
 import store from '@/store'
-import Vue from 'vue'
 import { constantRoutes } from '@/router'
 
 export function openTaskPage(taskId) {
@@ -80,24 +79,20 @@ export function hasActionPerm(route, action) {
   return hasPermission(permsRequired)
 }
 
-const viewRequirePermsMapper = {
-  console: 'rbac.view_console',
-  audit: 'rbac.view_audit',
-  workbench: 'rbac.view_workbench'
+export function getPermedViews() {
+  const viewShowMapper = [
+    ['console', store.getters.consoleOrgs.length > 0],
+    ['audit', store.getters.auditOrgs.length > 0],
+    ['workbench', true]
+  ]
+  return viewShowMapper.filter(i => {
+    return i[1]
+  }).map(i => i[0])
 }
 
-export function getViewRequirePerms(view) {
-  return viewRequirePermsMapper[view] || 'super'
-}
-
-export function getPermedPreferView() {
-  for (const [view, perms] of Object.entries(viewRequirePermsMapper)) {
-    const hasPerm = hasPermission(perms)
-    Vue.$log.debug('Has view perm: ', view, hasPerm)
-    if (hasPerm) {
-      return view
-    }
-  }
+export function getFirstPermedView() {
+  const hasPermedViews = getPermedViews()
+  return hasPermedViews[0]
 }
 
 export function isSameView(to, from) {
@@ -107,13 +102,13 @@ export function isSameView(to, from) {
 }
 
 export function getPropView() {
+  const hasPermedViews = getPermedViews()
   const preView = localStorage.getItem('PreView')
-  const preViewRequirePerms = getViewRequirePerms(preView)
-  const hasPerm = hasPermission(preViewRequirePerms)
+  const hasPerm = hasPermedViews.indexOf(preView) > -1
   if (hasPerm) {
     return preView
   }
-  const preferView = getPermedPreferView()
+  const preferView = getFirstPermedView()
   if (preferView) {
     return preferView
   }
@@ -125,18 +120,9 @@ export function getApiUrlRequirePerms(url, action) {
   return [`${app}.${action}_${resource}`]
 }
 
-export function getRouteViewRequirePerms(route) {
-  const viewName = route.path.split('/')[1]
-  return getViewRequirePerms(viewName)
-}
-
 export function hasRouteViewPerm(route) {
-  if (route.name) {
-    return hasPermission(route.meta.permissions)
-  }
   const viewName = route.path.split('/')[1]
-  const perms = getViewRequirePerms(viewName)
-  return hasPermission(perms)
+  return getPermedViews().indexOf(viewName) > -1
 }
 
 export function getConstRouteName() {
diff --git a/src/utils/org.js b/src/utils/org.js
index 69533341c..4ff1b8e88 100644
--- a/src/utils/org.js
+++ b/src/utils/org.js
@@ -3,10 +3,9 @@ import { getOrgDetail } from '@/api/orgs'
 import store from '@/store'
 
 export const DEFAULT_ORG_ID = '00000000-0000-0000-0000-000000000002'
-// const ROOT_ORG_ID = '00000000-0000-0000-0000-000000000000'
 
 function getPropOrg() {
-  const orgs = store.getters.orgs
+  const orgs = store.getters.usingOrgs
   const defaultOrg = orgs.find((item) => item.is_default)
   if (defaultOrg) {
     return defaultOrg
@@ -22,7 +21,7 @@ function change2PropOrg() {
 function hasCurrentOrgPermission() {
   const currentOrg = store.getters.currentOrg
   const currentOrgId = currentOrg.id
-  const orgs = store.getters.orgs
+  const orgs = store.getters.usingOrgs
   return orgs.find((item) => item.id === currentOrgId)
 }
 
@@ -33,7 +32,6 @@ async function changeOrg(orgId) {
   } else {
     console.debug('Change to org: ', org)
   }
-  localStorage.setItem('PreView', '')
 
   store.dispatch('users/setCurrentOrg', org).then(() => {
     // console.log('Set current org to: ', org)
diff --git a/src/utils/startup.js b/src/utils/startup.js
index fe3b399df..7d237bc88 100644
--- a/src/utils/startup.js
+++ b/src/utils/startup.js
@@ -1,15 +1,15 @@
 // import getPageTitle from '@/utils/get-page-title'
 import store from '@/store'
-import router from '@/router'
+import router, { resetRouter } from '@/router'
 import Vue from 'vue'
 import { Message } from 'element-ui'
 import 'nprogress/nprogress.css' // progress bar style
 import { getTokenFromCookie } from '@/utils/auth'
 import orgUtil from '@/utils/org'
-import { getCurrentOrg } from '@/api/orgs'
+import orgs from '@/api/orgs'
+import { getPropView, hasRouteViewPerm } from '@/utils/jms'
 
 const whiteList = ['/login', process.env.VUE_APP_LOGIN_PATH] // no redirect whitelist
-let initial = false
 
 function reject(msg) {
   return new Promise((resolve, reject) => reject(msg))
@@ -51,25 +51,28 @@ async function getPublicSetting({ to, from, next }) {
 }
 
 async function refreshCurrentOrg() {
-  getCurrentOrg().then(org => {
+  orgs.getCurrentOrg().then(org => {
     store.dispatch('users/setCurrentOrg', org)
   })
 }
 
 async function changeCurrentOrgIfNeed({ to, from, next }) {
-  await store.dispatch('users/getInOrgs')
-  const adminOrgs = store.getters.orgs
-  if (!adminOrgs || adminOrgs.length === 0) {
+  await store.dispatch('users/getProfile')
+
+  const usingOrgs = store.getters.usingOrgs
+  if (!usingOrgs || usingOrgs.length === 0) {
+    Vue.$log.error('No using orgs, return: ', usingOrgs)
     return
   }
   await refreshCurrentOrg()
   const currentOrg = store.getters.currentOrg
   if (!currentOrg || typeof currentOrg !== 'object') {
+    Vue.$log.error('Current org is null or not a object')
     orgUtil.change2PropOrg()
     return reject('Change prop org')
   }
   if (!orgUtil.hasCurrentOrgPermission()) {
-    console.error('Not has current org permission')
+    Vue.$log.error('Not has current org permission')
     orgUtil.change2PropOrg()
     return reject('Change prop org')
   }
@@ -77,6 +80,7 @@ async function changeCurrentOrgIfNeed({ to, from, next }) {
 
 export async function generatePageRoutes({ to, from, next }) {
   // determine whether the user has obtained his permission roles through getProfile
+  resetRouter()
 
   try {
     // try get user profile
@@ -113,18 +117,38 @@ export async function checkUserFirstLogin({ to, from, next }) {
   }
 }
 
-export async function startup({ to, from, next }) {
-  if (initial) {
-    return true
+export async function changeCurrentViewIfNeed({ to, from, next }) {
+  let view = to.path.split('/')[1]
+  if (['console', 'audit', 'workbench', ''].indexOf(view) < 0) {
+    Vue.$log.debug('Current view no need check', view)
+    return
   }
-  initial = true
+
+  const hasPerm = hasRouteViewPerm(to)
+  Vue.$log.debug('Change has current view, has perm: ', hasPerm)
+  if (hasPerm) {
+    await store.dispatch('users/changeToView', view)
+    return
+  }
+  view = getPropView()
+  // Next 之前要重置 init 状态，否则这些路由守卫就不走了
+  await store.dispatch('app/reset')
+  next(`/${view}/`)
+  return new Promise((resolve, reject) => reject(''))
+}
+
+export async function startup({ to, from, next }) {
+  // if (store.getters.inited) { return true }
+  console.log('Start up')
+  if (store.getters.inited) { return true }
+  await store.dispatch('app/init')
 
   // set page title
   await getPublicSetting({ to, from, next })
-  // await setHeadTitle({ to, from, next })
   await checkLogin({ to, from, next })
-  await generatePageRoutes({ to, from, next })
+  await changeCurrentViewIfNeed({ to, from, next })
   await changeCurrentOrgIfNeed({ to, from, next })
+  await generatePageRoutes({ to, from, next })
   await checkUserFirstLogin({ to, from, next })
   return true
 }
diff --git a/src/views/myassets/index.vue b/src/views/myassets/index.vue
index d6f57dca3..5e522bb97 100644
--- a/src/views/myassets/index.vue
+++ b/src/views/myassets/index.vue
@@ -1,6 +1,5 @@
 <template>
   <div>
-    <Announcement />
     <GenericTreeListPage
       :table-config="tableConfig"
       :header-actions="headerActions"
diff --git a/src/views/profile/TempPassword.vue b/src/views/profile/TempPassword.vue
index 88bcd17ef..616855ee7 100644
--- a/src/views/profile/TempPassword.vue
+++ b/src/views/profile/TempPassword.vue
@@ -9,6 +9,7 @@
 
 <script>
 import { GenericListPage } from '@/layout/components'
+import { ShowKeyFormatter } from '@/components/TableFormatters'
 
 export default {
   components: {
@@ -26,7 +27,8 @@ export default {
         ],
         columnsMeta: {
           secret: {
-            label: this.$t('common.nav.TempPassword')
+            label: this.$t('common.nav.TempPassword'),
+            formatter: ShowKeyFormatter
           },
           expire: {
             label: this.$t('setting.Expired') + '( s )'
diff --git a/src/views/tickets/TicketFlow/FlowCreateUpdate.vue b/src/views/tickets/TicketFlow/FlowCreateUpdate.vue
index 1a109be5e..806ab7f3e 100644
--- a/src/views/tickets/TicketFlow/FlowCreateUpdate.vue
+++ b/src/views/tickets/TicketFlow/FlowCreateUpdate.vue
@@ -56,8 +56,8 @@ export default {
     }
   },
   mounted() {
-    if (this.$store.state.users.profile.orgs?.length > 0) {
-      this.initial.org_id = this.$store.state.users.profile.orgs[0].id
+    if (this.$store.getters.consoleOrgs) {
+      this.initial.org_id = this.$store.getters.consoleOrgs[0].id
     }
     this.loading = false
   }
diff --git a/src/views/tickets/components/Session.vue b/src/views/tickets/components/Session.vue
index 52e528344..ac4558c9d 100644
--- a/src/views/tickets/components/Session.vue
+++ b/src/views/tickets/components/Session.vue
@@ -84,7 +84,6 @@ export default {
   },
   methods: {
     init() {
-      console.log('this.object.state', this.object)
       this.loading = true
       const url = `/api/v1/tickets/tickets/${this.object.id}/session/`
       this.$axios({