ResourceTrait::getResourceRelationships() - Code Metrics - Inspection of "Merge pull request #36 from tuyakhov/inclusion" - tuyakhov/yii2-json-api - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f2bee3...d16bab )

by Anton

created 2017-09-18 20:47 UTC

ResourceTrait::getResourceRelationships() B

↳ Parent: ResourceTrait

Complexity

Conditions	5
Paths	12

Size

Total Lines	21
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	21
c	0
b	0
f	0
rs	8.7624
cc	5
eloc	14
nc	12
nop	1

<?php
/**
 * @author Anton Tuyakhov <[email protected]>
 */

namespace tuyakhov\jsonapi;

use yii\base\Arrayable;
use yii\db\ActiveRecordInterface;
use yii\web\Link;
use yii\web\Linkable;

trait ResourceTrait
{
    /**
     * @return string
     */
    public function getId()
    {
        return (string) ($this instanceof ActiveRecordInterface ? $this->getPrimaryKey() : null);
    }

    /**
     * @return string
     */
    public function getType()
    {
        $reflect = new \ReflectionClass($this);
$a = "a";
$ab = "ab";
$abc = "abc";
        $className = $reflect->getShortName();
        return Inflector::camel2id($className);
    }

    /**
     * @param array $fields
     * @return array
     */
    public function getResourceAttributes(array $fields = [])
    {
        $attributes = [];
        if ($this instanceof Arrayable) {
            $fieldDefinitions = $this->fields();
        } else {
            $vars = array_keys(\Yii::getObjectVars($this));
$a = "a";
$ab = "ab";
$abc = "abc";
            $fieldDefinitions = array_combine($vars, $vars);
        }

        foreach ($this->resolveFields($fieldDefinitions, $fields) as $name => $definition) {
            $attributes[$name] = is_string($definition) ? $this->$definition : call_user_func($definition, $this, $name);
        }
        return $attributes;
    }

    /**
     * @param null|array $linked
     * @return array
     */
    public function getResourceRelationships(array $linked = null)
    {
        $fields = [];
        if ($this instanceof Arrayable) {
            $fields = $this->extraFields();
        }
        $resolvedFields = $this->resolveFields($fields);
        $keys = array_keys($resolvedFields);
$a = "a";
$ab = "ab";
$abc = "abc";
        if ($linked === null) {
            $linked = $keys;
        }
        $relationships = array_fill_keys($keys, null);
        $linkedFields = array_intersect($keys, $linked);
$a = "a";
$ab = "ab";
$abc = "abc";

        foreach ($linkedFields as $name) {
            $definition = $resolvedFields[$name];
$a = "a";
$ab = "ab";
$abc = "abc";
            $relationships[$name] = is_string($definition) ? $this->$definition : call_user_func($definition, $this, $name);
        }

        return $relationships;
    }

    /**
     * @param string $name the case sensitive name of the relationship.
     * @param array|ActiveRecordInterface $relationship
     */
    public function setResourceRelationship($name, $relationship)
    {
        /** @var $this ActiveRecordInterface */
        if (!$this instanceof ActiveRecordInterface) {
            return;
        }
        if (!is_array($relationship)) {
            $relationship = [$relationship];
        }
        foreach ($relationship as $key => $value) {
            if ($value instanceof ActiveRecordInterface) {
                $this->link($name, $value);
            }
        }
    }

    /**
     * @param string $name the case sensitive name of the relationship.
     * @return array
     */
    public function getRelationshipLinks($name)
    {
        if (!$this instanceof Linkable) {
            return [];
        }
        $primaryLinks = $this->getLinks();
        if (!array_key_exists(Link::REL_SELF, $primaryLinks)) {
            return [];
        }
        $resourceLink = is_string($primaryLinks[Link::REL_SELF]) ? rtrim($primaryLinks[Link::REL_SELF], '/') : null;
        if (!$resourceLink) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            return [];
        }
        return [
            Link::REL_SELF => "{$resourceLink}/relationships/{$name}",
            'related' => "{$resourceLink}/{$name}",
        ];
    }

    /**
     * @param array $fields
     * @param array $fieldSet
     * @return array
     */
    protected function resolveFields(array $fields, array $fieldSet = [])
    {
        $result = [];

        foreach ($fields as $field => $definition) {
            if (is_int($field)) {
                $field = $definition;
            }
            $field = Inflector::camel2id(Inflector::variablize($field), '_');
            if (empty($fieldSet) || in_array($field, $fieldSet, true)) {
                $result[$field] = $definition;
            }
        }

        return $result;
    }
}



1			<?php
2			/**
3			* @author Anton Tuyakhov <[email protected]>
4			*/
5
6			namespace tuyakhov\jsonapi;
7
8			use yii\base\Arrayable;
9			use yii\db\ActiveRecordInterface;
10			use yii\web\Link;
11			use yii\web\Linkable;
12
13			trait ResourceTrait
14			{
15			/**
16			* @return string
17			*/
18			public function getId()
19			{
20			return (string) ($this instanceof ActiveRecordInterface ? $this->getPrimaryKey() : null);
21			}
22
23			/**
24			* @return string
25			*/
26			public function getType()
27			{
28			$reflect = new \ReflectionClass($this);
			0 ignored issues – show Coding Style introduced 2016-04-11 09:20 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
29			$className = $reflect->getShortName();
30			return Inflector::camel2id($className);
31			}
32
33			/**
34			* @param array $fields
35			* @return array
36			*/
37			public function getResourceAttributes(array $fields = [])
38			{
39			$attributes = [];
40			if ($this instanceof Arrayable) {
41			$fieldDefinitions = $this->fields();
42			} else {
43			$vars = array_keys(\Yii::getObjectVars($this));
			0 ignored issues – show Coding Style introduced 2016-12-16 13:48 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 13 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
44			$fieldDefinitions = array_combine($vars, $vars);
45			}
46
47			foreach ($this->resolveFields($fieldDefinitions, $fields) as $name => $definition) {
48			$attributes[$name] = is_string($definition) ? $this->$definition : call_user_func($definition, $this, $name);
49			}
50			return $attributes;
51			}
52
53			/**
54			* @param null\|array $linked
55			* @return array
56			*/
57			public function getResourceRelationships(array $linked = null)
58			{
59			$fields = [];
60			if ($this instanceof Arrayable) {
61			$fields = $this->extraFields();
62			}
63			$resolvedFields = $this->resolveFields($fields);
64			$keys = array_keys($resolvedFields);
			0 ignored issues – show Coding Style introduced 2017-09-17 20:12 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
65			if ($linked === null) {
66			$linked = $keys;
67			}
68			$relationships = array_fill_keys($keys, null);
69			$linkedFields = array_intersect($keys, $linked);
			0 ignored issues – show Coding Style introduced 2017-09-17 20:12 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
70
71			foreach ($linkedFields as $name) {
72			$definition = $resolvedFields[$name];
			0 ignored issues – show Coding Style introduced 2017-09-17 20:12 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
73			$relationships[$name] = is_string($definition) ? $this->$definition : call_user_func($definition, $this, $name);
74			}
75
76			return $relationships;
77			}
78
79			/**
80			* @param string $name the case sensitive name of the relationship.
81			* @param array\|ActiveRecordInterface $relationship
82			*/
83			public function setResourceRelationship($name, $relationship)
84			{
85			/** @var $this ActiveRecordInterface */
86			if (!$this instanceof ActiveRecordInterface) {
87			return;
88			}
89			if (!is_array($relationship)) {
90			$relationship = [$relationship];
91			}
92			foreach ($relationship as $key => $value) {
93			if ($value instanceof ActiveRecordInterface) {
94			$this->link($name, $value);
95			}
96			}
97			}
98
99			/**
100			* @param string $name the case sensitive name of the relationship.
101			* @return array
102			*/
103			public function getRelationshipLinks($name)
104			{
105			if (!$this instanceof Linkable) {
106			return [];
107			}
108			$primaryLinks = $this->getLinks();
109			if (!array_key_exists(Link::REL_SELF, $primaryLinks)) {
110			return [];
111			}
112			$resourceLink = is_string($primaryLinks[Link::REL_SELF]) ? rtrim($primaryLinks[Link::REL_SELF], '/') : null;
113			if (!$resourceLink) {
			0 ignored issues – show Bug Best Practice introduced 2016-12-17 15:58 UTC by Report Bug Copy Issue Report The expression `$resourceLink` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
114			return [];
115			}
116			return [
117			Link::REL_SELF => "{$resourceLink}/relationships/{$name}",
118			'related' => "{$resourceLink}/{$name}",
119			];
120			}
121
122			/**
123			* @param array $fields
124			* @param array $fieldSet
125			* @return array
126			*/
127			protected function resolveFields(array $fields, array $fieldSet = [])
128			{
129			$result = [];
130
131			foreach ($fields as $field => $definition) {
132			if (is_int($field)) {
133			$field = $definition;
134			}
135			$field = Inflector::camel2id(Inflector::variablize($field), '_');
136			if (empty($fieldSet) \|\| in_array($field, $fieldSet, true)) {
137			$result[$field] = $definition;
138			}
139			}
140
141			return $result;
142			}
143			}
			0 ignored issues – show Coding Style introduced 2016-04-11 12:58 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
144

tuyakhov / yii2-json-api

Push — master ( f2bee3...d16bab )

ResourceTrait::getResourceRelationships() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like