Associate::run() - Code Metrics - Inspection of "2.0.0 changelog" - flipboxfactory/craft-element-lists - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a20c7b...d885ca )

by Nate

created 2019-03-26 22:58 UTC

Associate::run() B

↳ Parent: Associate

Complexity

Conditions	6
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

Metric	Value
dl	0
loc	39
ccs	0
cts	30
cp	0
rs	8.6737
c	0
b	0
f	0
cc	6
nc	4
nop	5
crap	42

<?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\actions\source;

use craft\base\Field;
use flipbox\craft\ember\actions\ManageTrait;
use flipbox\craft\ember\helpers\SiteHelper;
use flipbox\craft\element\lists\actions\ResolverTrait;
use flipbox\craft\element\lists\records\Association;
use yii\base\Action;

/**
 * @author Flipbox Factory <[email protected]>
 * @since 2.0.0
 */
class Associate extends Action
{
    use ManageTrait,
        ResolverTrait;

    /**
     * @param string $field
     * @param string $source
     * @param string $target
     * @param int|null $siteId
     * @param int|null $sortOrder
     * @return mixed
     * @throws \yii\web\HttpException
     */
    public function run(
        string $field,
        string $source,
        string $target,
        int $siteId = null,
        int $sortOrder = null
    ) {

        // Resolve
        $field = $this->resolveField($field);
        $source = $this->resolveElement($source);
        $target = $this->resolveElement($target);

        /** @var Field $field */

        $siteId = SiteHelper::ensureSiteId($siteId ?: $source->siteId);

        $record = Association::find()
            ->fieldId($field->id)
            ->sourceId($source->getId() ?: false)
            ->targetId($target->getId() ?: false)
            ->siteId($siteId)
            ->one();

        if (!$record) {
            $record = new Association([
                'fieldId' => $field->id,
                'sourceId' => $source->getId(),
                'targetId' => $target->getId(),
                'siteId' => $siteId
            ]);
        }

        if ($sortOrder) {
            $record->sortOrder = $sortOrder;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        }

        return $this->runInternal($record);
    }

    /**
     * @inheritdoc
     * @param Association $record
     * @throws \Exception
     */
    protected function performAction(Association $record): bool
    {
        return $record->save();
    }
}


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\actions\source;
10
11			use craft\base\Field;
12			use flipbox\craft\ember\actions\ManageTrait;
13			use flipbox\craft\ember\helpers\SiteHelper;
14			use flipbox\craft\element\lists\actions\ResolverTrait;
15			use flipbox\craft\element\lists\records\Association;
16			use yii\base\Action;
17
18			/**
19			* @author Flipbox Factory <[email protected]>
20			* @since 2.0.0
21			*/
22			class Associate extends Action
23			{
24			use ManageTrait,
25			ResolverTrait;
26
27			/**
28			* @param string $field
29			* @param string $source
30			* @param string $target
31			* @param int\|null $siteId
32			* @param int\|null $sortOrder
33			* @return mixed
34			* @throws \yii\web\HttpException
35			*/
36			public function run(
37			string $field,
38			string $source,
39			string $target,
40			int $siteId = null,
41			int $sortOrder = null
42			) {
43
44			// Resolve
45			$field = $this->resolveField($field);
46			$source = $this->resolveElement($source);
47			$target = $this->resolveElement($target);
48
49			/** @var Field $field */
50
51			$siteId = SiteHelper::ensureSiteId($siteId ?: $source->siteId);
52
53			$record = Association::find()
54			->fieldId($field->id)
55			->sourceId($source->getId() ?: false)
56			->targetId($target->getId() ?: false)
57			->siteId($siteId)
58			->one();
59
60			if (!$record) {
61			$record = new Association([
62			'fieldId' => $field->id,
63			'sourceId' => $source->getId(),
64			'targetId' => $target->getId(),
65			'siteId' => $siteId
66			]);
67			}
68
69			if ($sortOrder) {
70			$record->sortOrder = $sortOrder;
			0 ignored issues – show Documentation introduced 2019-03-26 22:38 UTC by Report Bug Copy Issue Report The property `sortOrder` does not exist on `object<flipbox\craft\ele...ts\records\Association>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
71			}
72
73			return $this->runInternal($record);
74			}
75
76			/**
77			* @inheritdoc
78			* @param Association $record
79			* @throws \Exception
80			*/
81			protected function performAction(Association $record): bool
82			{
83			return $record->save();
84			}
85			}
86

flipboxfactory / craft-element-lists

Push — master ( a20c7b...d885ca )

Associate::run() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like