Fix user course progress log links

vatger · May 20, 2024 · 7ee71a8 · 7ee71a8
1 parent a8ff671
commit 7ee71a8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/backend/src/controllers/user-course-progress/UserCourseProgressAdministrationController.ts b/backend/src/controllers/user-course-progress/UserCourseProgressAdministrationController.ts
@@ -79,7 +79,7 @@ async function getInformation(request: Request, response: Response, next: NextFu
                 },
                 {
                     association: User.associations.training_sessions,
-                    attributes: ["uuid", "completed", "date"],
+                    attributes: ["id", "uuid", "completed", "date"],
                     where: {
                         course_id: course_id,
                     },
@@ -101,7 +101,7 @@ async function getInformation(request: Request, response: Response, next: NextFu
                     association: User.associations.training_logs,
                     attributes: ["uuid"],
                     through: {
-                        attributes: ["id", "user_id", "passed"],
+                        attributes: ["id", "training_session_id", "user_id", "passed"],
                     },
                 },
             ],

diff --git a/backend/src/models/extensions/TrainingSessionExtensions.ts b/backend/src/models/extensions/TrainingSessionExtensions.ts
@@ -32,28 +32,21 @@ function userCanCreateLogs(this: TrainingSession, user: User): boolean {
  * (for passing the training on)
  */
 async function getAvailableMentorGroups(this: TrainingSession): Promise<MentorGroup[]> {
+    // TODO: Test if we can limit the attributes, without breaking the includes.
+
     const trainingSession = await TrainingSession.findOne({
         where: {
             uuid: this.uuid,
         },
         include: [
             {
                 association: TrainingSession.associations.course,
-                through: {
-                    attributes: [],
-                },
                 include: [
                     {
                         association: Course.associations.mentor_groups,
-                        through: {
-                            attributes: [],
-                        },
                         include: [
                             {
                                 association: MentorGroup.associations.users,
-                                through: {
-                                    attributes: [],
-                                },
                             },
                         ],
                     },