LogsActivity::activity() - Code Metrics - Inspection of "Merge branch 'master' of github.com:spatie/laravel..." - spatie/laravel-activitylog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 139551...f61205 )

by Freek

created 2016-07-04 09:13 UTC

LogsActivity::activity() A

↳ Parent: LogsActivity

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Spatie\Activitylog\Traits;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphMany;
use Illuminate\Support\Collection;
use Spatie\Activitylog\ActivityLogger;
use Spatie\Activitylog\Models\Activity;

trait LogsActivity
{
    use DetectsChanges;

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

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

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

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

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

        });
    }

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

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

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

        return collect([
            'created',
            'updated',
            'deleted',
        ]);
    }
}


1			<?php
2
3			namespace Spatie\Activitylog\Traits;
4
5			use Illuminate\Database\Eloquent\Model;
6			use Illuminate\Database\Eloquent\Relations\MorphMany;
7			use Illuminate\Support\Collection;
8			use Spatie\Activitylog\ActivityLogger;
9			use Spatie\Activitylog\Models\Activity;
10
11			trait LogsActivity
12			{
13			use DetectsChanges;
14
15			protected static function bootLogsActivity()
16			{
17			static::eventsToBeRecorded()->each(function ($eventName) {
18
19			return static::$eventName(function (Model $model) use ($eventName) {
20
21			$description = $model->getDescriptionForEvent($eventName);
22
23			if ($description == '') {
24			return;
25			}
26
27			app(ActivityLogger::class)
28			->performedOn($model)
29			->withProperties($model->attributeValuesToBeLogged($eventName))
30			->log($description);
31			});
32
33			});
34			}
35
36			public function activity(): MorphMany
37			{
38			return $this->morphMany(Activity::class, 'subject');
			0 ignored issues – show Bug introduced 2016-07-04 09:19 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...
39			}
40
41			public function getDescriptionForEvent(string $eventName): string
42			{
43			return $eventName;
44			}
45
46			/*
47			* Get the event names that should be recorded.
48			*/
49			protected static function eventsToBeRecorded(): Collection
50			{
51			if (isset(static::$recordEvents)) {
52			return collect(static::$recordEvents);
53			}
54
55			return collect([
56			'created',
57			'updated',
58			'deleted',
59			]);
60			}
61			}
62

spatie / laravel-activitylog

Push — master ( 139551...f61205 )

LogsActivity::activity() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like