From 5215833fe4b661c46dfed88eb29754011dbb899f Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Fri, 23 Sep 2016 14:30:24 +0200
Subject: [PATCH] delete shares to an user

Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
---
 apps/dav/lib/CalDAV/CalDavBackend.php       | 10 ++++++++++
 apps/dav/lib/DAV/Sharing/Backend.php        |  8 ++++++++
 apps/dav/lib/HookManager.php                |  1 +
 apps/dav/tests/unit/DAV/HookManagerTest.php |  1 +
 4 files changed, 20 insertions(+)

diff --git a/apps/dav/lib/CalDAV/CalDavBackend.php b/apps/dav/lib/CalDAV/CalDavBackend.php
index 17479c490a..367b239a67 100644
--- a/apps/dav/lib/CalDAV/CalDavBackend.php
+++ b/apps/dav/lib/CalDAV/CalDavBackend.php
@@ -507,6 +507,16 @@ class CalDavBackend extends AbstractBackend implements SyncSupport, Subscription
 		$this->sharingBackend->deleteAllShares($calendarId);
 	}
 
+	/**
+	 * Delete all of an user's shares
+	 *
+	 * @param string $principaluri
+	 * @return void
+	 */
+	function deleteAllSharesForUser($principaluri) {
+		$this->sharingBackend->deleteAllSharesByUser($principaluri);
+	}
+
 	/**
 	 * Returns all calendar objects within a calendar.
 	 *
diff --git a/apps/dav/lib/DAV/Sharing/Backend.php b/apps/dav/lib/DAV/Sharing/Backend.php
index 006fe334ba..f662d8e1b8 100644
--- a/apps/dav/lib/DAV/Sharing/Backend.php
+++ b/apps/dav/lib/DAV/Sharing/Backend.php
@@ -110,6 +110,14 @@ class Backend {
 			->execute();
 	}
 
+	public function deleteAllSharesByUser($principaluri) {
+		$query = $this->db->getQueryBuilder();
+		$query->delete('dav_shares')
+			->where($query->expr()->eq('principaluri', $query->createNamedParameter($principaluri)))
+			->andWhere($query->expr()->eq('type', $query->createNamedParameter($this->resourceType)))
+			->execute();
+	}
+
 	/**
 	 * @param IShareable $shareable
 	 * @param string $element
diff --git a/apps/dav/lib/HookManager.php b/apps/dav/lib/HookManager.php
index 03728eaf59..1680a90775 100644
--- a/apps/dav/lib/HookManager.php
+++ b/apps/dav/lib/HookManager.php
@@ -98,6 +98,7 @@ class HookManager {
 		foreach ($calendarsToDelete as $calendar) {
 			$this->calDav->deleteCalendar($calendar['id']);
 		}
+		$this->calDav->deleteAllSharesForUser('principals/users/' . $uid);
 
 		$addressBooksToDelete = $this->cardDav->getAddressBooksForUser('principals/users/' . $uid);
 		foreach ($addressBooksToDelete as $addressBook) {
diff --git a/apps/dav/tests/unit/DAV/HookManagerTest.php b/apps/dav/tests/unit/DAV/HookManagerTest.php
index 05384813f0..cde24214dd 100644
--- a/apps/dav/tests/unit/DAV/HookManagerTest.php
+++ b/apps/dav/tests/unit/DAV/HookManagerTest.php
@@ -172,6 +172,7 @@ class HookManagerTest extends TestCase {
 			['id' => 'personal']
 		]);
 		$cal->expects($this->once())->method('deleteCalendar');
+		$cal->expects($this->once())->method('deleteAllSharesForUser');
 
 		/** @var CardDavBackend | \PHPUnit_Framework_MockObject_MockObject $card */
 		$card = $this->getMockBuilder(CardDavBackend::class)