IncludesRelations::compileRelations() - Code Metrics - Inspection of "fixed travis build" - deviouspk/larapi - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b1742c...4b0d05 )

by Arthur

created 2018-11-15 20:19 UTC

IncludesRelations::compileRelations() A

↳ Parent: IncludesRelations

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php
/**
 * Created by PhpStorm.
 * User: arthur
 * Date: 04.11.18
 * Time: 19:59.
 */

namespace Foundation\Abstracts\Transformers;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Http\Resources\Json\JsonResource;

/**
 * Trait IncludesRelations.
 */
trait IncludesRelations
{

    /**
     * @return array
     */
    protected function parseRequestIncludeParameter()
    {
        $request = request();
        if (isset($request->include) && is_string($request->include)) {
            return explode(',', $request->include);
        }

        return [];
    }

    /**
     * @return array
     */
    public function compileRelations()
    {
        return array_unique(array_merge($this->include, array_intersect($this->availableIncludes, $this->parseRequestIncludeParameter())));
    }

    /**
     * @param $relation
     *
     * @return $this
     */
    public function include($relation)
    {
        if (is_array($relation)) {
            $this->include = array_merge($this->include, $relation);

        } else {
            $this->include[] = $relation;
        }

        return $this;
    }
}


1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: arthur
5			* Date: 04.11.18
6			* Time: 19:59.
7			*/
8
9			namespace Foundation\Abstracts\Transformers;
10
11			use Illuminate\Database\Eloquent\Model;
12			use Illuminate\Http\Resources\Json\JsonResource;
13
14			/**
15			* Trait IncludesRelations.
16			*/
17			trait IncludesRelations
18			{
19
20			/**
21			* @return array
22			*/
23			protected function parseRequestIncludeParameter()
24			{
25			$request = request();
26			if (isset($request->include) && is_string($request->include)) {
27			return explode(',', $request->include);
28			}
29
30			return [];
31			}
32
33			/**
34			* @return array
35			*/
36			public function compileRelations()
37			{
38			return array_unique(array_merge($this->include, array_intersect($this->availableIncludes, $this->parseRequestIncludeParameter())));
39			}
40
41			/**
42			* @param $relation
43			*
44			* @return $this
45			*/
46			public function include($relation)
47			{
48			if (is_array($relation)) {
49			$this->include = array_merge($this->include, $relation);
			0 ignored issues – show Bug Best Practice introduced 2018-11-15 22:00 UTC by Report Bug Copy Issue Report The property `include` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
50			} else {
51			$this->include[] = $relation;
52			}
53
54			return $this;
55			}
56			}
57

deviouspk / larapi

Push — master ( b1742c...4b0d05 )

IncludesRelations::compileRelations() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like