fix tapActivity being called twice

The tapActivity method was being called both in LogsActivity trait and
in ActivityLogger::log(). Since log() already handles calling tapActivity
when a subject is set, the duplicate call in LogsActivity is removed.

Fixes #1440

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: freekmurze

src/Traits/LogsActivity.php

@@ -76,17 +76,12 @@ protected static function bootLogsActivity(): void
                 }
 
                 // Actual logging
-                $logger = app(ActivityLogger::class)
+                app(ActivityLogger::class)
                     ->useLog($logName)
                     ->event($eventName)
                     ->performedOn($model)
-                    ->withProperties($event->changes);
-
-                if (method_exists($model, 'tapActivity')) {
-                    $logger->tap([$model, 'tapActivity'], $eventName);
-                }
-
-                $logger->log($description);
+                    ->withProperties($event->changes)
+                    ->log($description);
 
                 // Reset log options so the model can be serialized.
                 $model->activitylogOptions = null;

tests/LogsActivityTest.php

@@ -437,6 +437,32 @@ public function tapActivity(Activity $activity, string $eventName)
     $this->assertEquals('my custom event', $firstActivity->event);
 });
 
+it('will only call tapActivity once', function () {
+    $callCount = 0;
+
+    $model = new class() extends Article {
+        use LogsActivity;
+
+        public static $tapCallCount = 0;
+
+        public function getActivitylogOptions(): LogOptions
+        {
+            return LogOptions::defaults();
+        }
+
+        public function tapActivity(Activity $activity, string $eventName)
+        {
+            static::$tapCallCount++;
+        }
+    };
+
+    $model::$tapCallCount = 0;
+    $entity = new $model();
+    $entity->save();
+
+    $this->assertEquals(1, $model::$tapCallCount);
+});
+
 it('will not submit log when there is no changes', function () {
     $model = new class() extends Article {
         use LogsActivity;