only save if need to

laravel · Sep 30, 2022 · 9a68cf2 · 9a68cf2
1 parent ebc9045
commit 9a68cf2
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/Actions/DisableTwoFactorAuthentication.php b/src/Actions/DisableTwoFactorAuthentication.php
@@ -15,12 +15,16 @@ class DisableTwoFactorAuthentication
      */
     public function __invoke($user)
     {
-        $user->forceFill([
-            'two_factor_secret' => null,
-            'two_factor_recovery_codes' => null,
-        ] + (Fortify::confirmsTwoFactorAuthentication() ? [
-            'two_factor_confirmed_at' => null,
-        ] : []))->save();
+        if (! is_null($user->two_factor_secret) ||
+            ! is_null($user->two_factor_recovery_codes) ||
+            ! is_null($user->two_factor_confirmed_at)) {
+            $user->forceFill([
+                'two_factor_secret' => null,
+                'two_factor_recovery_codes' => null,
+            ] + (Fortify::confirmsTwoFactorAuthentication() ? [
+                'two_factor_confirmed_at' => null,
+            ] : []))->save();
+        }
 
         TwoFactorAuthenticationDisabled::dispatch($user);
     }