From 4858f7c82f82725cfa561fc89a35f6f31404e67c Mon Sep 17 00:00:00 2001
From: kolaente <k@knt.li>
Date: Mon, 20 Jan 2025 12:25:38 +0100
Subject: [PATCH] fix(project): transfer ownership after deleting a user

This fixes a bug where the ownership of a project was not transferred when the user was deleted, leading to errors when viewing the project, as the owner user could not be found.

Resolves https://kolaente.dev/vikunja/vikunja/issues/2827
---
 pkg/models/user_delete.go | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/pkg/models/user_delete.go b/pkg/models/user_delete.go
index 5f8a6d88d..56bb1983e 100644
--- a/pkg/models/user_delete.go
+++ b/pkg/models/user_delete.go
@@ -192,6 +192,17 @@ func ensureProjectAdminUser(s *xorm.Session, l *Project) (hadUsers bool, err err
 	_, err = s.Where("id = ?", firstUser.ID).
 		Cols("right").
 		Update(firstUser)
+	if err != nil {
+		return true, err
+	}
+
+	_, err = s.Where("id = ?", l.ID).
+		Cols("owner_id").
+		Update(&Project{OwnerID: firstUser.UserID})
+	if err != nil {
+		return true, err
+	}
+
 	return true, err
 }