fix logUnguarded() to respect Model::unguard()

When Model::unguard() is called globally, the $guarded property still
contains its default value but isUnguarded() returns true. The code was
only checking $guarded, not the unguarded state.

Now checks isUnguarded() to properly detect globally unguarded models.

Fixes #1123

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

Author: freekmurze

src/Traits/LogsActivity.php

@@ -213,12 +213,15 @@ public function attributesToBeLogged(): array
 
         // Determine if unguarded attributes will be logged.
         if ($this->shouldLogUnguarded()) {
-
-            // Get only attribute names, not intrested in the values here then guarded
-            // attributes. get only keys than not present in guarded array, because
-            // we are logging the unguarded attributes and we cant have both!
-
-            $attributes = array_merge($attributes, array_diff(array_keys($this->getAttributes()), $this->getGuarded()));
+            // If globally unguarded, log all attributes
+            if (static::isUnguarded()) {
+                $attributes = array_merge($attributes, array_keys($this->getAttributes()));
+            } else {
+                // Get only attribute names, not interested in the values here then guarded
+                // attributes. get only keys than not present in guarded array, because
+                // we are logging the unguarded attributes and we can't have both!
+                $attributes = array_merge($attributes, array_diff(array_keys($this->getAttributes()), $this->getGuarded()));
+            }
         }
 
         if (! empty($this->activitylogOptions->logAttributes)) {
@@ -247,6 +250,12 @@ public function shouldLogUnguarded(): bool
             return false;
         }
 
+        // If the model is globally unguarded via Model::unguard(),
+        // all attributes should be considered unguarded.
+        if (static::isUnguarded()) {
+            return true;
+        }
+
         // This case means all of the attributes are guarded
         // so we'll not have any unguarded anyway.
         if (in_array('*', $this->getGuarded())) {

tests/DetectsChangesTest.php

@@ -2,6 +2,7 @@
 
 use Carbon\Carbon;
 use Carbon\CarbonInterval;
+use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
 use Illuminate\Support\Arr;
 use Spatie\Activitylog\Contracts\LoggablePipe;
@@ -1300,6 +1301,47 @@ public function getActivitylogOptions(): LogOptions
     $this->assertEquals([], $this->getLastActivity()->changes()->toArray());
 });
 
+it('can log unguarded attributes when Model::unguard() is called globally', function () {
+    // Model without explicit $fillable or $guarded
+    $articleClass = new class() extends Article {
+        use LogsActivity;
+
+        public function getActivitylogOptions(): LogOptions
+        {
+            return LogOptions::defaults()
+            ->logUnguarded()
+            ->logExcept(['id', 'created_at', 'updated_at', 'deleted_at']);
+        }
+    };
+
+    // Globally unguard all models
+    Model::unguard();
+
+    try {
+        $article = new $articleClass();
+        $article->name = 'my name';
+        $article->text = 'my text';
+        $article->save();
+
+        $expectedChanges = [
+            'attributes' => [
+                'name' => 'my name',
+                'text' => 'my text',
+                'user_id' => null,
+                'json' => null,
+                'price' => null,
+                'interval' => null,
+                'status' => null,
+            ],
+        ];
+
+        $this->assertEquals($expectedChanges, $this->getLastActivity()->changes()->toArray());
+    } finally {
+        // Re-guard models
+        Model::reguard();
+    }
+});
+
 it('can use hidden as loggable attributes', function () {
     $articleClass = new class() extends Article {
         use LogsActivity;