Delete calendar subscriptions as well when deleting user

Closes #28418

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

1 files changed, 13 insertions, 2 deletions

diff --git a/apps/dav/lib/HookManager.php b/apps/dav/lib/HookManager.php
index 03e7551e971..1287104fd10 100644
--- a/apps/dav/lib/HookManager.php
+++ b/apps/dav/lib/HookManager.php
@@ -58,6 +58,9 @@ class HookManager {
 	private $calendarsToDelete = [];
 
 	/** @var array */
+	private $subscriptionsToDelete = [];
+
+	/** @var array */
 	private $addressBooksToDelete = [];
 
 	/** @var Defaults */
@@ -116,9 +119,11 @@ class HookManager {
 
 	public function preDeleteUser($params) {
 		$uid = $params['uid'];
+		$userPrincipalUri = 'principals/users/' . $uid;
 		$this->usersToDelete[$uid] = $this->userManager->get($uid);
-		$this->calendarsToDelete = $this->calDav->getUsersOwnCalendars('principals/users/' . $uid);
-		$this->addressBooksToDelete = $this->cardDav->getUsersOwnAddressBooks('principals/users/' . $uid);
+		$this->calendarsToDelete = $this->calDav->getUsersOwnCalendars($userPrincipalUri);
+		$this->subscriptionsToDelete = $this->calDav->getSubscriptionsForUser($userPrincipalUri);
+		$this->addressBooksToDelete = $this->cardDav->getUsersOwnAddressBooks($userPrincipalUri);
 	}
 
 	public function preUnassignedUserId($uid) {
@@ -137,6 +142,12 @@ class HookManager {
 				true // Make sure the data doesn't go into the trashbin, a new user with the same UID would later see it otherwise
 			);
 		}
+
+		foreach ($this->subscriptionsToDelete as $subscription) {
+			$this->calDav->deleteSubscription(
+				$subscription['id'],
+			);
+		}
 		$this->calDav->deleteAllSharesByUser('principals/users/' . $uid);
 
 		foreach ($this->addressBooksToDelete as $addressBook) {