fix: RiskyTruthyFalsyComparison

Signed-off-by: Luka Trovic <[email protected]>
nextcloud · May 31, 2024 · 7e64905 · 7e64905
1 parent 73fb85b
commit 7e64905
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/apps/files_sharing/lib/Command/FixBrokenShares.php b/apps/files_sharing/lib/Command/FixBrokenShares.php
@@ -49,7 +49,10 @@ public function execute(InputInterface $input, OutputInterface $output): int {
 			if ($this->orphanHelper->isShareValid($share['owner'], $share['fileid']) || !$this->orphanHelper->fileExists($share['fileid'])) {
 				continue;
 			}
-			if ($owner = $this->orphanHelper->findOwner($share['fileid'])) {
+
+			$owner = $this->orphanHelper->findOwner($share['fileid']);
+
+			if (!empty($owner)) {
 				$this->orphanHelper->updateShareOwner($share['id'], $owner);
 				$output->writeln("Share {$share['id']} updated to owner $owner");
 			}