Merge pull request '修改分模块' (#3) from zwq into develop

Reviewed-on: #3

src/lang/i18n.js

@@ -254,12 +254,8 @@ export default {
   },
   routerTitle: {
     dashboard: {
-      zh: '',
-      en: '',
-      dashboard: {
-        zh: '仪表盘',
-        en: 'Dashboard'
-      }
+      zh: '仪表盘',
+      en: 'Dashboard'
     },
     basic: {
       zh: '系统管理',

src/layout/components/Sidebar/index.vue

@@ -1,8 +1,8 @@
 <!--
  * @Author: your name
  * @Date: 2021-01-27 10:07:42
- * @LastEditTime: 2021-03-08 11:59:59
- * @LastEditors: gtz
+ * @LastEditTime: 2021-12-22 16:06:25
+ * @LastEditors: zwq
  * @Description: In User Settings Edit
  * @FilePath: \mt-bus-fe\src\layout\components\Sidebar\index.vue
 -->
@@ -31,31 +31,17 @@ import { mapGetters } from 'vuex'
 import Logo from './Logo'
 import SidebarItem from './SidebarItem'
 import variables from '@/styles/variables.scss'
-// import { constantRoutes } from '@/router'
-import store from '@/store'
+import { constantRoutes } from '@/router'
 export default {
   components: { SidebarItem, Logo },
   computed: {
     ...mapGetters([
       'sidebar',
-      'choicepart',
+      'permission_routes',
       'language'
     ]),
-    partList() {
-      const cangoList = []
-      const constantRoutes = store.getters.permission_routes
-      constantRoutes.map(item => {
-        if (!item.hidden && item.meta) {
-          cangoList.push(item)
-        }
-      })
-      const formatList = cangoList.map((item, index) => {
-        return this.setIndex(item, index)
-      })
-      return formatList
-    },
     routeList() {
-      return [this.partList[this.choicepart]]
+      return constantRoutes
     },
     activeMenu() {
       const route = this.$route
@@ -67,6 +53,7 @@ export default {
       return path
     },
     showLogo() {
+      console.log(this.$route)
       return this.$store.state.settings.sidebarLogo
     },
     variables() {
@@ -75,24 +62,6 @@ export default {
     isCollapse() {
       return !this.sidebar.opened
     }
-  },
-  watch: {
-    $route: function(val) {
-      if (val.meta.routeIndex >= 0) {
-        this.$store.dispatch('app/setChoicepart', val.meta.routeIndex)
-      }
-    }
-  },
-  methods: {
-    setIndex(list, index) {
-      list.meta.routeIndex = index
-      if (list.children) {
-        list.children.map(item => {
-          this.setIndex(item, index)
-        })
-      }
-      return list
-    }
   }
 }
 </script>

src/layout/components/TagsView/index.vue

@@ -45,8 +45,8 @@ export default {
       return this.$store.state.tagsView.visitedViews
     },
     routes() {
-      // return this.$store.state.permission.routes
-      return []
+      return this.$store.state.permission.routes
+      // return []
     }
   },
   watch: {

src/permission.js

@@ -1,12 +1,11 @@
 /*
  * @Date: 2020-12-14 09:07:03
- * @LastEditors: gtz
- * @LastEditTime: 2021-02-25 09:42:36
+ * @LastEditors: zwq
+ * @LastEditTime: 2021-12-22 16:13:30
  * @FilePath: \basic-admin\src\permission.js
  * @Description: 路由权限检查
  */
 import router from './router'
-// import { resetRouter } from '@/router'
 import { Message } from 'element-ui'
 import NProgress from 'nprogress' // progress bar
 import 'nprogress/nprogress.css' // progress bar style
@@ -35,38 +34,28 @@ router.beforeEach(async(to, from, next) => {
       next({ path: '/' })
       NProgress.done() // hack: https://github.com/PanJiaChen/vue-element-admin/pull/2939
     } else {
-      const hasRoles = store.getters.menus && store.getters.menus.length > 0
-      console.log('store.getters.menus', store.getters.menus)
-      console.log('store.getters', store.getters)
-      console.log('hasRoles=', hasRoles)
-      if (hasRoles) {
-        console.log('hasRoles=', store.getters.menus)
+      // next()
+      NProgress.done()
+      try {
+      //   // generate accessible routes map based on roles
+        // const accessRoutes = await store.dispatch('permission/generateRoutes')
+        // console.log(accessRoutes)
+        // router.addRoutes(accessRoutes)
         next()
-        NProgress.done()
-      } else {
-        try {
-          //   // generate accessible routes map based on roles
-          const { menus } = await store.dispatch('user/getInfo')
-          console.log(menus)
-          const accessRoutes = await store.dispatch('permission/generateRoutes', menus)
-          console.log('accessRoutes', accessRoutes)
-          router.addRoutes(accessRoutes)
-          next({ ...to, replace: true })
         // hack method to ensure that addRoutes is complete
         // set the replace: true, so the navigation will not leave a history record
         // next({ ...to, replace: true })
-        } catch (error) {
+      } catch (error) {
         // remove token and go to login page to re-login
-          next(`/login?redirect=${to.path}`)
-          await store.dispatch('user/resetToken')
-          Message.error(error || 'Has Error')
-          NProgress.done()
-        }
+        next(`/login?redirect=${to.path}`)
+        await store.dispatch('user/resetToken')
+        Message.error(error || 'Has Error')
+        NProgress.done()
       }
     }
   } else {
     /* has no token*/
-    console.log('has no token to.path=' + to.path)
+    console.log('to.path=' + to.path)
     console.log('whiteList.indexOf(to.path)=' + whiteList.indexOf(to.path))
     if (whiteList.indexOf(to.path) !== -1) {
       // in the free login whitelist, go directly

src/router/index.js

@@ -2,7 +2,6 @@ import Vue from 'vue'
 import Router from 'vue-router'
 import i18n from '@/lang/i18n'
 import Cookies from 'js-cookie'
-import store from '@/store'
 
 const routerTitle = i18n.routerTitle
 const language = Cookies.get('language')
@@ -60,11 +59,24 @@ export const constantRoutes = [
   },
   {
     path: '/',
-    component: () => import('@/views/ChoicePart'),
-    hidden: true,
-    meta: { requireToken: true }
+    component: Layout,
+    redirect: '/dashboard',
+    children: [
+      {
+        path: 'dashboard',
+        component: () => import('@/views/dashboard/index'),
+        name: 'Dashboard',
+        meta: { title: routerTitle.dashboard?.[language] || routerTitle.dashboard.en, icon: 'form', iconPart: 'dashboard', affix: true, required: true, requireToken: true }
+      }
+    ]
   },
   // {
+  //   path: '/',
+  //   component: () => import('@/views/ChoicePart'),
+  //   hidden: true,
+  //   meta: { requireToken: true }
+  // },
+  // {
   //   path: '/auth-redirect',
   //   component: () => import('@/views/login/auth-redirect'),
   //   hidden: true
@@ -78,28 +90,7 @@ export const constantRoutes = [
     path: '/401',
     component: () => import('@/views/error-page/401'),
     hidden: true
-  }
-  // 工单管理、厂务管理、包装管理、接口管理、数据采集、报表、质量管理、SPC
-  /** when your routing map is too long, you can split it into small modules **/
-  // 404 page must be placed at the end !!!
-  // { path: '*', redirect: '/404', hidden: true }
-]
-
-export const dynamicRoutes = [
-  // {
-  //   path: '/dashboard',
-  //   component: Layout,
-  //   redirect: '/dashboard',
-  //   name: 'dashboard',
-  //   children: [
-  //     {
-  //       path: 'dashboard',
-  //       component: () => import('@/views/dashboard/index'),
-  //       name: 'Dashboard',
-  //       meta: { title: routerTitle.dashboard.dashboard?.[language] || routerTitle.dashboard.dashboard.en, icon: 'dashboard', affix: true }
-  //     }
-  //   ]
-  // },
+  },
   {
     path: '/basicData',
     component: Layout,
@@ -511,7 +502,7 @@ export const dynamicRoutes = [
       path: 'recipe',
       component: () => import('@/views/EquipmentManager/RecipeManager'),
       name: 'RecipeManage',
-      meta: { title: routerTitle.equipment.recipe?.[language] || routerTitle.equipment.recipe?.en, icon: 'form', affix: true }
+      meta: { title: routerTitle.equipment.recipe?.[language] || routerTitle.equipment.recipe.en, icon: 'form', affix: true }
     }, {
       path: 'sparepart',
       component: () => import('@/views/EquipmentManager/sqarepart'),
@@ -529,7 +520,7 @@ export const dynamicRoutes = [
       component: () => import('@/views/EquipmentManager/RecipeManager/subpage/detail'),
       name: 'RecipeParamManage',
       hidden: true,
-      meta: { title: routerTitle.equipment.recipeform?.[language] || routerTitle.equipment.recipeform?.en, icon: 'form', affix: true }
+      meta: { title: routerTitle.equipment.recipeform?.[language] || routerTitle.equipment.recipeform.en, icon: 'form', affix: true }
     }, {
       path: 'statussetting',
       component: () => import('@/views/EquipmentManager/StatusSetting'),
@@ -1187,18 +1178,18 @@ export const dynamicRoutes = [
         path: 'plan',
         component: () => import('@/views/QualityManager/plan'),
         name: 'PlanManage',
-        meta: { title: routerTitle.quality.plan?.[language] || routerTitle.quality.plan?.en, icon: 'form', affix: true }
+        meta: { title: routerTitle.quality.plan?.[language] || routerTitle.quality.plan.en, icon: 'form', affix: true }
       }, {
         path: 'planparam',
         component: () => import('@/views/QualityManager/plan/subpage/detail'),
         name: 'PlanParamManage',
         hidden: true,
-        meta: { title: routerTitle.quality.planform?.[language] || routerTitle.quality.planform?.en, icon: 'form', affix: true }
+        meta: { title: routerTitle.quality.planform?.[language] || routerTitle.quality.planform.en, icon: 'form', affix: true }
       }, {
         path: 'issue',
         component: () => import('@/views/QualityManager/plan/issuedplan'),
         name: 'PlanIssuedManage',
-        meta: { title: routerTitle.quality.issuedplan?.[language] || routerTitle.quality.issuedplan?.en, icon: 'form', affix: true }
+        meta: { title: routerTitle.quality.issuedplan?.[language] || routerTitle.quality.issuedplan.en, icon: 'form', affix: true }
       },
       {
         path: '/offlineDetec',
@@ -1296,49 +1287,23 @@ export const dynamicRoutes = [
   },
   { path: '*', redirect: '/404', hidden: true }
 ]
+export const asyncRoutes = [
+  // 404 page must be placed at the end !!!
+  { path: '*', redirect: '/404', hidden: true }
+]
 
-const createRouter = (r) =>
+const createRouter = () =>
   new Router({
     // mode: 'history', // require service support
     scrollBehavior: () => ({ y: 0 }),
-    routes: r
+    routes: constantRoutes
   })
 
-const router = createRouter(constantRoutes)
-
-router.beforeEach((to, from, next) => {
-  console.log('route.index', to, to.meta.routeIndex)
-  if (to.meta.routeIndex >= 0) {
-    store.dispatch('app/setChoicepart', to.meta.routeIndex)
-  }
-  // 拦截器
-  console.log('route,to.path:', to.path)
-  if (to.meta.requireToken) {
-    if (Cookies.get('Admin-Token')) {
-      if (to.meta.required) {
-        if (Cookies.get('choicepart') === 'undefined' || !Cookies.get('choicepart')) {
-          next({
-            path: '/'
-          })
-        } else {
-          next()
-        }
-      } else {
-        next()
-      }
-    } else {
-      next({
-        path: '/login'
-      })
-    }
-  } else {
-    next()
-  }
-})
+const router = createRouter()
 
 // Detail see: https://github.com/vuejs/vue-router/issues/1234#issuecomment-357941465
 export function resetRouter() {
-  const newRouter = createRouter(constantRoutes)
+  const newRouter = createRouter()
   router.matcher = newRouter.matcher // reset router
 }
 

src/store/getters.js

@@ -1,7 +1,7 @@
 /*
  * @Date: 2020-12-14 09:07:03
- * @LastEditors: gtz
- * @LastEditTime: 2021-01-29 14:24:20
+ * @LastEditors: zwq
+ * @LastEditTime: 2021-12-22 11:38:46
  * @FilePath: \basic-admin\src\store\getters.js
  * @Description:
  */
@@ -19,7 +19,6 @@ const getters = {
   introduction: state => state.user.introduction,
   username: state => state.user.username,
   roles: state => state.user.roles,
-  menus: state => state.user.menus,
   permission_routes: state => state.permission.routes
 }
 export default getters

src/store/modules/permission.js

@@ -1,36 +1,38 @@
-import { dynamicRoutes, constantRoutes } from '@/router'
-import { getListItems } from '@/utils/tree'
+/*
+ * @Author: zwq
+ * @Date: 2021-09-18 16:09:08
+ * @LastEditors: zwq
+ * @LastEditTime: 2021-12-22 15:43:46
+ * @Description:
+ */
+import { asyncRoutes, constantRoutes } from '@/router'
 
 /**
  * Use meta.role to determine if the current user has permission
  * @param roles
  * @param route
  */
-function hasPermission(menuList, route) {
-  // if (route.meta && route.meta.roles) {
-  //   return roles.some(role => route.meta.roles.includes(role))
-  // } else {
-  //   return true
-  // }
-  if (route.hidden) {
+function hasPermission(roles, route) {
+  if (route.meta && route.meta.roles) {
+    return roles.some(role => route.meta.roles.includes(role))
+  } else {
     return true
   }
-  return menuList.some(m => route.path === m.h)
 }
 
 /**
  * Filter asynchronous routing tables by recursion
  * @param routes asyncRoutes
- * @param menuList
+ * @param roles
  */
-export function filterAsyncRoutes(routes, menuList) {
+export function filterAsyncRoutes(routes, roles) {
   const res = []
 
   routes.forEach(route => {
     const tmp = { ...route }
-    if (hasPermission(menuList, tmp)) {
+    if (hasPermission(roles, tmp)) {
       if (tmp.children) {
-        tmp.children = filterAsyncRoutes(tmp.children, menuList)
+        tmp.children = filterAsyncRoutes(tmp.children, roles)
       }
       res.push(tmp)
     }
@@ -52,19 +54,13 @@ const mutations = {
 }
 
 const actions = {
-  generateRoutes({ commit }, menus) {
+  generateRoutes({ commit }, roles) {
     return new Promise(resolve => {
       let accessedRoutes
-      console.log(menus)
-      if (!menus) {
-        accessedRoutes = []
+      if (roles.includes('admin')) {
+        accessedRoutes = asyncRoutes || []
       } else {
-        const menuList = []
-        getListItems(menus, node => menuList.push(node))
-        console.log(menuList)
-        console.log(dynamicRoutes)
-        accessedRoutes = filterAsyncRoutes(dynamicRoutes, menuList)
-        console.log(accessedRoutes)
+        accessedRoutes = filterAsyncRoutes(asyncRoutes, roles)
       }
       commit('SET_ROUTES', accessedRoutes)
       resolve(accessedRoutes)

src/store/modules/user.js

@@ -9,7 +9,6 @@ const state = {
   avatar: '',
   introduction: '',
   roles: [],
-  menus: [],
   username: Cookies.get('username') || ''
 }
 
@@ -26,9 +25,6 @@ const mutations = {
   SET_AVATAR: (state, avatar) => {
     state.avatar = avatar
   },
-  SET_MENUS: (state, menus) => {
-    state.menus = menus
-  },
   SET_ROLES: (state, roles) => {
     state.roles = roles
   },
@@ -70,17 +66,16 @@ const actions = {
           reject('Verification failed, please Login again.')
         }
 
-        const { roles, name, avatar, introduction, menus } = data
+        const { roles, name, avatar, introduction } = data
 
         // roles must be a non-empty array
-        if (!menus || menus.length <= 0) {
-          reject('getInfo: menus must be a non-null array!')
+        if (!roles || roles.length <= 0) {
+          reject('getInfo: roles must be a non-null array!')
         }
 
-        commit('SET_ROLES', roles || [])
+        commit('SET_ROLES', roles)
         commit('SET_NAME', name)
         commit('SET_AVATAR', avatar)
-        commit('SET_MENUS', menus)
         commit('SET_INTRODUCTION', introduction)
         resolve(data)
       }).catch(error => {
@@ -97,7 +92,6 @@ const actions = {
       dispatch('tagsView/delAllViews', null, { root: true })
       commit('SET_TOKEN', '')
       commit('SET_ROLES', [])
-      commit('SET_MENUS', [])
       commit('SET_USERNAME')
       Cookies.set('username', '')
       localStorage.clear()