LogsActivity::activities() - Code Metrics - Inspection of "v3 (#441)" - spatie/laravel-activitylog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 28f481...c1dd49 )

by Freek

created 2018-10-16 10:51 UTC

LogsActivity::activities() A

↳ Parent: LogsActivity

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

namespace Spatie\Activitylog\Traits;

use Illuminate\Support\Collection;
use Spatie\Activitylog\ActivityLogger;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;
use Spatie\Activitylog\ActivitylogServiceProvider;
use Illuminate\Database\Eloquent\Relations\MorphMany;

trait LogsActivity
{
    use DetectsChanges;

    protected $enableLoggingModelsEvents = true;

    protected static function bootLogsActivity()
    {
        static::eventsToBeRecorded()->each(function ($eventName) {
            return static::$eventName(function (Model $model) use ($eventName) {
                if (! $model->shouldLogEvent($eventName)) {
                    return;
                }

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

                $logName = $model->getLogNameToUse($eventName);

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

                app(ActivityLogger::class)
                    ->useLog($logName)
                    ->performedOn($model)
                    ->withProperties($model->attributeValuesToBeLogged($eventName))
                    ->log($description);
            });
        });
    }

    public function disableLogging()
    {
        $this->enableLoggingModelsEvents = false;

        return $this;
    }

    public function enableLogging()
    {
        $this->enableLoggingModelsEvents = true;

        return $this;
    }

    public function activities(): MorphMany
    {
        return $this->morphMany(ActivitylogServiceProvider::determineActivityModel(), 'subject');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

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

    public function getLogNameToUse(string $eventName = ''): string

    {
        if (isset(static::$logName)) {
            return static::$logName;
        }

        return config('activitylog.default_log_name');
    }

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

        $events = collect([
            'created',
            'updated',
            'deleted',
        ]);

        if (collect(class_uses_recursive(static::class))->contains(SoftDeletes::class)) {
            $events->push('restored');
        }

        return $events;
    }

    public function attributesToBeIgnored(): array
    {
        if (! isset(static::$ignoreChangedAttributes)) {
            return [];
        }

        return static::$ignoreChangedAttributes;
    }

    protected function shouldLogEvent(string $eventName): bool
    {
        if (! $this->enableLoggingModelsEvents) {
            return false;
        }

        if (! in_array($eventName, ['created', 'updated'])) {
            return true;
        }

        if (array_has($this->getDirty(), 'deleted_at')) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            if ($this->getDirty()['deleted_at'] === null) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
                return false;
            }
        }

        //do not log update event if only ignored attributes are changed
        return (bool) count(array_except($this->getDirty(), $this->attributesToBeIgnored()));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			namespace Spatie\Activitylog\Traits;
4
5			use Illuminate\Support\Collection;
6			use Spatie\Activitylog\ActivityLogger;
7			use Illuminate\Database\Eloquent\Model;
8			use Illuminate\Database\Eloquent\SoftDeletes;
9			use Spatie\Activitylog\ActivitylogServiceProvider;
10			use Illuminate\Database\Eloquent\Relations\MorphMany;
11
12			trait LogsActivity
13			{
14			use DetectsChanges;
15
16			protected $enableLoggingModelsEvents = true;
17
18			protected static function bootLogsActivity()
19			{
20			static::eventsToBeRecorded()->each(function ($eventName) {
21			return static::$eventName(function (Model $model) use ($eventName) {
22			if (! $model->shouldLogEvent($eventName)) {
23			return;
24			}
25
26			$description = $model->getDescriptionForEvent($eventName);
27
28			$logName = $model->getLogNameToUse($eventName);
29
30			if ($description == '') {
31			return;
32			}
33
34			app(ActivityLogger::class)
35			->useLog($logName)
36			->performedOn($model)
37			->withProperties($model->attributeValuesToBeLogged($eventName))
38			->log($description);
39			});
40			});
41			}
42
43			public function disableLogging()
44			{
45			$this->enableLoggingModelsEvents = false;
46
47			return $this;
48			}
49
50			public function enableLogging()
51			{
52			$this->enableLoggingModelsEvents = true;
53
54			return $this;
55			}
56
57			public function activities(): MorphMany
58			{
59			return $this->morphMany(ActivitylogServiceProvider::determineActivityModel(), 'subject');
			0 ignored issues – show Bug introduced 2016-10-13 19:21 UTC by Report Bug Copy Issue Report It seems like `morphMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
60			}
61
62			public function getDescriptionForEvent(string $eventName): string
63			{
64			return $eventName;
65			}
66
67			public function getLogNameToUse(string $eventName = ''): string
			0 ignored issues – show Unused Code introduced 2017-07-31 12:11 UTC by Report Bug Copy Issue Report The parameter `$eventName` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
68			{
69			if (isset(static::$logName)) {
70			return static::$logName;
71			}
72
73			return config('activitylog.default_log_name');
74			}
75
76			/*
77			* Get the event names that should be recorded.
78			*/
79			protected static function eventsToBeRecorded(): Collection
80			{
81			if (isset(static::$recordEvents)) {
82			return collect(static::$recordEvents);
83			}
84
85			$events = collect([
86			'created',
87			'updated',
88			'deleted',
89			]);
90
91			if (collect(class_uses_recursive(static::class))->contains(SoftDeletes::class)) {
92			$events->push('restored');
93			}
94
95			return $events;
96			}
97
98			public function attributesToBeIgnored(): array
99			{
100			if (! isset(static::$ignoreChangedAttributes)) {
101			return [];
102			}
103
104			return static::$ignoreChangedAttributes;
105			}
106
107			protected function shouldLogEvent(string $eventName): bool
108			{
109			if (! $this->enableLoggingModelsEvents) {
110			return false;
111			}
112
113			if (! in_array($eventName, ['created', 'updated'])) {
114			return true;
115			}
116
117			if (array_has($this->getDirty(), 'deleted_at')) {
			0 ignored issues – show Bug introduced 2016-10-10 13:17 UTC by Report Bug Copy Issue Report It seems like `getDirty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
118			if ($this->getDirty()['deleted_at'] === null) {
			0 ignored issues – show Bug introduced 2016-10-10 13:17 UTC by Report Bug Copy Issue Report It seems like `getDirty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
119			return false;
120			}
121			}
122
123			//do not log update event if only ignored attributes are changed
124			return (bool) count(array_except($this->getDirty(), $this->attributesToBeIgnored()));
			0 ignored issues – show Bug introduced 2016-10-10 13:17 UTC by Report Bug Copy Issue Report It seems like `getDirty()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
125			}
126			}
127

spatie / laravel-activitylog

Push — master ( 28f481...c1dd49 )

LogsActivity::activities() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like