Skip to content

Commit fa2a7b9

Browse files
refactor(files): use explicit empty check instead of elvis operator
Signed-off-by: Cristian Scheid <[email protected]>
1 parent 40d3118 commit fa2a7b9

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

apps/files/lib/Service/OwnershipTransferService.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,12 @@ public function transfer(
108108
if ($useUserId) {
109109
$cleanUserName = $sourceUid;
110110
} else {
111-
$cleanUserName = $this->sanitizeFolderName($sourceUser->getDisplayName()) ?: $sourceUid;
111+
$cleanUserName = $this->sanitizeFolderName($sourceUser->getDisplayName());
112+
if (empty($cleanUserName)) {
113+
$cleanUserName = $sourceUid;
114+
}
112115
}
113-
116+
114117
$finalTarget = "$destinationUid/files/" . $this->sanitizeFolderName($l->t('Transferred from %1$s on %2$s', [$cleanUserName, $date]));
115118
try {
116119
$view->verifyPath(dirname($finalTarget), basename($finalTarget));

0 commit comments

Comments
 (0)