feat: Store user which has downloaded file in activity feed

Signed-off-by: Konstantin Myakshin <molodchick@gmail.com>

Author: Koc

apps/files_sharing/lib/Activity/Providers/Downloads.php

@@ -55,11 +55,15 @@ public function parseLongVersion(IEvent $event, ?IEvent $previousEvent = null) {
 
 		if ($event->getSubject() === self::SUBJECT_PUBLIC_SHARED_FILE_DOWNLOADED ||
 			$event->getSubject() === self::SUBJECT_PUBLIC_SHARED_FOLDER_DOWNLOADED) {
-			if (!isset($parsedParameters['remote-address-hash']['type'])) {
+			if (isset($parsedParameters['actor'])) {
+				$subject = $this->l->t('{file} downloaded via public link by {actor}');
+			} else {
 				$subject = $this->l->t('{file} downloaded via public link');
+			}
+
+			if (!isset($parsedParameters['remote-address-hash']['type'])) {
 				$this->setSubjects($event, $subject, $parsedParameters);
 			} else {
-				$subject = $this->l->t('{file} downloaded via public link');
 				$this->setSubjects($event, $subject, $parsedParameters);
 				$event = $this->eventMerger->mergeEvents('file', $event, $previousEvent);
 			}
@@ -101,6 +105,7 @@ protected function getParsedParameters(IEvent $event) {
 							'name' => $parameters[1],
 							'link' => '',
 						],
+						'actor' => isset($parameters[2]) ? $this->getUser($parameters[2]) : null,
 					];
 				}
 				return [

apps/files_sharing/lib/Controller/ShareController.php

@@ -30,7 +30,9 @@
 use OCP\IRequest;
 use OCP\ISession;
 use OCP\IURLGenerator;
+use OCP\IUser;
 use OCP\IUserManager;
+use OCP\IUserSession;
 use OCP\Security\ISecureRandom;
 use OCP\Share;
 use OCP\Share\Exceptions\ShareNotFound;
@@ -45,6 +47,7 @@
 #[OpenAPI(scope: OpenAPI::SCOPE_IGNORE)]
 class ShareController extends AuthPublicShareController {
 	protected ?Share\IShare $share = null;
+	private ?IUser $currentUser = null;
 
 	public const SHARE_ACCESS = 'access';
 	public const SHARE_AUTH = 'auth';
@@ -68,8 +71,11 @@ public function __construct(
 		protected ISecureRandom $secureRandom,
 		protected Defaults $defaults,
 		private IPublicShareTemplateFactory $publicShareTemplateFactory,
+		IUserSession $userSession,
 	) {
 		parent::__construct($appName, $request, $session, $urlGenerator);
+
+		$this->currentUser = $userSession->getUser();
 	}
 
 	/**
@@ -511,11 +517,11 @@ protected function singleFileDownloaded(Share\IShare $share, \OCP\Files\Node $no
 		} else {
 			if ($node instanceof \OCP\Files\File) {
 				$subject = Downloads::SUBJECT_PUBLIC_SHARED_FILE_DOWNLOADED;
-				$parameters[] = $remoteAddressHash;
 			} else {
 				$subject = Downloads::SUBJECT_PUBLIC_SHARED_FOLDER_DOWNLOADED;
-				$parameters[] = $remoteAddressHash;
 			}
+			$parameters[] = $remoteAddressHash;
+			$parameters[] = $this->currentUser->getUID();
 		}
 
 		$this->publishActivity($subject, $parameters, $share->getSharedBy(), $fileId, $userPath);