From 0049ab272b7f30e7ac43a5fb3112181d4fefa762 Mon Sep 17 00:00:00 2001
From: Joao Ramos <joaomframos.1@gmail.com>
Date: Thu, 7 Dec 2023 23:07:35 +0000
Subject: [PATCH] Added dynamic generation of exams as an option

---
 src/dashboards/AssignmentCreator.tsx |  11 ++-
 src/pages/api/assignments/index.ts   | 100 +++++++++++++++++++--------
 src/utils/exams.be.ts                |   2 +-
 3 files changed, 82 insertions(+), 31 deletions(-)

diff --git a/src/dashboards/AssignmentCreator.tsx b/src/dashboards/AssignmentCreator.tsx
index e55faa60..fe1b9c68 100644
--- a/src/dashboards/AssignmentCreator.tsx
+++ b/src/dashboards/AssignmentCreator.tsx
@@ -18,6 +18,7 @@ import {getExam} from "@/utils/exams";
 import {toast} from "react-toastify";
 import {uuidv4} from "@firebase/util";
 import {Assignment} from "@/interfaces/results";
+import Checkbox from "@/components/Low/Checkbox";
 
 interface Props {
 	isCreating: boolean;
@@ -35,6 +36,8 @@ export default function AssignmentCreator({isCreating, assignment, assigner, gro
 	const [isLoading, setIsLoading] = useState(false);
 	const [startDate, setStartDate] = useState<Date | null>(assignment ? moment(assignment.startDate).toDate() : moment().add(1, "day").toDate());
 	const [endDate, setEndDate] = useState<Date | null>(assignment ? moment(assignment.endDate).toDate() : moment().add(8, "day").toDate());
+	// creates a new exam for each assignee or just one exam for all assignees
+	const [generateMultiple, setGenerateMultiple] = useState<boolean>(false);
 
 	const toggleModule = (module: Module) => {
 		const modules = selectedModules.filter((x) => x !== module);
@@ -55,7 +58,8 @@ export default function AssignmentCreator({isCreating, assignment, assigner, gro
         name,
         startDate,
         endDate,
-		selectedModules
+		selectedModules,
+		generateMultiple,
       }
     )
       .then(() => {
@@ -280,6 +284,11 @@ export default function AssignmentCreator({isCreating, assignment, assigner, gro
 						))}
 					</div>
 				</section>
+				<div className="flex gap-4 w-full justify-end">
+					<Checkbox isChecked={generateMultiple} onChange={() => setGenerateMultiple(d => !d)}>
+						Generate an unique exam for each student
+					</Checkbox>
+				</div>
 				<div className="flex gap-4 w-full justify-end">
 					<Button className="w-full max-w-[200px]" variant="outline" onClick={cancelCreation} disabled={isLoading} isLoading={isLoading}>
 						Cancel
diff --git a/src/pages/api/assignments/index.ts b/src/pages/api/assignments/index.ts
index 37ee47a1..f5a87acd 100644
--- a/src/pages/api/assignments/index.ts
+++ b/src/pages/api/assignments/index.ts
@@ -49,40 +49,82 @@ function getRandomIndex(arr: any[]): number {
   return randomIndex;
 }
 
+const generateExams = async (
+  generateMultiple: Boolean,
+  selectedModules: Module[],
+  assignees: string[]
+): Promise<ExamWithUser[]> => {
+  if (generateMultiple) {
+    // for optimization purposes, it would be better to create a new endpoint that returned the answers for all users at once
+    const allExams = await assignees.map(async (assignee) => {
+      const selectedModulePromises = await selectedModules.map(
+        async (module: Module) => {
+          try {
+            const exams: Exam[] = await getExams(db, module, "true", assignee);
+
+            const exam = exams[getRandomIndex(exams)];
+            if (exam) {
+              return { module: exam.module, id: exam.id, assignee };
+            }
+            return null;
+          } catch (e) {
+            console.error(e);
+            return null;
+          }
+        },
+        []
+      );
+      const newModules = await Promise.all(selectedModulePromises);
+
+      return newModules;
+    }, []);
+
+    const exams = flatten(await Promise.all(allExams)).filter(
+      (x) => x !== null
+    ) as ExamWithUser[];
+    return exams;
+  }
+
+  const selectedModulePromises = await selectedModules.map(
+    async (module: Module) => {
+      const exams: Exam[] = await getExams(db, module, "false", undefined);
+      const exam = exams[getRandomIndex(exams)];
+
+      if (exam) {
+        return { module: exam.module, id: exam.id };
+      }
+      return null;
+    }
+  );
+
+  const exams = await Promise.all(selectedModulePromises);
+  const examesFiltered = exams.filter((x) => x !== null) as ExamWithUser[];
+  return flatten(
+    assignees.map((assignee) =>
+      examesFiltered.map((exam) => ({ ...exam, assignee }))
+    )
+  );
+};
+
 async function POST(req: NextApiRequest, res: NextApiResponse) {
-  const { selectedModules, assignees, ...body } = req.body as {
+  const {
+    selectedModules,
+    assignees,
+    // Generarte multiple true would generate an unique exam for eacah user
+    // false would generate the same exam for all usersa
+    generateMultiple = false,
+    ...body
+  } = req.body as {
     selectedModules: Module[];
     assignees: string[];
+    generateMultiple: Boolean;
   };
 
-  // for optimization purposes, it would be better to create a new endpoint that returned the answers for all users at once
-  const allExams = await assignees.map(async (assignee) => {
-    const selectedModulePromises = await selectedModules.map(
-      async (module: Module) => {
-        try {
-          const exams: Exam[] = await getExams(db, module, "true", assignee);
-
-          const exam = exams[getRandomIndex(exams)];
-          if (exam) {
-            return { module: exam.module, id: exam.id, assignee };
-          }
-          return null;
-        } catch (e) {
-          console.error(e);
-          return null;
-        }
-      },
-      []
-    );
-    const newModules = await Promise.all(selectedModulePromises);
-
-    return newModules;
-  }, []);
-
-  const exams = flatten(await Promise.all(allExams)).filter(
-    (x) => x !== null
-  ) as ExamWithUser[];
-
+  const exams: ExamWithUser[] = await generateExams(
+    generateMultiple,
+    selectedModules,
+    assignees
+  );
   if (exams.length === 0) {
     res
       .status(400)
diff --git a/src/utils/exams.be.ts b/src/utils/exams.be.ts
index ee9832f3..78ac1914 100644
--- a/src/utils/exams.be.ts
+++ b/src/utils/exams.be.ts
@@ -18,7 +18,7 @@ export const getExams = async (
   // added userId as due to assignments being set from the teacher to the student
   // we need to make sure we are serving exams not executed by the user and not
   // by the teacher that performed the request
-  userId: string
+  userId: string | undefined
 ): Promise<Exam[]> => {
   const moduleRef = collection(db, module);