Merge pull request #1068 from arodu/feature/lastlogin-config

fix: config for updateLastLogin
CakeDC · Feb 7, 2024 · 88f053f · 88f053f
2 parents 71ce812 + a7d7dbe
commit 88f053f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/config/users.php b/config/users.php
@@ -47,6 +47,10 @@
             // determines if registration workflow includes email validation
             'validate' => true,
         ],
+        'Login' => [
+            'updateLastLogin' => true,
+            'lastLoginField' => 'last_login',
+        ],
         'Registration' => [
             // determines if the register is enabled
             'active' => true,

diff --git a/src/Controller/Component/LoginComponent.php b/src/Controller/Component/LoginComponent.php
@@ -236,10 +236,14 @@ protected function checkSafeHost(?string $queryRedirect = null): bool
      */
     protected function updateLastLogin($user)
     {
+        if (!Configure::read('Users.Login.updateLastLogin', true)) {
+            return;
+        }
+        $field = Configure::read('Users.Login.lastLoginField', 'last_login');
         $now = \Cake\I18n\FrozenTime::now();
-        $user->set('last_login', $now);
+        $user->set($field, $now);
         $this->getController()->getUsersTable()->updateAll(
-            ['last_login' => $now->format('Y-m-d H:i:s')],
+            [$field => $now->format('Y-m-d H:i:s')],
             ['id' => $user->id]
         );
     }