permissions for groups overview

SELab-2 · May 20, 2024 · ceb7f4c · ceb7f4c
1 parent 9f997b7
commit ceb7f4c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 17 deletions.
diff --git a/frontend/src/router/index.ts b/frontend/src/router/index.ts
@@ -9,12 +9,10 @@ import useCanVisit, {
     useIsInstructorOfSubjectCondition,
     useIsStudentOfProjectCondition,
     useIsInstructorOfProjectCondition,
-    useIsPartOfSubjectCondition,
     useAndCondition,
     useOrCondition,
     useIsInGroupOfProjectCondition,
 } from "./middleware/canVisit";
-import { ref } from "vue";
 
 declare module "vue-router" {
     interface RouteMeta {
@@ -87,6 +85,14 @@ const router = createRouter({
             name: "groups",
             component: () => import("../views/GroupsView.vue"),
             props: (route) => ({ projectId: Number(route.params.projectId) }),
+            meta: {
+                middleware: useCanVisit(
+                    useOrCondition(
+                        useIsStudentOfProjectCondition,
+                        useIsInstructorOfProjectCondition
+                    )
+                ),
+            },
         },
         {
             path: "/groups/:groupId(\\d+)",

diff --git a/frontend/src/router/middleware/canVisit.ts b/frontend/src/router/middleware/canVisit.ts
@@ -129,18 +129,3 @@ export const useIsInGroupOfProjectCondition: CanVisitCondition = (qc, ctx) => {
     });
     return { condition, isLoading };
 };
-
-export const useIsPartOfSubjectCondition: CanVisitCondition = (qc, ctx) => {
-    const subjectId = Number(ctx.to.params.subjectId);
-    const { data: subjects, isLoading } = useSubjectsQuery(qc);
-    const condition = computed(() => {
-        const student_subjects = subjects.value?.as_student || [];
-        const instructor_subjects = subjects.value?.as_instructor || [];
-        return (
-            [...student_subjects, ...instructor_subjects].findIndex(
-                (subject) => subject.id === subjectId
-            ) !== -1
-        );
-    });
-    return { condition, isLoading };
-};