Spatie\Activitylog\Traits\LogsActivity

Complexity

Total Complexity

9

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	bootLogsActivity()	0	26	3
A	shouldLogChanges()	0	8	2
A	causesActivity()	0	4	1
A	getDescriptionForEvent()	0	4	1
A	eventsToBeRecorded()	0	12	2

<?php

namespace Spatie\Activitylog\Traits;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphTo;
use Spatie\Activitylog\ActivityLogger;

trait LogsActivity
{
    use DetectsChanges;

    protected static function bootLogsActivity()
    {
        collect(static::eventsToBeRecorded())->each(function ($eventName) {

            return static::$eventName(function (Model $model) use ($eventName) {

                $description = $model->getDescriptionForEvent($eventName);

                if ($description == '') {
                    return;
                }

                $extraProperties = [];

                if ($model->shouldLogChanges()) {
                    $extraProperties['changes'] = $model->getChangedValues();
                }

                app(ActivityLogger::class)
                    ->performedOn($model)
                    ->withExtraProperties($extraProperties)
                    ->log($description);
            });

        });
    }

    public function shouldLogChanges(): bool
    {
        if (!isset($this->logChangesOnAttributes)) {
            return false;
        }

        return count($this->logChangesOnAttributes);

The property logChangesOnAttributes does not exist. Did you maybe forget to declare it?

    }

    public function causesActivity(): MorphTo
    {
        return $this->morphTo();

It seems like morphTo() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function getDescriptionForEvent(string $eventName): string
    {
        return $eventName;
    }

    /*
     * Get the event names that should be recorded.
     */
    protected static function eventsToBeRecorded(): array
    {
        if (isset(static::$recordEvents)) {
            return static::$recordEvents;
        }

        return [
            'created',
            'updated',
            'deleted',
        ];
    }
}