fix: Fix handling of deleting share from self

If a share is deleted from self and it is not a group share it was
handled as a user share. However, other types of shares can be deleted
from self (for example, circle or room shares), so user shares need to
be explicitly checked.

Note that as those other share types are not explicitly handled no entry
is added to the activity for them, but that was already the case before
(this change just prevents an exception to be thrown).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>

Author: danxuliu

lib/FilesHooks.php

@@ -826,7 +826,7 @@ public function unShareSelf(IShare $share) {
 		if (in_array($share->getNodeType(), ['file', 'folder'], true)) {
 			if ($share->getShareType() === IShare::TYPE_GROUP) {
 				$this->unshareFromSelfGroup($share);
-			} else {
+			} elseif ($share->getShareType() === IShare::TYPE_USER) {
 				$this->unshareFromUser($share);
 			}
 		}

tests/FilesHooksTest.php

@@ -962,6 +962,8 @@ public function testLeaveShare(): void {
 			->willReturn('file');
 		$share->method('getSharedWith')
 			->willReturn('with');
+		$share->method('getShareType')
+			->willReturn(IShare::TYPE_USER);
 
 		$this->settings->expects($this->exactly(3))
 			->method('getUserSetting')