Merge pull request #119 from kenjis/add-email-null-check

Add email null check

Author: kenjis

src/Authentication/Actions/EmailActivator.php

@@ -5,6 +5,7 @@
 use CodeIgniter\Exceptions\PageNotFoundException;
 use CodeIgniter\HTTP\IncomingRequest;
 use CodeIgniter\HTTP\RedirectResponse;
+use CodeIgniter\Shield\Exceptions\LogicException;
 use CodeIgniter\Shield\Exceptions\RuntimeException;
 use CodeIgniter\Shield\Models\UserIdentityModel;
 
@@ -23,6 +24,13 @@ public function show(): string
             throw new RuntimeException('Cannot get the User.');
         }
 
+        $userEmail = $user->getAuthEmail();
+        if ($userEmail === null) {
+            throw new LogicException(
+                'Email Activation needs user email address. user_id: ' . $user->getAuthId()
+            );
+        }
+
         /** @var UserIdentityModel $identityModel */
         $identityModel = model(UserIdentityModel::class);
 
@@ -44,7 +52,7 @@ public function show(): string
         // Send the email
         helper('email');
         $return = emailer()->setFrom(setting('Email.fromEmail'), setting('Email.fromName') ?? '')
-            ->setTo($user->getAuthEmail())
+            ->setTo($userEmail)
             ->setSubject(lang('Auth.emailActivateSubject'))
             ->setMessage(view(setting('Auth.views')['action_email_activate_email'], ['code' => $code]))
             ->send();