Ignore updating last_used_at for deciding the DB connection host (#283)

* moved the updating of last_used_at timestamp to terminating callback

* updated the style

* fixed style ci issues

* fixed style ci issues

* reverted issues

* removed the forceFill method

* fixed style ci issues

* Update Guard.php

* Update PersonalAccessToken.php

* Update Guard.php

Co-authored-by: Taylor Otwell <taylor@laravel.com>

Author: alexjose

src/Guard.php

@@ -65,13 +65,25 @@ public function __invoke(Request $request)
 
             $accessToken = $model::findToken($token);
 
-            if (! $this->isValidAccessToken($accessToken)) {
+            if (! $this->isValidAccessToken($accessToken) ||
+                ! $this->supportsTokens($accessToken->tokenable)) {
                 return;
             }
 
-            return $this->supportsTokens($accessToken->tokenable) ? $accessToken->tokenable->withAccessToken(
-                tap($accessToken->forceFill(['last_used_at' => now()]))->save()
-            ) : null;
+            if (method_exists($accessToken->getConnection(), 'hasModifiedRecords') &&
+                method_exists($accessToken->getConnection(), 'recordsHaveBeenModified')) {
+                tap($accessToken->getConnection()->hasModifiedRecords(), function ($hasModifiedRecords) use ($accessToken) {
+                    $accessToken->forceFill(['last_used_at' => now()])->save();
+
+                    $accessToken->getConnection()->recordsHaveBeenModified($hasModifiedRecords);
+                });
+            } else {
+                $accessToken->forceFill(['last_used_at' => now()])->save();
+            }
+
+            return $accessToken->tokenable->withAccessToken(
+                $accessToken
+            );
         }
     }