DissociateUserFromOrganization::run() - Code Metrics - Inspection of "making actions more robust" - flipboxfactory/organizations - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( c5e17b...b595d4 )

by Nate

created 2019-11-18 19:40 UTC

DissociateUserFromOrganization::run() A

↳ Parent: DissociateUserFromOrganization

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	17
ccs	0
cts	15
cp	0
rs	9.7
c	0
b	0
f	0
cc	3
nc	3
nop	2
crap	12

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://flipboxfactory.com/software/organization/license
 * @link       https://www.flipboxfactory.com/software/organization/
 */

namespace flipbox\organizations\actions\organizations;

use flipbox\craft\ember\actions\records\DeleteRecordTrait;
use flipbox\organizations\records\UserAssociation;
use yii\base\Action;
use yii\web\HttpException;

/**
 * @author Flipbox Factory <[email protected]>
 * @since 3.0.0
 */
class DissociateUserFromOrganization extends Action
{
    use DeleteRecordTrait, LookupAssociationTrait;

    /**
     * @param string $user
     * @param string $organization
     * @return null|\yii\base\Model|\yii\web\Response
     * @throws HttpException
     */
    public function run(
        string $user,
        string $organization
    )
    {
        if (null === ($user = $this->findUser($user))) {
            return $this->handleNotFoundResponse();
        }

        if (null === ($organization = $this->findOrganization($organization))) {
            return $this->handleNotFoundResponse();
        }

        return $this->runInternal(
            $organization->getUsers()->findOrCreate($user)
        );
    }

    /**
     * @inheritdoc
     * @param UserAssociation $record
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     * @return bool
     */
    protected function performAction(UserAssociation $association): bool
    {
        return $association->save();
    }
}


1			<?php
2
3			/**
4			* @copyright Copyright (c) Flipbox Digital Limited
5			* @license https://flipboxfactory.com/software/organization/license
6			* @link https://www.flipboxfactory.com/software/organization/
7			*/
8
9			namespace flipbox\organizations\actions\organizations;
10
11			use flipbox\craft\ember\actions\records\DeleteRecordTrait;
12			use flipbox\organizations\records\UserAssociation;
13			use yii\base\Action;
14			use yii\web\HttpException;
15
16			/**
17			* @author Flipbox Factory <[email protected]>
18			* @since 3.0.0
19			*/
20			class DissociateUserFromOrganization extends Action
21			{
22			use DeleteRecordTrait, LookupAssociationTrait;
23
24			/**
25			* @param string $user
26			* @param string $organization
27			* @return null\|\yii\base\Model\|\yii\web\Response
28			* @throws HttpException
29			*/
30			public function run(
31			string $user,
32			string $organization
33			)
34			{
35			if (null === ($user = $this->findUser($user))) {
36			return $this->handleNotFoundResponse();
37			}
38
39			if (null === ($organization = $this->findOrganization($organization))) {
40			return $this->handleNotFoundResponse();
41			}
42
43			return $this->runInternal(
44			$organization->getUsers()->findOrCreate($user)
45			);
46			}
47
48			/**
49			* @inheritdoc
50			* @param UserAssociation $record
			0 ignored issues – show Bug introduced 2019-11-18 18:50 UTC by Report Bug Copy Issue Report There is no parameter named `$record`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
51			* @return bool
52			*/
53			protected function performAction(UserAssociation $association): bool
54			{
55			return $association->save();
56			}
57			}
58

flipboxfactory / organizations

Push — develop ( c5e17b...b595d4 )

DissociateUserFromOrganization::run() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like