feat: Store user which has downloaded file in activity feed

Signed-off-by: Konstantin Myakshin <molodchick@gmail.com>
nextcloud · Jul 11, 2024 · 6e9b00f · 6e9b00f
1 parent d707441
commit 6e9b00f
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 18 deletions.
diff --git a/apps/files_sharing/lib/Activity/Providers/Downloads.php b/apps/files_sharing/lib/Activity/Providers/Downloads.php
@@ -55,12 +55,14 @@ 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'])) {
-				$subject = $this->l->t('{file} downloaded via public link');
-				$this->setSubjects($event, $subject, $parsedParameters);
+			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');
-				$this->setSubjects($event, $subject, $parsedParameters);
+			}
+
+			$this->setSubjects($event, $subject, $parsedParameters);
+			if (isset($parsedParameters['remote-address-hash']['type'])) {
 				$event = $this->eventMerger->mergeEvents('file', $event, $previousEvent);
 			}
 		} elseif ($event->getSubject() === self::SUBJECT_SHARED_FILE_BY_EMAIL_DOWNLOADED ||
@@ -92,20 +94,25 @@ protected function getParsedParameters(IEvent $event) {
 		switch ($subject) {
 			case self::SUBJECT_PUBLIC_SHARED_FILE_DOWNLOADED:
 			case self::SUBJECT_PUBLIC_SHARED_FOLDER_DOWNLOADED:
+				$parsedParameters = [
+					'file' => $this->getFile($parameters[0], $event),
+				];
+
 				if (isset($parameters[1])) {
-					return [
-						'file' => $this->getFile($parameters[0], $event),
-						'remote-address-hash' => [
-							'type' => 'highlight',
-							'id' => $parameters[1],
-							'name' => $parameters[1],
-							'link' => '',
-						],
+					$parsedParameters['remote-address-hash'] = [
+						'type' => 'highlight',
+						'id' => $parameters[1],
+						'name' => $parameters[1],
+						'link' => '',
 					];
 				}
-				return [
-					'file' => $this->getFile($parameters[0], $event),
-				];
+
+				if (isset($parameters[2])) {
+					$parsedParameters['actor'] = $this->getUser($parameters[2]);
+				}
+
+				return $parsedParameters;
+
 			case self::SUBJECT_SHARED_FILE_BY_EMAIL_DOWNLOADED:
 			case self::SUBJECT_SHARED_FOLDER_BY_EMAIL_DOWNLOADED:
 				return [

diff --git a/apps/files_sharing/lib/Controller/ShareController.php b/apps/files_sharing/lib/Controller/ShareController.php
@@ -30,21 +30,23 @@
 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;
 use OCP\Share\IManager as ShareManager;
 use OCP\Share\IPublicShareTemplateFactory;
 use OCP\Share\IShare;
-use OCP\Template;
 
 /**
  * @package OCA\Files_Sharing\Controllers
  */
 #[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 +70,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 +516,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);

diff --git a/apps/files_sharing/tests/Controller/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php
@@ -37,6 +37,7 @@
 use OCP\IURLGenerator;
 use OCP\IUser;
 use OCP\IUserManager;
+use OCP\IUserSession;
 use OCP\Security\ISecureRandom;
 use OCP\Share\Exceptions\ShareNotFound;
 use OCP\Share\IPublicShareTemplateFactory;
@@ -84,6 +85,8 @@ class ShareControllerTest extends \Test\TestCase {
 	private $defaults;
 	/** @var IPublicShareTemplateFactory|MockObject */
 	private $publicShareTemplateFactory;
+	/** @var IUserSession|MockObject */
+	private $userSession;
 
 	protected function setUp(): void {
 		parent::setUp();
@@ -124,6 +127,8 @@ protected function setUp(): void {
 				)
 			);
 
+		$this->userSession = $this->createMock(IUserSession::class);
+
 		$this->shareController = new \OCA\Files_Sharing\Controller\ShareController(
 			$this->appName,
 			$this->createMock(IRequest::class),
@@ -142,6 +147,7 @@ protected function setUp(): void {
 			$this->secureRandom,
 			$this->defaults,
 			$this->publicShareTemplateFactory,
+			$this->userSession,
 		);