Dissociate::performAction() - Code Metrics - Inspection of "non interface attribute" - flipboxfactory/craft-element-lists - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (17e8cf)

by Nate

created 2019-03-24 15:38 UTC

Dissociate::performAction() A

↳ Parent: Dissociate

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	4
cp	0
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	2

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://www.flipboxfactory.com/software/element-lists/license
 * @link       https://www.flipboxfactory.com/software/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 Dissociate extends Action
{
    use ManageTrait,
        ResolverTrait;

    /**
     * @var int
     */
    public $statusCodeSuccess = 201;

    /**
     * @param string $field
     * @param string $source
     * @param string $target
     * @param int|null $siteId
     * @return mixed
     * @throws \yii\web\HttpException
     */
    public function run(
        string $field,
        string $source,
        string $target,
        int $siteId = 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) {
            return true;
        }

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

    /**
     * @param Association $record
     * @return bool
     * @throws \Throwable
     * @throws \yii\db\StaleObjectException
     */
    protected function performAction(Association $record): bool
    {
        return $record->delete();
    }
}


1			<?php
2
3			/**
4			* @copyright Copyright (c) Flipbox Digital Limited
5			* @license https://www.flipboxfactory.com/software/element-lists/license
6			* @link https://www.flipboxfactory.com/software/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 Dissociate extends Action
23			{
24			use ManageTrait,
25			ResolverTrait;
26
27			/**
28			* @var int
29			*/
30			public $statusCodeSuccess = 201;
31
32			/**
33			* @param string $field
34			* @param string $source
35			* @param string $target
36			* @param int\|null $siteId
37			* @return mixed
38			* @throws \yii\web\HttpException
39			*/
40			public function run(
41			string $field,
42			string $source,
43			string $target,
44			int $siteId = null
45			) {
46
47			// Resolve
48			$field = $this->resolveField($field);
49			$source = $this->resolveElement($source);
50			$target = $this->resolveElement($target);
51
52			/** @var Field $field */
53
54			$siteId = SiteHelper::ensureSiteId($siteId ?: $source->siteId);
55
56			$record = Association::find()
57			->fieldId($field->id)
58			->sourceId($source->getId() ?: false)
			0 ignored issues – show Security Bug introduced 2019-03-24 15:48 UTC by Report Bug Copy Issue Report It seems like `$source->getId() ?: false` can also be of type `false`; however, `flipbox\craft\element\li...iationQuery::sourceId()` does only seem to accept `integer\|array<integer,integer>\|null`, did you maybe forget to handle an error condition? Loading history...
59			->targetId($target->getId() ?: false)
			0 ignored issues – show Security Bug introduced 2019-03-24 15:48 UTC by Report Bug Copy Issue Report It seems like `$target->getId() ?: false` can also be of type `false`; however, `flipbox\craft\element\li...iationQuery::targetId()` does only seem to accept `integer\|array<integer,integer>\|null`, did you maybe forget to handle an error condition? Loading history...
60			->siteId($siteId)
61			->one();
62
63			if (!$record) {
64			return true;
65			}
66
67			return $this->runInternal($record);
68			}
69
70			/**
71			* @param Association $record
72			* @return bool
73			* @throws \Throwable
74			* @throws \yii\db\StaleObjectException
75			*/
76			protected function performAction(Association $record): bool
77			{
78			return $record->delete();
79			}
80			}
81

flipboxfactory / craft-element-lists

Branch — develop (17e8cf)

Dissociate::performAction() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like