CascadesDeletes::bootCascadesDeletes() - Code Metrics - Inspection of "Updated travisci php versions." - shiftonelabs/laravel-cascade-deletes - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (65080d)

by Patrick

created 2016-12-07 06:07 UTC

CascadesDeletes::bootCascadesDeletes() C

↳ Parent: CascadesDeletes

Complexity

Conditions	9
Paths	1

Size

Total Lines	69
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	28
CRAP Score	9

Importance

Changes

Metric	Value
dl	0
loc	69
ccs	28
cts	28
cp	1
rs	6.2192
c	0
b	0
f	0
cc	9
eloc	26
nc	1
nop	0
crap	9

How to fix Long Method

<?php

namespace ShiftOneLabs\LaravelCascadeDeletes;

use LogicException;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletingScope;
use Illuminate\Database\Eloquent\Relations\Relation;
use Illuminate\Database\Eloquent\Relations\HasOneOrMany;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;

trait CascadesDeletes
{
    /**
     * Use the boot function to setup model event listeners.
     *
     * @return void
     */
    public static function bootCascadesDeletes()
    {
        // Setup the 'deleting' event listener.
        static::deleting(function ($model) {


            // Wrap all of the cascading deletes inside of a transaction to make this an
            // all or nothing operation. Any exceptions thrown inside the transaction
            // need to bubble up to make sure all transactions will be rolled back.
            $model->getConnectionResolver()->transaction(function () use ($model) {

                $relations = $model->getCascadeDeletesRelations();

                if ($invalidRelations = $model->getInvalidCascadeDeletesRelations($relations)) {
                    throw new LogicException(sprintf('[%s]: invalid relationship(s) for cascading deletes. Relationship method(s) [%s] must return an object of type Illuminate\Database\Eloquent\Relations\Relation.', static::class, implode(', ', $invalidRelations)));
                }

                $deleteMethod = $model->isCascadeDeletesForceDeleting() ? 'forceDelete' : 'delete';

                foreach ($relations as $relationName => $relation) {
                    $expected = 0;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
                    $deleted = 0;

                    if ($relation instanceof BelongsToMany) {
                        // Process the many-to-many relationships on the model.
                        // These relationships should not delete the related
                        // record, but should just detach from each other.

                        $expected = $model->getCascadeDeletesRelationQuery($relationName)->count();

                        $deleted = $model->getCascadeDeletesRelationQuery($relationName)->detach();
                    } elseif ($relation instanceof HasOneOrMany) {
                        // Process the one-to-one and one-to-many relationships
                        // on the model. These relationships should actually
                        // delete the related records from the database.

                        $children = $model->getCascadeDeletesRelationQuery($relationName)->get();

                        // To protect against potential relationship defaults,
                        // filter out any children that may not actually be
                        // Model instances, or that don't actually exist.
                        $children = $children->filter(function ($child) {
                            return $child instanceof Model && $child->exists;
                        })->all();

                        $expected = count($children);

                        foreach ($children as $child) {
                            // Delete the record using the proper method.
                            $child->$deleteMethod();

                            // forceDelete doesn't return anything until Laravel 5.2. Check
                            // exists property to determine if the delete was successful
                            // since that is the last thing set before delete returns.
                            $deleted += !$child->exists;
                        }
                    } else {
                        // Not all relationship types make sense for cascading. As an
                        // example, for a BelongsTo relationship, it does not make
                        // sense to delete the parent when the child is deleted.
                        throw new LogicException(sprintf('[%s]: error occurred deleting [%s]. Relation type [%s] not handled.', static::class, $relationName, get_class($relation)));
                    }

                    if ($deleted < $expected) {
                        throw new LogicException(sprintf('[%s]: error occurred deleting [%s]. Only deleted [%d] out of [%d] records.', static::class, $relationName, $deleted, $expected));
                    }
                }
            });
        });
    }

    /**
     * Get the value of the cascadeDeletes attribute, if it exists.
     *
     * @return mixed
     */
    public function getCascadeDeletes()
    {
        return property_exists($this, 'cascadeDeletes') ? $this->cascadeDeletes : [];
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * Set the cascadeDeletes attribute.
     *
     * @param  mixed  $cascadeDeletes
     *
     * @return void
     */
    public function setCascadeDeletes($cascadeDeletes)
    {
        $this->cascadeDeletes = $cascadeDeletes;
    }

    /**
     * Get an array of cascading relation names.
     *
     * @return array
     */
    public function getCascadeDeletesRelationNames()
    {
        $deletes = $this->getCascadeDeletes();

        return array_filter(is_array($deletes) ? $deletes : [ $deletes ]);
    }

    /**
     * Get an array of the cascading relation names mapped to their relation types.
     *
     * @return array
     */
    public function getCascadeDeletesRelations()
    {
        $names = $this->getCascadeDeletesRelationNames();

        return array_combine($names, array_map(function ($name) {
            $relation = method_exists($this, $name) ? $this->$name() : null;

            return $relation instanceof Relation ? $relation : null;
        }, $names));
    }

    /**
     * Get an array of the invalid cascading relation names.
     *
     * @param  array|null  $relations
     *
     * @return array
     */
    public function getInvalidCascadeDeletesRelations(array $relations = null)
    {
        return array_keys($relations ?: $this->getCascadeDeletesRelations(), null);
    }

    /**
     * Get the relationship query to use for the specified relation.
     *
     * @param  string  $relation
     *
     * @return \Illuminate\Database\Eloquent\Relations\Relation
     */
    public function getCascadeDeletesRelationQuery($relation)
    {
        $query = $this->$relation();

        // If this is a force delete and the related model is using soft deletes,
        // we need to use the withTrashed() scope on the relationship query to
        // ensure all related records, plus soft deleted, are force deleted.
        if ($this->isCascadeDeletesForceDeleting()) {
            // Laravel 4.1 has the withTrashed method directly on the Eloquent
            // query builder, however Laravel 4.2+ uses a withTrashed macro
            // on the Eloquent query builder.
            if (!class_exists(SoftDeletingScope::class) || !is_null($query->getMacro('withTrashed'))) {
                $query = $query->withTrashed();
            }
        }

        return $query;
    }

    /**
     * Check if this cascading delete is a force delete.
     *
     * @return boolean
     */
    public function isCascadeDeletesForceDeleting()
    {
        // Laravel 4.1 uses the softDelete property, and the only
        // indication that the delete should be forced is when
        // the softDelete property is set to false.
        if (property_exists($this, 'softDelete') && !class_exists(SoftDeletingScope::class)) {
            return !$this->softDelete;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        }

        return property_exists($this, 'forceDeleting') && $this->forceDeleting;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }
}


1		<?php
2
3		namespace ShiftOneLabs\LaravelCascadeDeletes;
4
5		use LogicException;
6		use Illuminate\Database\Eloquent\Model;
7		use Illuminate\Database\Eloquent\SoftDeletingScope;
8		use Illuminate\Database\Eloquent\Relations\Relation;
9		use Illuminate\Database\Eloquent\Relations\HasOneOrMany;
10		use Illuminate\Database\Eloquent\Relations\BelongsToMany;
11
12		trait CascadesDeletes
13		{
14		/**
15		* Use the boot function to setup model event listeners.
16		*
17		* @return void
18		*/
19	16	public static function bootCascadesDeletes()
20		{
21		// Setup the 'deleting' event listener.
22		static::deleting(function ($model) {
		1 ignored issue – show Bug introduced 2016-12-07 06:11 UTC by Report Bug Copy Issue Report The method `deleting()` does not exist on `ShiftOneLabs\LaravelCascadeDeletes\CascadesDeletes`. Did you maybe mean `isCascadeDeletesForceDeleting()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
23
24		// Wrap all of the cascading deletes inside of a transaction to make this an
25		// all or nothing operation. Any exceptions thrown inside the transaction
26		// need to bubble up to make sure all transactions will be rolled back.
27		$model->getConnectionResolver()->transaction(function () use ($model) {
28
29	16	$relations = $model->getCascadeDeletesRelations();
30
31	16	if ($invalidRelations = $model->getInvalidCascadeDeletesRelations($relations)) {
32	1	throw new LogicException(sprintf('[%s]: invalid relationship(s) for cascading deletes. Relationship method(s) [%s] must return an object of type Illuminate\Database\Eloquent\Relations\Relation.', static::class, implode(', ', $invalidRelations)));
33		}
34
35	15	$deleteMethod = $model->isCascadeDeletesForceDeleting() ? 'forceDelete' : 'delete';
36
37	15	foreach ($relations as $relationName => $relation) {
38	15	$expected = 0;
		1 ignored issue – show Unused Code introduced 2016-12-07 06:11 UTC by Report Bug Copy Issue Report `$expected` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
39	15	$deleted = 0;
40
41	15	if ($relation instanceof BelongsToMany) {
42		// Process the many-to-many relationships on the model.
43		// These relationships should not delete the related
44		// record, but should just detach from each other.
45
46	13	$expected = $model->getCascadeDeletesRelationQuery($relationName)->count();
47
48	13	$deleted = $model->getCascadeDeletesRelationQuery($relationName)->detach();
49	15	} elseif ($relation instanceof HasOneOrMany) {
50		// Process the one-to-one and one-to-many relationships
51		// on the model. These relationships should actually
52		// delete the related records from the database.
53
54	14	$children = $model->getCascadeDeletesRelationQuery($relationName)->get();
55
56		// To protect against potential relationship defaults,
57		// filter out any children that may not actually be
58		// Model instances, or that don't actually exist.
59		$children = $children->filter(function ($child) {
60	14	return $child instanceof Model && $child->exists;
61	14	})->all();
62
63	14	$expected = count($children);
64
65	14	foreach ($children as $child) {
66		// Delete the record using the proper method.
67	14	$child->$deleteMethod();
68
69		// forceDelete doesn't return anything until Laravel 5.2. Check
70		// exists property to determine if the delete was successful
71		// since that is the last thing set before delete returns.
72	14	$deleted += !$child->exists;
73	14	}
74	14	} else {
75		// Not all relationship types make sense for cascading. As an
76		// example, for a BelongsTo relationship, it does not make
77		// sense to delete the parent when the child is deleted.
78	4	throw new LogicException(sprintf('[%s]: error occurred deleting [%s]. Relation type [%s] not handled.', static::class, $relationName, get_class($relation)));
79		}
80
81	14	if ($deleted < $expected) {
82	1	throw new LogicException(sprintf('[%s]: error occurred deleting [%s]. Only deleted [%d] out of [%d] records.', static::class, $relationName, $deleted, $expected));
83		}
84	13	}
85	16	});
86	16	});
87	16	}
88
89		/**
90		* Get the value of the cascadeDeletes attribute, if it exists.
91		*
92		* @return mixed
93		*/
94	34	public function getCascadeDeletes()
95		{
96	34	return property_exists($this, 'cascadeDeletes') ? $this->cascadeDeletes : [];
		0 ignored issues – show Bug introduced 2016-12-07 06:11 UTC by Report Bug Copy Issue Report The property `cascadeDeletes` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
97		}
98
99		/**
100		* Set the cascadeDeletes attribute.
101		*
102		* @param mixed $cascadeDeletes
103		*
104		* @return void
105		*/
106	15	public function setCascadeDeletes($cascadeDeletes)
107		{
108	15	$this->cascadeDeletes = $cascadeDeletes;
109	15	}
110
111		/**
112		* Get an array of cascading relation names.
113		*
114		* @return array
115		*/
116	32	public function getCascadeDeletesRelationNames()
117		{
118	32	$deletes = $this->getCascadeDeletes();
119
120	32	return array_filter(is_array($deletes) ? $deletes : [ $deletes ]);
121		}
122
123		/**
124		* Get an array of the cascading relation names mapped to their relation types.
125		*
126		* @return array
127		*/
128	21	public function getCascadeDeletesRelations()
129		{
130	21	$names = $this->getCascadeDeletesRelationNames();
131
132	21	return array_combine($names, array_map(function ($name) {
133	21	$relation = method_exists($this, $name) ? $this->$name() : null;
134
135	21	return $relation instanceof Relation ? $relation : null;
136	21	}, $names));
137		}
138
139		/**
140		* Get an array of the invalid cascading relation names.
141		*
142		* @param array\|null $relations
143		*
144		* @return array
145		*/
146	18	public function getInvalidCascadeDeletesRelations(array $relations = null)
147		{
148	18	return array_keys($relations ?: $this->getCascadeDeletesRelations(), null);
149		}
150
151		/**
152		* Get the relationship query to use for the specified relation.
153		*
154		* @param string $relation
155		*
156		* @return \Illuminate\Database\Eloquent\Relations\Relation
157		*/
158	17	public function getCascadeDeletesRelationQuery($relation)
159		{
160	17	$query = $this->$relation();
161
162		// If this is a force delete and the related model is using soft deletes,
163		// we need to use the withTrashed() scope on the relationship query to
164		// ensure all related records, plus soft deleted, are force deleted.
165	17	if ($this->isCascadeDeletesForceDeleting()) {
166		// Laravel 4.1 has the withTrashed method directly on the Eloquent
167		// query builder, however Laravel 4.2+ uses a withTrashed macro
168		// on the Eloquent query builder.
169	5	if (!class_exists(SoftDeletingScope::class) \|\| !is_null($query->getMacro('withTrashed'))) {
170	5	$query = $query->withTrashed();
171	5	}
172	5	}
173
174	17	return $query;
175		}
176
177		/**
178		* Check if this cascading delete is a force delete.
179		*
180		* @return boolean
181		*/
182	20	public function isCascadeDeletesForceDeleting()
183		{
184		// Laravel 4.1 uses the softDelete property, and the only
185		// indication that the delete should be forced is when
186		// the softDelete property is set to false.
187	20	if (property_exists($this, 'softDelete') && !class_exists(SoftDeletingScope::class)) {
188		return !$this->softDelete;
		0 ignored issues – show Bug introduced 2016-12-07 06:11 UTC by Report Bug Copy Issue Report The property `softDelete` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
189		}
190
191	20	return property_exists($this, 'forceDeleting') && $this->forceDeleting;
		0 ignored issues – show Bug introduced 2016-12-07 06:11 UTC by Report Bug Copy Issue Report The property `forceDeleting` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
192		}
193		}
194

shiftonelabs / laravel-cascade-deletes

Branch — master (65080d)

CascadesDeletes::bootCascadesDeletes() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like