From fb426a6e22df80ec95173ce7b24a81cb6758464f Mon Sep 17 00:00:00 2001
From: kolaente <k@knt.li>
Date: Wed, 17 Sep 2025 17:04:26 +0200
Subject: [PATCH] fix(webhook): actually fetch project before enriching details

This fixes a bug where the project is fetched before adding more details
through ReadOne since ReadOne does not fetch the project. In the normal
project reading flow through the api, this is done in the permission
check.

Resolves https://github.com/go-vikunja/vikunja/issues/1498
---
 pkg/models/listeners.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/pkg/models/listeners.go b/pkg/models/listeners.go
index d1120894b..a9fe52c6a 100644
--- a/pkg/models/listeners.go
+++ b/pkg/models/listeners.go
@@ -906,7 +906,11 @@ func reloadEventData(s *xorm.Session, event map[string]interface{}, projectID in
 	}
 
 	if _, has := event["project"]; has && doerID != 0 {
-		project := &Project{ID: projectID}
+		var project *Project
+		project, err = GetProjectSimpleByID(s, projectID)
+		if err != nil && !IsErrProjectDoesNotExist(err) {
+			return
+		}
 		err = project.ReadOne(s, &user.User{ID: doerID})
 		if err != nil && !IsErrProjectDoesNotExist(err) {
 			return