fix #889biest-889

3 files changed, 34 insertions, 159 deletions

diff --git a/resources/vue/store/courseware/courseware-comments.module.js b/resources/vue/store/courseware/courseware-comments.module.js
index c65ee11..1afed9a 100644
--- a/resources/vue/store/courseware/courseware-comments.module.js
+++ b/resources/vue/store/courseware/courseware-comments.module.js
@@ -4,7 +4,6 @@ const getDefaultState = () => {
         httpClient: null,
         userId: null,
         userIsTeacher: false,
-        teacherStatusLoaded: false,
         typeFilter: 'all', // all, blocks, elements
         createdFilter: 'all', // all, oneDay, oneWeek
         unitFilter: 'all', // all or unit id
@@ -26,9 +25,6 @@ const getters = {
     userIsTeacher(state) {
         return state.userIsTeacher;
     },
-    teacherStatusLoaded(state) {
-        return state.teacherStatusLoaded;
-    },
     typeFilter(state) {
         return state.typeFilter;
     },
@@ -62,55 +58,9 @@ export const actions = {
     setUnitFilter({ commit }, id) {
         commit('setUnitFilter', id);
     },
-    // other actions
-    async loadTeacherStatus({ dispatch, rootGetters, state, commit, getters }, userId) {
-        const user = rootGetters['users/byId']({ id: userId });
-
-        if (user.attributes.permission === 'root') {
-            commit('setUserIsTeacher', true);
-            return;
-        }
-        if (user.attributes.permission === 'admin') {
-            await dispatch('courses/loadById', { id: state.context.id });
-            const course = rootGetters['courses/byId']({id: state.context.id });
-            const instituteId = course.relationships.institute.data.id;
-
-            const parent = { type: 'users', id: `${userId}` };
-            const relationship = 'institute-memberships';
-            const options = {};
-            await dispatch('institute-memberships/loadRelated', { parent, relationship, options }, { root: true });
-            const instituteMemberships = rootGetters['institute-memberships/all'];
-            const instituteMembership = instituteMemberships.filter(membership => membership.relationships.institute.data.id === instituteId);
-
-            if (instituteMembership.length > 0 && instituteMembership[0].attributes.permission === 'admin') {
-                commit('setUserIsTeacher', true);
-                return;
-            }
-        }
-
-        const membershipId = `${state.context.id}_${userId}`;
-        try {
-            await dispatch('course-memberships/loadById', { id: membershipId });
-        } catch (error) {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
-
-            return false;
-        }
-        const membership = rootGetters['course-memberships/byId']({ id: membershipId });
-        if (membership) {
-            const membershipPermission = membership.attributes.permission;
-            const isTeacher = membershipPermission === 'dozent' || membershipPermission === 'tutor';
-            commit('setUserIsTeacher', isTeacher);
-
-            return true;
-        } else {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
-
-            return false;
-        }
-    },
+    setUserIsTeacher({ commit }, isTeacher) {
+        commit('setUserIsTeacher', isTeacher);
+    }
 };
 
 export const mutations = {
@@ -133,7 +83,6 @@ export const mutations = {
         state.unitFilter = data;
     },
     setUserIsTeacher(state, isTeacher) {
-        state.teacherStatusLoaded = true;
         state.userIsTeacher = isTeacher;
     },
 };
@@ -143,4 +92,4 @@ export default {
     actions,
     mutations,
     getters,
-};
\ No newline at end of file
+};
diff --git a/resources/vue/store/courseware/courseware-shelf.module.js b/resources/vue/store/courseware/courseware-shelf.module.js
index d907ee4..783116a 100644
--- a/resources/vue/store/courseware/courseware-shelf.module.js
+++ b/resources/vue/store/courseware/courseware-shelf.module.js
@@ -1,6 +1,7 @@
 const getDefaultState = () => {
     return {
         context: null,
+        coursePerms: null,
         httpClient: null,
         showCompanionOverlay: false,
         msgCompanionOverlay: '',
@@ -87,6 +88,9 @@ const getters = {
     userIsTeacher(state) {
         return state.userIsTeacher;
     },
+    coursePerms(state) {
+        return state.coursePerms;
+    },
     urlHelper(state) {
         return state.urlHelper;
     },
@@ -131,6 +135,9 @@ export const actions = {
     setContext({ commit }, context) {
         commit('setContext', context);
     },
+    setCoursePerms({ commit }, perms) {
+        commit('setCoursePerms', perms);
+    },
     setHttpClient({ commit }, httpClient) {
         commit('setHttpClient', httpClient);
     },
@@ -546,52 +553,10 @@ export const actions = {
         return dispatch('courseware-instances/update', instance, { root: true });
     },
 
-    async loadTeacherStatus({ dispatch, rootGetters, state, commit, getters }, userId) {
-        const user = rootGetters['users/byId']({ id: userId });
-
-        if (user.attributes.permission === 'root') {
-            commit('setUserIsTeacher', true);
-            return;
-        }
-        if (user.attributes.permission === 'admin') {
-            await dispatch('courses/loadById', { id: state.context.id });
-            const course = rootGetters['courses/byId']({id: state.context.id });
-            const instituteId = course.relationships.institute.data.id;
-
-            const parent = { type: 'users', id: `${userId}` };
-            const relationship = 'institute-memberships';
-            const options = {};
-            await dispatch('institute-memberships/loadRelated', { parent, relationship, options }, { root: true });
-            const instituteMemberships = rootGetters['institute-memberships/all'];
-            const instituteMembership = instituteMemberships.filter(membership => membership.relationships.institute.data.id === instituteId);
-
-            if (instituteMembership.length > 0 && instituteMembership[0].attributes.permission === 'admin') {
-                commit('setUserIsTeacher', true);
-                return;
-            }
-        }
-
-        const membershipId = `${state.context.id}_${userId}`;
-        try {
-            await dispatch('course-memberships/loadById', { id: membershipId });
-        } catch (error) {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
+    loadTeacherStatus({ commit, getters }) {
+        const isTeacher = getters.coursePerms['tutor'];
 
-            return false;
-        }
-        const membership = rootGetters['course-memberships/byId']({ id: membershipId });
-        if (membership) {
-            const membershipPermission = membership.attributes.permission;
-            commit('setUserIsTeacher', membershipPermission === 'dozent' || membershipPermission === 'tutor');
-
-            return true;
-        } else {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
-
-            return false;
-        }
+        commit('setUserIsTeacher', isTeacher);
     },
 
     uploadImageForStructuralElement({ dispatch, state }, { structuralElement, file }) {
@@ -746,6 +711,9 @@ export const mutations = {
     setContext(state, data){
         state.context = data;
     },
+    setCoursePerms(state, perms) {
+        state.coursePerms = perms;
+    },
     setHttpClient(state, data){
         state.httpClient = data;
     },
@@ -789,7 +757,6 @@ export const mutations = {
         state.exportProgress = exportProgress;
     },
     setUserIsTeacher(state, isTeacher) {
-        state.teacherStatusLoaded = true;
         state.userIsTeacher = isTeacher;
     },
     setUrlHelper(state, urlHelper) {
diff --git a/resources/vue/store/courseware/courseware.module.js b/resources/vue/store/courseware/courseware.module.js
index 784b750..237f12f 100644
--- a/resources/vue/store/courseware/courseware.module.js
+++ b/resources/vue/store/courseware/courseware.module.js
@@ -22,8 +22,8 @@ const getDefaultState = () => {
         userId: null,
         viewMode: 'read',
         dashboardViewMode: 'default',
+        coursePerms: null,
         userIsTeacher: false,
-        teacherStatusLoaded: false,
 
         showStructuralElementEditDialog: false,
         showStructuralElementAddDialog: false,
@@ -184,8 +184,8 @@ const getters = {
     userIsTeacher(state) {
         return state.userIsTeacher;
     },
-    teacherStatusLoaded(state) {
-        return state.teacherStatusLoaded;
+    coursePerms(state) {
+        return state.coursePerms;
     },
     pluginManager(state) {
         return state.pluginManager;
@@ -1290,63 +1290,19 @@ export const actions = {
         );
     },
 
-    async loadTeacherStatus({ dispatch, rootGetters, state, commit, getters }, userId) {
-        const user = rootGetters['users/byId']({ id: userId });
-
-        if (user.attributes.permission === 'root') {
-            commit('setUserIsTeacher', true);
-            return;
-        }
-        if (user.attributes.permission === 'admin') {
-            await dispatch('courses/loadById', { id: state.context.id });
-            const course = rootGetters['courses/byId']({id: state.context.id });
-            const instituteId = course.relationships.institute.data.id;
-
-            const parent = { type: 'users', id: `${userId}` };
-            const relationship = 'institute-memberships';
-            const options = {};
-            await dispatch('institute-memberships/loadRelated', { parent, relationship, options }, { root: true });
-            const instituteMemberships = rootGetters['institute-memberships/all'];
-            const instituteMembership = instituteMemberships.filter(membership => membership.relationships.institute.data.id === instituteId);
-
-            if (instituteMembership.length > 0 && instituteMembership[0].attributes.permission === 'admin') {
-                commit('setUserIsTeacher', true);
-                return;
-            }
-        }
-
-        const membershipId = `${state.context.id}_${userId}`;
-        try {
-            await dispatch('course-memberships/loadById', { id: membershipId });
-        } catch (error) {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
-
-            return false;
-        }
-        const membership = rootGetters['course-memberships/byId']({ id: membershipId });
-        if (membership) {
-            let editingLevel = 'tutor';
-            if (getters.courseware.attributes) {
-                editingLevel = getters.courseware.attributes['editing-permission-level'];
-            }
-            const membershipPermission = membership.attributes.permission;
+    setCoursePerms({ commit }, perms) {
+        commit('setCoursePerms', perms);
+    },
 
-            let isTeacher = false;
-            if (editingLevel === 'dozent') {
-                isTeacher = membershipPermission === 'dozent';
-            } else if (editingLevel === 'tutor') {
-                isTeacher = membershipPermission === 'dozent' || membershipPermission === 'tutor';
-            }
-            commit('setUserIsTeacher', isTeacher);
+    setUserIsTeacher({ commit }, isTeacher) {
+        commit('setUserIsTeacher', isTeacher);
+    },
 
-            return true;
-        } else {
-            console.error(`Could not find course membership for ${membershipId}.`);
-            commit('setUserIsTeacher', false);
+    loadTeacherStatus({ commit, getters }) {
+        const editingLevel = getters.courseware.attributes['editing-permission-level'];
+        const isTeacher = getters.coursePerms[editingLevel];
 
-            return false;
-        }
+        commit('setUserIsTeacher', isTeacher);
     },
 
     loadFeedback({ dispatch }, blockId) {
@@ -1623,10 +1579,13 @@ export const mutations = {
     },
 
     setUserIsTeacher(state, isTeacher) {
-        state.teacherStatusLoaded = true;
         state.userIsTeacher = isTeacher;
     },
 
+    setCoursePerms(state, perms) {
+        state.coursePerms = perms;
+    },
+
     setPluginManager(state, pluginManager) {
         state.pluginManager = pluginManager;
     },