groupfolder activities on move/rename

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
nextcloud · May 11, 2023 · 232757b · 232757b
1 parent dcb12e4
commit 232757b
Showing 1 changed file with 22 additions and 1 deletion.
diff --git a/lib/FilesHooks.php b/lib/FilesHooks.php
@@ -354,7 +354,16 @@ public function fileMove($oldPath, $newPath) {
 			$this->moveCase = false;
 			return;
 		}
-		$this->oldAccessList = $this->getUserPathsFromPath($this->oldParentPath, $this->oldParentOwner);
+
+		$oldAccessList = $this->getUserPathsFromPath($this->oldParentPath, $this->oldParentOwner);
+
+		// file can be shared using GroupFolders, including ACL check
+		if ($this->config->getSystemValueBool('activity_use_cached_mountpoints', false)) {
+			[, , $oldFileId] = $this->getSourcePathAndOwner($oldPath);
+			$oldAccessList['users'] = array_merge($oldAccessList['users'], $this->getAffectedUsersFromCachedMounts($oldFileId));
+		}
+
+		$this->oldAccessList = $oldAccessList;
 	}
 
 
@@ -415,6 +424,12 @@ protected function fileRenaming($oldPath, $newPath) {
 		$this->generateRemoteActivity($renameRemotes, Files::TYPE_FILE_CHANGED, time(), $this->currentUser->getCloudId());
 
 		$affectedUsers = $accessList['users'];
+
+		// file can be shared using GroupFolders, including ACL check
+		if ($this->config->getSystemValueBool('activity_use_cached_mountpoints', false)) {
+			$affectedUsers = array_merge($affectedUsers, $this->getAffectedUsersFromCachedMounts($fileId));
+		}
+
 		[$filteredEmailUsers, $filteredNotificationUsers] = $this->getFileChangeActivitySettings($fileId, array_keys($affectedUsers));
 
 		foreach ($affectedUsers as $user => $path) {
@@ -464,6 +479,12 @@ protected function fileMoving($oldPath, $newPath) {
 		$affectedUsers = $accessList['users'];
 		$oldUsers = $this->oldAccessList['users'];
 
+		// file can be shared using GroupFolders, including ACL check
+		if ($this->config->getSystemValueBool('activity_use_cached_mountpoints', false)) {
+			$this->userMountCache->clear(); // clear cache for new data
+			$affectedUsers = array_merge($affectedUsers, $this->getAffectedUsersFromCachedMounts($fileId));
+		}
+
 		$beforeUsers = array_keys($oldUsers);
 		$afterUsers = array_keys($affectedUsers);