Builder::__get() - Code Metrics - Inspection of "refactor QueryBuilders" - aaemnnosttv/silk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( d77dc3...4e0099 )

by Evan

created 2016-08-15 22:02 UTC

Builder::__get() A

↳ Parent: Builder

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	8
cc	2
eloc	4
nc	2
nop	1
rs	9.4285

<?php

namespace Silk\Query;

use Silk\Type\Model;
use Silk\Contracts\BuildsQueries;
use Illuminate\Support\Collection;

/**
 * @property-read object $query
 * @property-read Model  $model
 */
abstract class Builder implements BuildsQueries
{
    /**
     * The query instance
     * @var object  The query instance
     */
    protected $query;

    /**
     * @var Model|null  The model instance if set, otherwise null
     */
    protected $model;

    /**
     * Execute the query and return the raw results.
     *
     * @return array
     */
    abstract protected function query();

    /**
     * Get the results as a collection
     *
     * @return Collection
     */
    public function results()
    {
        if ($this->model) {
            return $this->collectModels();
        }

        return new Collection($this->query());
    }

    /**
     * Get the results as a collection of model instances.
     *
     * @return Collection
     */
    protected function collectModels()
    {
        $modelClass = get_class($this->model);

        return Collection::make($this->query())
                         ->map(function ($result) use ($modelClass) {
                             return new $modelClass($result);
                         });
    }

    /**
     * Set the model for this query.
     *
     * @param Model $model
     *
     * @return $this
     */
    public function setModel(Model $model)
    {
        $this->model = $model;

        return $this;
    }

    /**
     * Get the model
     *
     * @return Model

     */
    public function getModel()
    {
        return $this->model;
    }

    /**
     * Magic Getter.
     *
     * @param $property
     *
     * @return null
     */
    public function __get($property)
    {
        if (in_array($property, ['query','model'])) {
            return $this->$property;
        }

        return null;
    }
}


1			<?php
2
3			namespace Silk\Query;
4
5			use Silk\Type\Model;
6			use Silk\Contracts\BuildsQueries;
7			use Illuminate\Support\Collection;
8
9			/**
10			* @property-read object $query
11			* @property-read Model $model
12			*/
13			abstract class Builder implements BuildsQueries
14			{
15			/**
16			* The query instance
17			* @var object The query instance
18			*/
19			protected $query;
20
21			/**
22			* @var Model\|null The model instance if set, otherwise null
23			*/
24			protected $model;
25
26			/**
27			* Execute the query and return the raw results.
28			*
29			* @return array
30			*/
31			abstract protected function query();
32
33			/**
34			* Get the results as a collection
35			*
36			* @return Collection
37			*/
38			public function results()
39			{
40			if ($this->model) {
41			return $this->collectModels();
42			}
43
44			return new Collection($this->query());
45			}
46
47			/**
48			* Get the results as a collection of model instances.
49			*
50			* @return Collection
51			*/
52			protected function collectModels()
53			{
54			$modelClass = get_class($this->model);
55
56			return Collection::make($this->query())
57			->map(function ($result) use ($modelClass) {
58			return new $modelClass($result);
59			});
60			}
61
62			/**
63			* Set the model for this query.
64			*
65			* @param Model $model
66			*
67			* @return $this
68			*/
69			public function setModel(Model $model)
70			{
71			$this->model = $model;
72
73			return $this;
74			}
75
76			/**
77			* Get the model
78			*
79			* @return Model
			0 ignored issues – show Documentation introduced 2016-08-15 22:05 UTC by Report Bug Copy Issue Report Should the return type not be `Model\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
80			*/
81			public function getModel()
82			{
83			return $this->model;
84			}
85
86			/**
87			* Magic Getter.
88			*
89			* @param $property
90			*
91			* @return null
92			*/
93			public function __get($property)
94			{
95			if (in_array($property, ['query','model'])) {
96			return $this->$property;
97			}
98
99			return null;
100			}
101			}
102

aaemnnosttv / silk

Push — develop ( d77dc3...4e0099 )

Builder::__get() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like