Fix webhook payload issue by moving event dispatch after project reload

Co-authored-by: kolaente <13721712+kolaente@users.noreply.github.com>
Initial plan
2025-09-17 14:57:58 +00:00 · 2025-09-17 14:49:46 +00:00
1 changed files with 9 additions and 9 deletions
--- a/pkg/models/project.go
+++ b/pkg/models/project.go
@ -1013,14 +1013,6 @@ func UpdateProject(s *xorm.Session, project *Project, auth web.Auth, updateProje
 		return err
 	}

-	err = events.Dispatch(&ProjectUpdatedEvent{
-		Project: project,
-		Doer:    auth,
-	})
-	if err != nil {
-		return err
-	}
-
 	l, err := GetProjectSimpleByID(s, project.ID)
 	if err != nil {
 		return err
@ -1028,7 +1020,15 @@ func UpdateProject(s *xorm.Session, project *Project, auth web.Auth, updateProje

 	*project = *l
 	err = project.ReadOne(s, auth)
-	return
+	if err != nil {
+		return err
+	}
+
+	err = events.Dispatch(&ProjectUpdatedEvent{
+		Project: project,
+		Doer:    auth,
+	})
+	return err
 }

 func recalculateProjectPositions(s *xorm.Session, parentProjectID int64) (err error) {
Author	SHA1	Message	Date
copilot-swe-agent[bot]	480e164f39	Fix webhook payload issue by moving event dispatch after project reload Co-authored-by: kolaente <13721712+kolaente@users.noreply.github.com>	2025-09-17 14:57:58 +00:00
copilot-swe-agent[bot]	37415d4e4d	Initial plan	2025-09-17 14:49:46 +00:00