What do you think of this approach for logs in the pivot table?

[YannikFirre]

Hello,

EDIT :

In fact, you can simply use tapActivity like this

    public function tapActivity(Activity $activity, string $eventName)
    {
        $activity->subject()->associate($this->pivotParent);
    }

---

I would like to have your opinion on this approach to log changes made in a pivot table linked to a model without having to put an incremental field and be directly linked to the parent model.

Thanks :)

<?php

namespace App\Models\Concerns;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Arr;
use Illuminate\Support\Str;
use Spatie\Activitylog\ActivityLogger;

trait LogsActivityForPivot
{
    protected static function bootLogsActivityForPivot(): void
    {
        foreach (['created', 'updated', 'deleted'] as $eventName) {
            static::$eventName(static function (self $model) use ($eventName) {
                $attributes = Arr::except($model->getAttributes(), [$model->pivotParent->getForeignKey()]);

                foreach ($attributes as $attribute => $value) {
                    if (isset($model->logForeignKeys[$attribute]) === false) {
                        continue;
                    }

                    $attributes += self::getRelatedModelAttributeValue($model, $model->logForeignKeys[$attribute]);
                }

                app(ActivityLogger::class)
                    ->useLog('pivot')
                    ->event($eventName)
                    ->performedOn($model->pivotParent)
                    ->withProperties($attributes)
                    ->log($eventName);
            });
        }
    }

    protected static function getRelatedModelAttributeValue(Model $model, string $attribute): array
    {
        $relatedModelNames = explode('.', $attribute);
        $relatedAttribute = array_pop($relatedModelNames);

        $attributeName = [];
        $relatedModel = $model;

        do {
            $attributeName[] = $relatedModelName = static::getRelatedModelRelationName($relatedModel, array_shift($relatedModelNames));

            $relatedModel = $relatedModel->$relatedModelName ?? $relatedModel->$relatedModelName();
        } while (! empty($relatedModelNames));

        $attributeName[] = $relatedAttribute;

        return [implode('.', $attributeName) => $relatedModel->$relatedAttribute ?? null];
    }

    protected static function getRelatedModelRelationName(Model $model, string $relation): string
    {
        return Arr::first([
            $relation,
            Str::snake($relation),
            Str::camel($relation),
        ], function (string $method) use ($model): bool {
            return method_exists($model, $method);
        }, $relation);
    }
}