Owner::getEntityIdsQuery() - Code Metrics - Inspection of "Modify Readme, changelog after code fixes accordin..." - itstructure/yii2-multi-format-uploader - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c0984a...820d33 )

by Andrey

created 2018-08-09 09:02 UTC

Owner::getEntityIdsQuery() B

↳ Parent: Owner

Complexity

Conditions	10
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	34
rs	7.6666
c	0
b	0
f	0
cc	10
nc	17
nop	2

How to fix Complexity

<?php

namespace Itstructure\MFUploader\models;

use yii\db\ActiveQuery;
use yii\base\InvalidArgumentException;

/**
 * This is the base class for owners.
 *
 * @property int $ownerId Owner id.
 * @property string $owner Owner name (post, article, page e.t.c.).
 * @property string $ownerAttribute Owner attribute (thumbnail, image e.t.c.).
 *
 * @package Itstructure\MFUploader\models
 *
 * @author Andrey Girnik <[email protected]>
 */
class Owner extends \yii\db\ActiveRecord
{
    /**
     * {@inheritdoc}
     */
    public function rules()
    {
        return [
            [
                [
                    'ownerId',
                    'owner',
                    'ownerAttribute',
                ],
                'required',
            ],
            [
                'ownerId',
                'integer',
            ],
            [
                [
                    'owner',
                    'ownerAttribute',
                ],
                'string',
                'max' => 255,
            ],
        ];
    }

    /**
     * {@inheritdoc}
     */
    public function attributeLabels()
    {
        return [
            'ownerId' => 'Owner ID',
            'owner' => 'Owner',
            'ownerAttribute' => 'Owner Attribute',
        ];
    }

    /**
     * Get Id's by owner.
     *
     * @param string $nameId
     * @param array $args It can be an array of the next params: owner{string}, ownerId{int}, ownerAttribute{string}.
     *
     * @throws InvalidArgumentException
     *
     * @return ActiveQuery
     */
    protected static function getEntityIdsQuery(string $nameId, array $args): ActiveQuery
    {
        $conditions = [];

        if (isset($args['owner'])) {
            if (!is_string($args['owner']) || empty($args['owner'])) {
                throw new InvalidArgumentException('Parameter owner must be a string.');
            }
            $conditions['owner'] = $args['owner'];

            if (isset($args['ownerId'])) {
                if (!is_numeric($args['ownerId'])) {
                    throw new InvalidArgumentException('Parameter ownerId must be numeric.');
                }
                $conditions['ownerId'] = $args['ownerId'];
            }
        }

        if (isset($args['ownerAttribute'])) {
            if (!is_string($args['ownerAttribute']) || empty($args['ownerAttribute'])) {
                throw new InvalidArgumentException('Parameter ownerAttribute must be a string.');
            }
            $conditions['ownerAttribute'] = $args['ownerAttribute'];
        }

        $query = static::find()
            ->select($nameId);

        if (count($conditions) > 0) {
            return $query->where($conditions);
        }

        return $query;
    }
}


1			<?php
2
3			namespace Itstructure\MFUploader\models;
4
5			use yii\db\ActiveQuery;
6			use yii\base\InvalidArgumentException;
7
8			/**
9			* This is the base class for owners.
10			*
11			* @property int $ownerId Owner id.
12			* @property string $owner Owner name (post, article, page e.t.c.).
13			* @property string $ownerAttribute Owner attribute (thumbnail, image e.t.c.).
14			*
15			* @package Itstructure\MFUploader\models
16			*
17			* @author Andrey Girnik <[email protected]>
18			*/
19			class Owner extends \yii\db\ActiveRecord
20			{
21			/**
22			* {@inheritdoc}
23			*/
24			public function rules()
25			{
26			return [
27			[
28			[
29			'ownerId',
30			'owner',
31			'ownerAttribute',
32			],
33			'required',
34			],
35			[
36			'ownerId',
37			'integer',
38			],
39			[
40			[
41			'owner',
42			'ownerAttribute',
43			],
44			'string',
45			'max' => 255,
46			],
47			];
48			}
49
50			/**
51			* {@inheritdoc}
52			*/
53			public function attributeLabels()
54			{
55			return [
56			'ownerId' => 'Owner ID',
57			'owner' => 'Owner',
58			'ownerAttribute' => 'Owner Attribute',
59			];
60			}
61
62			/**
63			* Get Id's by owner.
64			*
65			* @param string $nameId
66			* @param array $args It can be an array of the next params: owner{string}, ownerId{int}, ownerAttribute{string}.
67			*
68			* @throws InvalidArgumentException
69			*
70			* @return ActiveQuery
71			*/
72			protected static function getEntityIdsQuery(string $nameId, array $args): ActiveQuery
73			{
74			$conditions = [];
75
76			if (isset($args['owner'])) {
77			if (!is_string($args['owner']) \|\| empty($args['owner'])) {
78			throw new InvalidArgumentException('Parameter owner must be a string.');
79			}
80			$conditions['owner'] = $args['owner'];
81
82			if (isset($args['ownerId'])) {
83			if (!is_numeric($args['ownerId'])) {
84			throw new InvalidArgumentException('Parameter ownerId must be numeric.');
85			}
86			$conditions['ownerId'] = $args['ownerId'];
87			}
88			}
89
90			if (isset($args['ownerAttribute'])) {
91			if (!is_string($args['ownerAttribute']) \|\| empty($args['ownerAttribute'])) {
92			throw new InvalidArgumentException('Parameter ownerAttribute must be a string.');
93			}
94			$conditions['ownerAttribute'] = $args['ownerAttribute'];
95			}
96
97			$query = static::find()
98			->select($nameId);
99
100			if (count($conditions) > 0) {
101			return $query->where($conditions);
102			}
103
104			return $query;
105			}
106			}
107

itstructure / yii2-multi-format-uploader

Push — master ( c0984a...820d33 )

Owner::getEntityIdsQuery() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like