Do not cache file ids in FileSystemTags

Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>

Author: st3iny

apps/workflowengine/lib/Check/FileSystemTags.php

@@ -6,6 +6,7 @@
  * @author Christoph Wurst <christoph@winzerhof-wurst.at>
  * @author Joas Schilling <coding@schilljs.com>
  * @author Julius Härtl <jus@bitgrid.net>
+ * @author Richard Steinmetz <richard@steinmetz.cloud>
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -26,6 +27,7 @@
 namespace OCA\WorkflowEngine\Check;
 
 use OCA\Files_Sharing\SharedStorage;
+use OCA\GroupFolders\Mount\GroupFolderStorage;
 use OCA\WorkflowEngine\Entity\File;
 use OCP\Files\Cache\ICache;
 use OCP\Files\IHomeStorage;
@@ -131,8 +133,11 @@ protected function getSystemTags() {
 	 * @return int[]
 	 */
 	protected function getFileIds(ICache $cache, $path, $isExternalStorage) {
+		// Do not cache file ids inside group folders because multiple file ids might be mapped to
+		// the same combination of cache id + path.
+		$shouldCacheFileIds = !$this->storage->instanceOfStorage(GroupFolderStorage::class);
 		$cacheId = $cache->getNumericStorageId();
-		if (isset($this->fileIds[$cacheId][$path])) {
+		if ($shouldCacheFileIds && isset($this->fileIds[$cacheId][$path])) {
 			return $this->fileIds[$cacheId][$path];
 		}
 
@@ -148,7 +153,9 @@ protected function getFileIds(ICache $cache, $path, $isExternalStorage) {
 			$parentIds[] = $cache->getId($path);
 		}
 
-		$this->fileIds[$cacheId][$path] = $parentIds;
+		if ($shouldCacheFileIds) {
+			$this->fileIds[$cacheId][$path] = $parentIds;
+		}
 
 		return $parentIds;
 	}