fix: prevent hidden attributes from being logged (#70)

src/Loggers/AbstractModelLogger.php

@@ -32,14 +32,27 @@ protected function getModelName(Model $model)
     protected function activityLogger(string $logName = null): ActivityLogger
     {
         $defaultLogName = $this->getLogName();
-        
+
         $logStatus = app(ActivityLogStatus::class);
 
         return app(ActivityLogger::class)
             ->useLog($logName ?? $defaultLogName)
             ->setLogStatus($logStatus);
     }
 
+    protected function getLoggableAttributes(Model $model, array $values = []): array
+    {
+        if (count($model->getVisible()) > 0) {
+            $values = array_intersect_key($values, array_flip($model->getVisible()));
+        }
+
+        if (count($model->getHidden()) > 0) {
+            $values = array_diff_key($values, array_flip($model->getHidden()));
+        }
+
+        return $values;
+    }
+
     protected function log(Model $model, string $event, ?string $description = null, mixed $attributes = null)
     {
         if(is_null($description)) {
@@ -53,15 +66,15 @@ protected function log(Model $model, string $event, ?string $description = null,
         $this->activityLogger()
             ->event($event)
             ->performedOn($model)
-            ->withProperties($attributes)
+            ->withProperties($this->getLoggableAttributes($model, $attributes))
             ->log($description);
     }
 
     public function created(Model $model)
     {
         $this->log($model, 'Created', attributes:$model->getAttributes());
     }
-    
+
     public function updated(Model $model)
     {
         $changes = $model->getChanges();
@@ -70,7 +83,7 @@ public function updated(Model $model)
         if (count($changes) === 1 && array_key_exists('remember_token', $changes)) {
             return;
         }
-        
+
         $this->log($model, 'Updated', attributes:$changes);
     }