Imbus/TTime
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Logic error in getAllWeeklyReports fixed

Browse source
This commit is contained in:
Imbus 2024-04-03 17:44:50 +02:00 • committed by Peter KW
parent 7313a27b31
commit 620073f688
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
backend/internal/handlers/reports/GetAllWeeklyReports.go
View file

@ -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())
}