refactor(files): use strict string comparison instead of empty()

cristianscheid · provokateurin · web-flow · commit 9272a13e9df1 · 2025-12-01T09:52:59.000-03:00
Co-authored-by: Kate &lt;26026535+provokateurin@users.noreply.github.com&gt;
Signed-off-by: Cristian Scheid &lt;74515775+cristianscheid@users.noreply.github.com&gt;
diff --git a/apps/files/lib/Service/OwnershipTransferService.php b/apps/files/lib/Service/OwnershipTransferService.php
@@ -109,7 +109,7 @@ public function transfer(
 				$cleanUserName = $sourceUid;
 			} else {
 				$cleanUserName = $this->sanitizeFolderName($sourceUser->getDisplayName());
-				if (empty($cleanUserName)) {
+				if ($cleanUserName === '') {
 					$cleanUserName = $sourceUid;
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ public function transfer(`
`109`	`109`	`$cleanUserName = $sourceUid;`
`110`	`110`	`} else {`
`111`	`111`	`$cleanUserName = $this->sanitizeFolderName($sourceUser->getDisplayName());`
`112`		`- if (empty($cleanUserName)) {`
	`112`	`+ if ($cleanUserName === '') {`
`113`	`113`	`$cleanUserName = $sourceUid;`
`114`	`114`	`}`
`115`	`115`	`}`