Merge pull request #41418 from nextcloud/backport/41404/stable27

[stable27] fix(workflow): Fix "Call to a member function getUID() on null" with …
nextcloud · Nov 13, 2023 · 5d081f7 · 5d081f7
2 parents 712a4c8 + 65047cb
commit 5d081f7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/apps/workflowengine/lib/Entity/File.php b/apps/workflowengine/lib/Entity/File.php
@@ -140,10 +140,10 @@ public function prepareRuleMatcher(IRuleMatcher $ruleMatcher, string $eventName,
 		}
 	}
 
-	public function isLegitimatedForUserId(string $uid): bool {
+	public function isLegitimatedForUserId(string $userId): bool {
 		try {
 			$node = $this->getNode();
-			if ($node->getOwner()->getUID() === $uid) {
+			if ($node->getOwner()?->getUID() === $userId) {
 				return true;
 			}
 
@@ -154,7 +154,7 @@ public function isLegitimatedForUserId(string $uid): bool {
 				$fileId = $node->getId();
 			}
 
-			$mountInfos = $this->userMountCache->getMountsForFileId($fileId, $uid);
+			$mountInfos = $this->userMountCache->getMountsForFileId($fileId, $userId);
 			foreach ($mountInfos as $mountInfo) {
 				$mount = $this->mountManager->getMountFromMountInfo($mountInfo);
 				if ($mount && $mount->getStorage() && !empty($mount->getStorage()->getCache()->get($fileId))) {