Fix admin notification when user has no email address

Fixes #406.

Signed-off-by: Daniel Thwaites <danthwaites30@btinternet.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/lib/Service/MailService.php b/lib/Service/MailService.php
index 2682658..35e0bde 100644
--- a/lib/Service/MailService.php
+++ b/lib/Service/MailService.php
@@ -147,7 +147,7 @@ class MailService {
 		}
 	}
 
-	public function notifyAdmins(string $userId, string $userEMailAddress, bool $userIsEnabled, string $userGroupId): void {
+	public function notifyAdmins(string $userId, ?string $userEMailAddress, bool $userIsEnabled, string $userGroupId): void {
 		// Notify admin
 		$adminUsers = $this->groupManager->get('admin')->getUsers();
 
@@ -186,7 +186,7 @@ class MailService {
 	 * @param bool $userIsEnabled the new user account is enabled
 	 * @throws \Exception
 	 */
-	private function sendNewUserNotifyEmail(array $to, string $username, string $userEMailAddress, bool $userIsEnabled): void {
+	private function sendNewUserNotifyEmail(array $to, string $username, ?string $userEMailAddress, bool $userIsEnabled): void {
 		$link = $this->urlGenerator->linkToRouteAbsolute('settings.Users.usersListByGroup', [
 			'group' => 'disabled',
 		]);
@@ -206,7 +206,7 @@ class MailService {
 			$template->addBodyText(
 				$this->l10n->t('"%1$s" (%2$s) registered a new account on %3$s.', [
 					$username,
-					$userEMailAddress,
+					$userEMailAddress ?? $this->l10n->t('no email address given'),
 					$this->defaults->getName(),
 				])
 			);
@@ -214,7 +214,7 @@ class MailService {
 			$template->addBodyText(
 				$this->l10n->t('"%1$s" (%2$s) registered a new account on %3$s and needs to be enabled.', [
 					$username,
-					$userEMailAddress,
+					$userEMailAddress ?? $this->l10n->t('no email address given'),
 					$this->defaults->getName(),
 				])
 			);