fix missing permissions

fix missing permissions in menu & prevent authentification error if not logged

src/store/index.js

@@ -3,7 +3,6 @@ const axios = require("axios");
 import Vue from 'vue';
 import Vuex from 'vuex';
 import router from '../router'
-//import modules from './modules';
 import feature_type from "./modules/feature_type"
 import feature from "./modules/feature"
 import map from "./modules/map"
@@ -100,7 +99,7 @@ export default new Vuex.Store({
     project: state => state.projects.find((project) => project.slug === state.project_slug),
     permissions: state => state.user_permissions ? state.user_permissions[state.project_slug] : noPermissions,
     project_types: state => state.projects.filter(projet => projet.is_project_type),
-    project_user: state => state.projects.filter(projet => projet.creator === state.user.id), // todo: add id to user in api
+    project_user: state => state.projects.filter(projet => projet.creator === state.user.id),
   },
 
   actions: {
@@ -121,7 +120,7 @@ export default new Vuex.Store({
           throw error;
         });
     },
-     GET_STATIC_PAGES({ commit }) {
+    GET_STATIC_PAGES({ commit }) {
       return axios
         .get(`${this.state.configuration.VUE_APP_DJANGO_API_BASE}flat-pages/`)
         .then((response) => (commit("SET_STATIC_PAGES", response.data)))
@@ -162,29 +161,32 @@ export default new Vuex.Store({
       }
     },
 
-    USER_INFO({ commit }) {
-      return axios
-        .get(`${this.state.configuration.VUE_APP_DJANGO_API_BASE}user_info/`)
-        .then((response) => {
-          if (response && response.status === 200) {
-            const user = response.data.user;
-            commit("SET_USER", user);
-            window.localStorage.setItem("user", JSON.stringify(user)); //? toujours nécessaire ?
-          }
-        })
-        .catch(() => {
-          router.push({ name: "login" });
-        });
+    USER_INFO({ state, commit }) {
+      if (state.user) {
+        axios
+          .get(`${this.state.configuration.VUE_APP_DJANGO_API_BASE}user_info/`)
+          .then((response) => {
+            if (response && response.status === 200) {
+              const user = response.data.user;
+              commit("SET_USER", user);
+              window.localStorage.setItem("user", JSON.stringify(user)); //? toujours nécessaire ?
+            }
+          })
+          .catch(() => {
+            router.push({ name: "login" });
+          });
+      }
     },
 
-    LOGOUT({ commit }) { // ? logout se fait bien dans django ?
+    LOGOUT({ commit }) {
       axios
         .get(`${this.state.configuration.VUE_APP_DJANGO_API_BASE}logout/`)
         .then((response) => {
           if (response && response.status === 200) {
-            commit("SET_USER", false); // ? better false or null
+            commit("SET_USER", false);
             commit("SET_USER_LEVEL_PROJECTS", null);
             commit("SET_USER_PERMISSIONS", null);
+            router.push("/");
           }
         })
         .catch((error) => {
@@ -204,6 +206,7 @@ export default new Vuex.Store({
           throw error;
         });
     },
+
     GET_USER_LEVEL_PROJECTS({ commit }) {
       return axios
         .get(`${this.state.configuration.VUE_APP_DJANGO_API_BASE}user-level-projects/`)
@@ -230,15 +233,12 @@ export default new Vuex.Store({
         });
     },
 
-    GET_PROJECT_INFO({ commit, dispatch }, slug) {
+    GET_PROJECT_INFO({ state, commit, dispatch }, slug) {
       commit("SET_PROJECT_SLUG", slug);
-      return Promise.all([
-        dispatch("GET_PROJECT_LAST_MESSAGES", slug),
-        dispatch("feature_type/GET_PROJECT_FEATURE_TYPES", slug),
-        dispatch("feature/GET_PROJECT_FEATURES", slug),
-        dispatch("map/GET_BASEMAPS", slug),
-      ]);
-    
+      dispatch("GET_PROJECT_LAST_MESSAGES", slug);
+      dispatch("feature_type/GET_PROJECT_FEATURE_TYPES", slug);
+      dispatch("feature/GET_PROJECT_FEATURES", slug);
+      if (state.user) dispatch("map/GET_BASEMAPS", slug);
     },
 
     GET_PROJECT_LAST_MESSAGES({ commit }, project_slug) {
@@ -253,52 +253,6 @@ export default new Vuex.Store({
           throw error;
         });
     },
-
-    /*  GET_PROJECT_USER({ commit }, project_slug) {
-       axios
-         .get(`${DJANGO_API_BASE}projects/${project_slug}/utilisateurs`)
-         .then((response) => (commit("SET_PROJECT_MEMBERS", response.data.members)))
-         .catch((error) => {
-           throw error;
-         });
-     }, */
-    /* GET_PROJECT_FEATURES({ commit }, project_slug) {
-      axios
-        .get(`${DJANGO_API_BASE}projects/${project_slug}/feature`)
-        .then((response) => commit("feature/SET_FEATURES", response.data.features))
-        .catch((error) => {
-          throw error;
-        });
-    },
-    */
-    /* GET_PROJECT({ commit }, project_slug) {
-      axios
-        .get(`${DJANGO_API_BASE}projects/${project_slug}/project`)
-        .then((response) => commit("SET_PROJECT", response.data))
-        .catch((error) => {
-          throw error;
-        });
-    },
-    */
-    /* GET_COOKIE({ commit }, name) {
-      let cookieValue = null;
-
-      if (document.cookie && document.cookie !== "") {
-        const cookies = document.cookie.split(";");
-        for (let i = 0; i < cookies.length; i++) {
-          const cookie = cookies[i].trim();
-
-          // Does this cookie string begin with the name we want?
-          if (cookie.substring(0, name.length + 1) === name + "=") {
-            cookieValue = decodeURIComponent(cookie.substring(name.length + 1));
-
-            break;
-          }
-        }
-      }
-      commit("SET_COOKIE", cookieValue);
-    }, */
-
   }
 
 });