refactor: move lastLogin() implementation to LoginModel

Author: kenjis

src/Entities/User.php

@@ -224,21 +224,13 @@ public function getPasswordHash(): ?string
     }
 
     /**
-     * Returns the last login information for this
-     * user as
+     * Returns the last login information for this user as
      */
     public function lastLogin(bool $allowFailed = false): ?Login
     {
+        /** @var LoginModel $logins */
         $logins = model(LoginModel::class);
 
-        if (! $allowFailed) {
-            $logins->where('success', 1)
-                ->where('user_id', $this->attributes['id'] ?? null);
-        }
-
-        return $logins
-            ->where('identifier', $this->getEmail())
-            ->orderBy('date', 'desc')
-            ->first();
+        return $logins->lastLogin($this, $allowFailed);
     }
 }

src/Models/LoginModel.php

@@ -5,6 +5,7 @@
 use CodeIgniter\I18n\Time;
 use CodeIgniter\Model;
 use CodeIgniter\Shield\Entities\Login;
+use CodeIgniter\Shield\Entities\User;
 use Exception;
 use Faker\Generator;
 
@@ -57,6 +58,22 @@ public function recordLoginAttempt(
         $this->checkQueryReturn($return);
     }
 
+    /**
+     * Returns the last login information for the user
+     */
+    public function lastLogin(User $user, bool $allowFailed = false): ?Login
+    {
+        if (! $allowFailed) {
+            $this->where('success', 1)
+                ->where('user_id', $user->id);
+        }
+
+        return $this
+            ->where('identifier', $user->email)
+            ->orderBy('date', 'desc')
+            ->first();
+    }
+
     /**
      * Generate a fake login for testing
      *