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

Proactive measure to avoid warnings in higher php versions as well possible type errors

Signed-off-by: fenn-cs <[email protected]>

Author: nfebe

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');
 		}