Pivot::boot() - Code Metrics - Inspection of "Fixed the boot method to actually call each trait'..." - tylercd100/laravel-model-improved - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (62cc10)

by Tyler

created 2017-09-13 18:07 UTC

Pivot::boot() A

↳ Parent: Pivot

Complexity

Conditions	2
Paths	1

Size

Total Lines	20
Code Lines	11

Duplication

Lines	20
Ratio	100 %

Code Coverage

Tests	7
CRAP Score	2.1922

Importance

Changes

Metric	Value
dl	20
loc	20
ccs	7
cts	11
cp	0.6364
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	11
nc	1
nop	0
crap	2.1922

<?php

namespace Tylercd100\Database\Eloquent\Relations;

use Illuminate\Database\Eloquent\Relations\Pivot as IlluminatePivot;
use Tylercd100\Database\Eloquent\Traits\AutoCache;
use Tylercd100\Database\Eloquent\Traits\AutoValidate;

abstract class Pivot extends IlluminatePivot

{
    use AutoCache, AutoValidate;
    
    /**
     * A method best used to register Eloquent events when the model boots up
     * @return void
     */
    protected static function boot()
    {
        parent::boot();

        // Auto Validate
        self::saving(function ($model) {
            if (!$model->validate()) {
                return false;
            }
        });

        // Auto Cache
        static::saved(function ($model) {
            $model->refresh();
        }, -1);


        static::deleted(function ($model) {
            $model->refresh();
        }, -1);

    }
}


1			<?php
2
3			namespace Tylercd100\Database\Eloquent\Relations;
4
5			use Illuminate\Database\Eloquent\Relations\Pivot as IlluminatePivot;
6			use Tylercd100\Database\Eloquent\Traits\AutoCache;
7			use Tylercd100\Database\Eloquent\Traits\AutoValidate;
8
9		View Code Duplication	abstract class Pivot extends IlluminatePivot
			0 ignored issues – show Duplication introduced 2017-09-13 18:09 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
10			{
11			use AutoCache, AutoValidate;
12
13			/**
14			* A method best used to register Eloquent events when the model boots up
15			* @return void
16			*/
17	3		protected static function boot()
18			{
19	3		parent::boot();
20
21			// Auto Validate
22			self::saving(function ($model) {
23			if (!$model->validate()) {
24			return false;
25			}
26	3		});
27
28			// Auto Cache
29			static::saved(function ($model) {
30			$model->refresh();
31	3		}, -1);
			0 ignored issues – show Unused Code introduced 2017-09-13 18:09 UTC by Report Bug Copy Issue Report The call to `Pivot::saved()` has too many arguments starting with `-1`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
32
33	3		static::deleted(function ($model) {
34			$model->refresh();
35	3		}, -1);
			0 ignored issues – show Unused Code introduced 2017-09-13 18:09 UTC by Report Bug Copy Issue Report The call to `Pivot::deleted()` has too many arguments starting with `-1`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
36	3		}
37			}
38

tylercd100 / laravel-model-improved

Branch — master (62cc10)

Pivot::boot() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like