Skip to content

Commit 0aa3720

Browse files
committed
refactor: call esolveMode() only once in loginAction
1 parent 84ab8d9 commit 0aa3720

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

src/Controllers/MagicLinkController.php

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,9 @@ public function loginAction()
103103
// Delete any previous magic-link identities
104104
$identityModel->deleteIdentitiesByType($user, Session::ID_TYPE_MAGIC_LINK);
105105

106-
// Generate the code and save it as an identity
107-
$token = $this->resolveMode()['token'];
106+
$mode = $this->resolveMode();
107+
108+
$token = $mode['token'];
108109

109110
$identityModel->insert([
110111
'user_id' => $user->id,
@@ -126,12 +127,10 @@ public function loginAction()
126127
->setFrom(setting('Email.fromEmail'), setting('Email.fromName') ?? '');
127128
$email->setTo($user->email);
128129

129-
$email->setSubject($this->resolveMode()['emailSubject']);
130-
131-
$emailBodyViewFile = $this->resolveMode()['emailView'];
130+
$email->setSubject($mode['emailSubject']);
132131

133132
$email->setMessage($this->view(
134-
config('Auth')->views[$emailBodyViewFile],
133+
setting('Auth.views')[$mode['emailView']],
135134
['token' => $token, 'user' => $user, 'ipAddress' => $ipAddress, 'userAgent' => $userAgent, 'date' => $date],
136135
['debug' => false],
137136
));

0 commit comments

Comments
 (0)