From 3f56444033a0400862ed5cd2f1d27d5ebd3809e4 Mon Sep 17 00:00:00 2001
From: bsilkyn <brent.silkyn@ugent.be>
Date: Sat, 4 May 2024 15:10:54 +0200
Subject: [PATCH] chore: csv download button implemented #400

---
 .../src/components/projects/DownloadCSV.vue   | 61 ++++++++++---------
 .../projects/roles/TeacherProjectView.vue     |  2 +-
 2 files changed, 33 insertions(+), 30 deletions(-)

diff --git a/frontend/src/components/projects/DownloadCSV.vue b/frontend/src/components/projects/DownloadCSV.vue
index 07e5cea2..2220f160 100644
--- a/frontend/src/components/projects/DownloadCSV.vue
+++ b/frontend/src/components/projects/DownloadCSV.vue
@@ -1,22 +1,19 @@
 <script setup lang="ts">
 import Button from 'primevue/button';
-import { ref } from "vue";
 import { useI18n } from 'vue-i18n';
 import { useGroup } from '@/composables/services/group.service.ts';
-
+import { useStudents } from '@/composables/services/student.service.ts';
 
 /* Props */
 const props = defineProps<{
-    projectId: string
-    projectName: string
+    projectId: string;
+    projectName: string;
 }>();
 
 /* Injections */
 const { t } = useI18n();
 const { groups, getGroupsByProject } = useGroup();
-
-/* State */
-const csv_content = ref<string|undefined>(undefined);
+const { students, getStudentsByGroup } = useStudents();
 
 /* Functions */
 /**
@@ -24,37 +21,43 @@ const csv_content = ref<string|undefined>(undefined);
  * with the projectId in this component's props.
  * After generating a csv, a download link is created and clicked.
  */
-const generateCSVAndDownload = async () => {
+const generateCSVAndDownload = async (): Promise<void> => {
     // retrieve all the groups associated with a given project
     await getGroupsByProject(props.projectId);
     // construct for every group's student a csv line according to ufora grade csv standard
     // and concatenate them all into one csv
-    csv_content.value = groups.value?.map(group => {
-        console.log(group);
-        return group.students?.map(student => {
-            console.log(student);
-            return `#${student.studentId},${student.last_name},${student.first_name},${student.email},${group.score},#`
-        }).join('\n');
-    }).join('\n');
+    const csvPromises =
+        groups.value?.map(async (group) => {
+            await getStudentsByGroup(group.id);
+            return (
+                students.value
+                    ?.map((student) => {
+                        // single csv line
+                        return `#${student.id},${student.last_name},${student.first_name},${student.email},${group.score},#`;
+                    })
+                    .join('\n') ?? ''
+            );
+        }) ?? [];
+
+    const csvList = await Promise.all(csvPromises);
+    const csvContent = csvList.join('\n');
 
-    if (csv_content.value !== undefined) {
-        // create a blob from the csv content
-        const blob = new Blob([csv_content.value], {type: 'text/plain'})
+    // create a blob from the csv content
+    const blob = new Blob([csvContent], { type: 'text/plain' });
 
-        // create a download url for this blob
-        const url = URL.createObjectURL(blob);
+    // create a download url for this blob
+    const url = URL.createObjectURL(blob);
 
-        // create an anchor element for downloading the file
-        const a = document.createElement('a');
-        a.href = url;
-        a.download = (props.projectName ?? props.projectId) + '.scores';
+    // create an anchor element for downloading the file
+    const a = document.createElement('a');
+    a.href = url;
+    a.download = (props.projectName ?? props.projectId) + '.scores';
 
-        // click anchor element
-        a.click();
+    // click anchor element
+    a.click();
 
-        // clean up URL
-        URL.revokeObjectURL(url);
-    }
+    // clean up URL
+    URL.revokeObjectURL(url);
 };
 </script>
 
diff --git a/frontend/src/views/projects/roles/TeacherProjectView.vue b/frontend/src/views/projects/roles/TeacherProjectView.vue
index 08c3ce77..5e1081cc 100644
--- a/frontend/src/views/projects/roles/TeacherProjectView.vue
+++ b/frontend/src/views/projects/roles/TeacherProjectView.vue
@@ -57,7 +57,7 @@ watch(
         </div>
         <div class="col-12">
             <template v-if="project !== null">
-                <DownloadCSV :project-id="project.id" :project-name="project.name"/>
+                <DownloadCSV :project-id="project.id" :project-name="project.name" />
             </template>
         </div>
     </div>