EquipmentRepository - Code Metrics - Inspection of "fixes" - ArthurGuy/BBMembershipSystem - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7e4a8d...1527e7 )

by Arthur

created 2018-02-03 16:53 UTC

EquipmentRepository A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
wmc	8
c	1
b	1
f	0
lcom	1
cbo	3
dl	0
loc	58
rs	10

7 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	all()	4	1
A	getRequiresInduction()	4	1
A	getDoesntRequireInduction()	4	1
A	allPaid()	4	1
A	findBySLug()	8	2
A	findByDeviceKey()	4	1

<?php namespace BB\Repo;

use BB\Entities\Equipment;
use Illuminate\Database\Eloquent\ModelNotFoundException;

class EquipmentRepository extends DBRepository
{

    /**
     * @var Equipment
     */
    protected $model;

    public function __construct(Equipment $model)
    {
        $this->model = $model;
    }

    public function all()
    {
        return $this->model->all();
    }

    public function getRequiresInduction()
    {
        return $this->model->where('requires_induction', true)->get();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    public function getDoesntRequireInduction()
    {
        return $this->model->where('requires_induction', false)->get();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }


    public function allPaid()
    {
        return $this->model->where('access_fee', '!=', 0)->get();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    /**
     * Return a device by its slug
     * @param $slug
     * @return Equipment
     */
    public function findBySLug($slug)
    {
        $record = $this->model->where('slug', $slug)->first();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        if ($record) {
            return $record;
        }
        throw new ModelNotFoundException();
    }

    /**
     * Return a device by its slug
     * @param $slug
     * @return Equipment
     */
    public function findByDeviceKey($device)
    {
        return $this->model->where('device_key', $device)->firstOrFail();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }
} 


1			<?php namespace BB\Repo;
2
3			use BB\Entities\Equipment;
4			use Illuminate\Database\Eloquent\ModelNotFoundException;
5
6			class EquipmentRepository extends DBRepository
7			{
8
9			/**
10			* @var Equipment
11			*/
12			protected $model;
13
14			public function __construct(Equipment $model)
15			{
16			$this->model = $model;
17			}
18
19			public function all()
20			{
21			return $this->model->all();
22			}
23
24			public function getRequiresInduction()
25			{
26			return $this->model->where('requires_induction', true)->get();
			0 ignored issues – show Documentation Bug introduced 2015-11-23 19:55 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<BB\Entities\Equipment>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
27			}
28
29			public function getDoesntRequireInduction()
30			{
31			return $this->model->where('requires_induction', false)->get();
			0 ignored issues – show Documentation Bug introduced 2015-11-23 19:55 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<BB\Entities\Equipment>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
32			}
33
34
35			public function allPaid()
36			{
37			return $this->model->where('access_fee', '!=', 0)->get();
			0 ignored issues – show Documentation Bug introduced 2015-11-23 19:55 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<BB\Entities\Equipment>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
38			}
39
40			/**
41			* Return a device by its slug
42			* @param $slug
43			* @return Equipment
44			*/
45			public function findBySLug($slug)
46			{
47			$record = $this->model->where('slug', $slug)->first();
			0 ignored issues – show Documentation Bug introduced 2015-11-23 19:55 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<BB\Entities\Equipment>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
48			if ($record) {
49			return $record;
50			}
51			throw new ModelNotFoundException();
52			}
53
54			/**
55			* Return a device by its slug
56			* @param $slug
57			* @return Equipment
58			*/
59			public function findByDeviceKey($device)
60			{
61			return $this->model->where('device_key', $device)->firstOrFail();
			0 ignored issues – show Documentation Bug introduced 2018-02-03 16:56 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<BB\Entities\Equipment>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
62			}
63			}
64

ArthurGuy / BBMembershipSystem

Push — master ( 7e4a8d...1527e7 )

EquipmentRepository A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

Duplication Side-by-Side

Filter issues like