From 12810075f9d9ce1eeaf490f51c12e15024bb496b Mon Sep 17 00:00:00 2001 From: Imbus <> Date: Wed, 3 Apr 2024 17:44:50 +0200 Subject: [PATCH] Logic error in getAllWeeklyReports fixed --- backend/internal/handlers/reports/GetAllWeeklyReports.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/backend/internal/handlers/reports/GetAllWeeklyReports.go b/backend/internal/handlers/reports/GetAllWeeklyReports.go index 825c0cc..ee81c82 100644 --- a/backend/internal/handlers/reports/GetAllWeeklyReports.go +++ b/backend/internal/handlers/reports/GetAllWeeklyReports.go @@ -31,7 +31,7 @@ func GetAllWeeklyReports(c *fiber.Ctx) error { return c.Status(400).SendString("Missing project name") } - // If the token user is not an admin, check if the target user is the same as the token user + // If the user is not a project manager, they can only view their own reports pm, err := db.GetDb(c).IsProjectManager(username, projectName) if err != nil { log.Info("Error checking if user is project manager:", err) @@ -44,9 +44,9 @@ func GetAllWeeklyReports(c *fiber.Ctx) error { } // Retrieve weekly reports for the user in the project from the database - reports, err := db.GetDb(c).GetAllWeeklyReports(username, projectName) + reports, err := db.GetDb(c).GetAllWeeklyReports(target_user, projectName) if err != nil { - log.Error("Error getting weekly reports for user:", username, "in project:", projectName, ":", err) + log.Error("Error getting weekly reports for user:", target_user, "in project:", projectName, ":", err) return c.Status(500).SendString(err.Error()) }