Merge pull request #13 from coderflexx/enhancements

Enhancement

Author: ousid

src/PendingVisit.php

@@ -110,20 +110,25 @@ protected function shouldBeLoggedAgain(Visit $visit): bool
 
     public function __destruct()
     {
-        if (! $this->isCrawler) {
-            $visit = $this->model
-                ->visits()
-                ->latest()
-                ->firstOrCreate($this->buildJsonColumns(), [
-                    'data' => $this->attributes,
-                ]);
-
-            $visit->when(
-                $this->shouldBeLoggedAgain($visit),
-                function () use ($visit) {
-                    $visit->replicate()->save();
-                }
-            );
+        // if the current request is a crawler,
+        // we don't need to log the visit
+        // because it's not a real visitor
+        if ($this->isCrawler) {
+            return null;
         }
+
+        $visit = $this->model
+            ->visits()
+            ->latest()
+            ->firstOrCreate($this->buildJsonColumns(), [
+                'data' => $this->attributes,
+            ]);
+
+        $visit->when(
+            $this->shouldBeLoggedAgain($visit),
+            function () use ($visit) {
+                $visit->replicate()->save();
+            }
+        );
     }
 }

tests/Feature/Visits/VisitsTest.php

@@ -225,5 +225,5 @@
     request()->headers->set('User-Agent', 'Googlebot');
     $post->visit();
 
-    expect($post->visits->first())->toBe(null);
+    expect($post->visits->count())->toBe(0);
 });