refactor(ShareApiController): Check for null and empty strings with e…

…mpty() Proactive measure to avoid warnings in higher php versions as well possible type errors Signed-off-by: fenn-cs <[email protected]>
nextcloud · Oct 24, 2024 · f183063 · f183063
1 parent bc6a3e8
commit f183063
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -661,13 +661,10 @@ public function createShare(
 		// Handle mail send
 		if (is_null($sendMail)) {
 			// Define a default behavior when sendMail is not provided
-			if ($shareType === IShare::TYPE_EMAIL && strlen($shareWith) !== 0) {
-				// For email shares, the default is to send the mail
-				$share->setMailSend(true);
-			} else {
-				// For all other share types, the default is to not send the mail
-				$share->setMailSend(false);
-			}
+			// For email shares with a valid recipient, the default is to send the mail
+			// For all other share types, the default is to not send the mail
+			$allowSendMail = ($shareType === IShare::TYPE_EMAIL && !empty($shareWith));
+			$share->setMailSend($allowSendMail);
 		} else {
 			$share->setMailSend($sendMail === 'true');
 		}