VolumeTrait - Code Metrics - leroy-merlin-br/laravel-axado-api - Measure and Improve Code Quality continuously with Scrutinizer

VolumeTrait A
last analyzed 2017-09-04 17:49 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

Metric	Value
wmc	1
c	0
b	0
f	0
lcom	0
cbo	0
dl	0
loc	20
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	volumeToArray()	0	12	1

<?php
namespace Axado\Volume;

trait VolumeTrait
{
    /**
     * Returns the instance as array.
     *
     * @return array
     */
    public function volumeToArray(): array
    {
        return [
            'sku' => $this->getSku(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'quantidade' => $this->getQuantity(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'preco' => $this->getPriceUnit(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'altura' => $this->getHeight(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'comprimento' => $this->getLength(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'largura' => $this->getWidth(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            'peso' => $this->getWeight(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        ];
    }
}


1			<?php
2			namespace Axado\Volume;
3
4			trait VolumeTrait
5			{
6			/**
7			* Returns the instance as array.
8			*
9			* @return array
10			*/
11			public function volumeToArray(): array
12			{
13			return [
14			'sku' => $this->getSku(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getSku()` 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...
15			'quantidade' => $this->getQuantity(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getQuantity()` 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...
16			'preco' => $this->getPriceUnit(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getPriceUnit()` 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...
17			'altura' => $this->getHeight(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getHeight()` 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...
18			'comprimento' => $this->getLength(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getLength()` 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...
19			'largura' => $this->getWidth(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getWidth()` 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...
20			'peso' => $this->getWeight(),
			0 ignored issues – show Bug introduced 2017-08-30 20:46 UTC by Report Bug Copy Issue Report It seems like `getWeight()` 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...
21			];
22			}
23			}
24

leroy-merlin-br / laravel-axado-api

VolumeTrait A last analyzed 2017-09-04 17:49 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like

VolumeTrait A
last analyzed 2017-09-04 17:49 UTC