AggregateRoot::__set() - Code Metrics - C4Tech/laravel-ray-emitter - Measure and Improve Code Quality continuously with Scrutinizer

AggregateRoot::__set() A
last analyzed 2016-04-18 16:59 UTC

↳ Parent: AggregateRoot

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
rs	10
ccs	3
cts	3
cp	1
cc	1
eloc	2
nc	1
nop	2
crap	1

<?php namespace C4tech\RayEmitter\Domain;

trait AggregateRoot // extends C4tech\RayEmitter\Contracts\Domain\AggregateRoot
{
    /**
     * @inheritDoc
     */
    public function makeEntity()
    {
        $class = get_parent_class($this);
        $entity = new $class($this->getId());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $properties = get_object_vars($this);
        foreach ($properties as $key => $value) {
            if ($key === 'identity') {
                continue;
            }

            $entity->set($key, $value);
        }

        return $entity;
    }

    /**
     * @inheritDoc
     */
    public function __set($property, $value)
    {
        $this->set($property, $value);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1		<?php namespace C4tech\RayEmitter\Domain;
2
3		trait AggregateRoot // extends C4tech\RayEmitter\Contracts\Domain\AggregateRoot
4		{
5		/**
6		* @inheritDoc
7		*/
8	1	public function makeEntity()
9		{
10	1	$class = get_parent_class($this);
11	1	$entity = new $class($this->getId());
		0 ignored issues – show Bug introduced 2016-01-06 20:38 UTC by Report Bug Copy Issue Report It seems like `getId()` 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...
12
13	1	$properties = get_object_vars($this);
14	1	foreach ($properties as $key => $value) {
15	1	if ($key === 'identity') {
16	1	continue;
17		}
18
19	1	$entity->set($key, $value);
20	1	}
21
22	1	return $entity;
23		}
24
25		/**
26		* @inheritDoc
27		*/
28	1	public function __set($property, $value)
29		{
30	1	$this->set($property, $value);
		0 ignored issues – show Bug introduced 2016-01-06 20:38 UTC by Report Bug Copy Issue Report It seems like `set()` 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...
31	1	}
32		}
33

C4Tech / laravel-ray-emitter

AggregateRoot::__set() A last analyzed 2016-04-18 16:59 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

AggregateRoot::__set() A
last analyzed 2016-04-18 16:59 UTC