fix: use hasEnabledTwoFactorAuthentication for improved logic clarity in session handling

pushpak1300 · pushpak1300 · commit da99c255d26d · 2025-08-20T01:37:34.000+05:30
diff --git a/app/Http/Controllers/Auth/AuthenticatedSessionController.php b/app/Http/Controllers/Auth/AuthenticatedSessionController.php
@@ -32,7 +32,7 @@ public function store(LoginRequest $request): RedirectResponse
     {
         $user = $request->validateCredentials();
 
-        if (Fortify::confirmsTwoFactorAuthentication() && filled($user->two_factor_confirmed_at)) {
+        if (Fortify::confirmsTwoFactorAuthentication() && $user->hasEnabledTwoFactorAuthentication()) {
             $request->session()->put([
                 'login.id' => $user->getKey(),
                 'login.remember' => $request->boolean('remember'),

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public function store(LoginRequest $request): RedirectResponse`
`32`	`32`	`{`
`33`	`33`	`$user = $request->validateCredentials();`
`34`	`34`
`35`		`- if (Fortify::confirmsTwoFactorAuthentication() && filled($user->two_factor_confirmed_at)) {`
	`35`	`+ if (Fortify::confirmsTwoFactorAuthentication() && $user->hasEnabledTwoFactorAuthentication()) {`
`36`	`36`	`$request->session()->put([`
`37`	`37`	`'login.id' => $user->getKey(),`
`38`	`38`	`'login.remember' => $request->boolean('remember'),`