refactor: use IUser for IShareSourceType::getSourceInteractionResource instead of user id string

Signed-off-by: Robin Appelman <robin@icewind.nl>

Author: icewind1991

apps/files/lib/Sharing/Source/NodeShareSourceType.php

@@ -27,6 +27,7 @@
 use OCP\Interaction\InteractionResource;
 use OCP\Interaction\Resources\NodeResource;
 use OCP\IURLGenerator;
+use OCP\IUser;
 use OCP\L10N\IFactory;
 
 /**
@@ -72,8 +73,8 @@ public function getSourceIcon(string $source): ShareIconURL {
 	}
 
 	#[\Override]
-	public function getSourceInteractionResource(string $userId, string $source): InteractionResource {
-		return new NodeResource((int)$source, $userId);
+	public function getSourceInteractionResource(IUser $user, string $source): InteractionResource {
+		return new NodeResource((int)$source, $user->getUID());
 	}
 
 	#[\Override]

lib/private/Sharing/SharingManager.php

@@ -733,7 +733,7 @@ private function validateInteraction(ShareAccessContext $accessContext, Share $s
 					continue;
 				}
 
-				$resources[] = $sourceType->getSourceInteractionResource($userToCheck->getUID(), $source->value);
+				$resources[] = $sourceType->getSourceInteractionResource($userToCheck, $source->value);
 			}
 
 			$event = new RestrictInteractionEvent($userToCheck->getUID(), $userToCheck, $resources, $action, $receivers);

lib/unstable/Sharing/Source/IShareSourceType.php

@@ -13,6 +13,7 @@
 use NCU\Sharing\Icon\ShareIconURL;
 use OCP\AppFramework\Attribute\Implementable;
 use OCP\Interaction\InteractionResource;
+use OCP\IUser;
 use OCP\L10N\IFactory;
 
 /**
@@ -52,9 +53,9 @@ public function getSourceDisplayName(string $source): ?string;
 	public function getSourceIcon(string $source): null|ShareIconSVG|ShareIconURL;
 
 	/**
-	 * @param non-empty-string $userId
+	 * @param IUser $user
 	 * @param non-empty-string $source
 	 * @experimental 35.0.0
 	 */
-	public function getSourceInteractionResource(string $userId, string $source): InteractionResource;
+	public function getSourceInteractionResource(IUser $user, string $source): InteractionResource;
 }

tests/lib/Sharing/TestInteractionResource.php

@@ -10,6 +10,7 @@
 namespace Test\Sharing;
 
 use OCP\Interaction\InteractionResource;
+use OCP\IUser;
 
 final readonly class TestInteractionResource implements InteractionResource {
 	public function __construct(

tests/lib/Sharing/TestShareSourceType1.php

@@ -13,6 +13,7 @@
 use NCU\Sharing\Icon\ShareIconURL;
 use NCU\Sharing\Source\IShareSourceType;
 use OCP\Interaction\InteractionResource;
+use OCP\IUser;
 use OCP\L10N\IFactory;
 
 class TestShareSourceType1 implements IShareSourceType {
@@ -45,7 +46,7 @@ public function getSourceIcon(string $source): null|ShareIconSVG|ShareIconURL {
 	}
 
 	#[\Override]
-	public function getSourceInteractionResource(string $userId, string $source): InteractionResource {
+	public function getSourceInteractionResource(IUser $user, string $source): InteractionResource {
 		return new TestInteractionResource($source);
 	}
 }