fixing the error that exercises and subjects have been showing even t…

…he membership was not there
ooemperor · Feb 18, 2024 · 8f69a21 · 8f69a21
1 parent 03834aa
commit 8f69a21
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/codeGrader/frontend/user/handlers/Exercise.py b/codeGrader/frontend/user/handlers/Exercise.py
@@ -48,8 +48,9 @@ def get(self) -> str:
         @return: The rendered template
         """
         exercises = self.api.get("/exercises")
+        temp_exercises = exercises["exercise"].copy()
 
-        for ex in exercises["exercise"]:
+        for ex in temp_exercises:
             # filtering only the exercises that are allowed by the memberships
             if not self.user.check_permission(subject_id=ex["subject_id"]):
                 exercises["exercise"].remove(ex)

diff --git a/codeGrader/frontend/user/handlers/Subject.py b/codeGrader/frontend/user/handlers/Subject.py
@@ -48,9 +48,9 @@ def get(self) -> str:
         @return: The rendered template
         """
         subjects = self.api.get("/subjects")
+        temp_subjects = subjects["subject"].copy()
 
-
-        for sub in subjects["subject"]:
+        for sub in temp_subjects:
             # filtering only the subjects that are allowed by the memberships
             if not self.user.check_permission(subject_id=sub["id"]):
                 subjects["subject"].remove(sub)