UserList::resolveElement() - Code Metrics - Inspection of "field value returns a `RelationshipInterface` inst..." - flipboxfactory/craft-element-lists - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( bb979b...2f50d2 )

by Nate

created 2019-06-14 05:29 UTC

UserList::resolveElement() A

↳ Parent: UserList

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	0
cts	10
cp	0
rs	9.8666
c	0
b	0
f	0
cc	3
nc	3
nop	1
crap	12

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://github.com/flipboxfactory/craft-element-lists/LICENSE
 * @link       https://github.com/flipboxfactory/craft-element-lists/
 */

namespace flipbox\craft\element\lists\fields;

use craft\elements\User;
use craft\fields\Users;
use flipbox\craft\element\lists\ElementList;

/**
 * @author Flipbox Factory <[email protected]>
 * @since 1.0.0
 */
class UserList extends Users implements SortableInterface, RelationalInterface
{
    use ElementListTrait;

    /**
     * @inheritdoc
     */
    public function init()
    {
        parent::init();

        $this->settingsTemplate = 'element-lists/_components/fieldtypes/settings';
        $this->inputTemplate = 'element-lists/_components/fieldtypes/input';
        $this->inputJsClass = 'Craft.NestedElementIndexSelectInput';
    }

    /**
     * @inheritdoc
     */
    public static function displayName(): string
    {
        return ElementList::t('User List');
    }

    /**
     * @inheritdoc
     */
    public function settingsAttributes(): array
    {
        return array_merge(
            parent::settingsAttributes(),
            [
                'sortable'
            ]
        );
    }

    /**
     * inheritDoc
     * @return User|null
     */
    public function resolveElement($element)
    {
        if (is_numeric($element)) {
            return \Craft::$app->getUsers()->getUserById($element);
        }

        if (is_string($element)) {
            return \Craft::$app->getUsers()->getUserByUsernameOrEmail($element);
        }

        return User::findOne($element);

    }
}


1			<?php
2
3			/**
4			* @copyright Copyright (c) Flipbox Digital Limited
5			* @license https://github.com/flipboxfactory/craft-element-lists/LICENSE
6			* @link https://github.com/flipboxfactory/craft-element-lists/
7			*/
8
9			namespace flipbox\craft\element\lists\fields;
10
11			use craft\elements\User;
12			use craft\fields\Users;
13			use flipbox\craft\element\lists\ElementList;
14
15			/**
16			* @author Flipbox Factory <[email protected]>
17			* @since 1.0.0
18			*/
19			class UserList extends Users implements SortableInterface, RelationalInterface
20			{
21			use ElementListTrait;
22
23			/**
24			* @inheritdoc
25			*/
26			public function init()
27			{
28			parent::init();
29
30			$this->settingsTemplate = 'element-lists/_components/fieldtypes/settings';
31			$this->inputTemplate = 'element-lists/_components/fieldtypes/input';
32			$this->inputJsClass = 'Craft.NestedElementIndexSelectInput';
33			}
34
35			/**
36			* @inheritdoc
37			*/
38			public static function displayName(): string
39			{
40			return ElementList::t('User List');
41			}
42
43			/**
44			* @inheritdoc
45			*/
46			public function settingsAttributes(): array
47			{
48			return array_merge(
49			parent::settingsAttributes(),
50			[
51			'sortable'
52			]
53			);
54			}
55
56			/**
57			* inheritDoc
58			* @return User\|null
59			*/
60			public function resolveElement($element)
61			{
62			if (is_numeric($element)) {
63			return \Craft::$app->getUsers()->getUserById($element);
64			}
65
66			if (is_string($element)) {
67			return \Craft::$app->getUsers()->getUserByUsernameOrEmail($element);
68			}
69
70			return User::findOne($element);
			0 ignored issues – show Bug Compatibility introduced 2019-06-14 05:39 UTC by Report Bug Copy Issue Report The expression `\craft\elements\User::findOne($element);` of type `craft\base\Element\|null\|craft\base\Element[]` adds the type `craft\base\Element[]` to the return on line 70 which is incompatible with the return type declared by the interface `flipbox\craft\element\li...terface::resolveElement` of type `craft\base\ElementInterface\|null`. Loading history...
71			}
72			}
73

flipboxfactory / craft-element-lists

Push — develop ( bb979b...2f50d2 )

UserList::resolveElement() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like