AbstractSettingsController::actionSave() - Code Metrics - Inspection of "Adding point to dev-develop" - flipboxfactory/saml-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( eb3b7f...ef26fd )

by Damien

created 2018-08-17 15:03 UTC

AbstractSettingsController::actionSave() A

↳ Parent: AbstractSettingsController

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
rs	9.7333
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php
/**
 * Created by PhpStorm.
 * User: dsmrt
 * Date: 3/9/18
 * Time: 2:48 PM
 */

namespace flipbox\saml\core\controllers;

use Craft;
use craft\helpers\ArrayHelper;
use flipbox\saml\core\AbstractPlugin;
use flipbox\saml\core\traits\EnsureSamlPlugin;
use yii\base\Action;

/**
 * Class AbstractGeneralController
 * @package flipbox\saml\core\controllers\cp\view
 */
abstract class AbstractSettingsController extends AbstractController
{
    use EnsureSamlPlugin;

    /**
     * @return array
     */
    public function behaviors()
    {
        return ArrayHelper::merge(
            parent::behaviors(),
            [
                'error'    => [
                    'default' => 'save'
                ],
                'redirect' => [
                    'only'    => ['save'],
                    'actions' => [
                        'save' => [200]
                    ]
                ],
                'flash'    => [
                    'actions' => [
                        'save' => [
                            200 => \Craft::t('saml-sp', "Settings successfully updated."),
                            401 => \Craft::t('saml-sp', "Failed to update settings.")
                        ]
                    ]
                ]
            ]
        );
    }

    /**
     * @return array
     */
    protected function verbs(): array
    {
        return [
            'save' => ['post', 'put']
        ];
    }

    /**
     * @return mixed
     * @throws \yii\base\InvalidConfigException
     */
    public function actionSave()
    {
        $entityId = Craft::$app->request->getRequiredParam('entityId');
        /** @var AbstractPlugin $plugin */
        $plugin = $this->getSamlPlugin();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $settings = [
            'entityId' => $entityId
        ];

        Craft::$app->plugins->savePluginSettings(
            $this->getSamlPlugin(),
            $settings
        );
        return $this->redirectToPostedUrl();
    }

    /**
     * @return bool
     * @throws \yii\web\ForbiddenHttpException
     */
    public function checkUpdateAccess(): bool
    {
        return $this->checkAdminAccess();
    }
}


1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: dsmrt
5			* Date: 3/9/18
6			* Time: 2:48 PM
7			*/
8
9			namespace flipbox\saml\core\controllers;
10
11			use Craft;
12			use craft\helpers\ArrayHelper;
13			use flipbox\saml\core\AbstractPlugin;
14			use flipbox\saml\core\traits\EnsureSamlPlugin;
15			use yii\base\Action;
16
17			/**
18			* Class AbstractGeneralController
19			* @package flipbox\saml\core\controllers\cp\view
20			*/
21			abstract class AbstractSettingsController extends AbstractController
22			{
23			use EnsureSamlPlugin;
24
25			/**
26			* @return array
27			*/
28			public function behaviors()
29			{
30			return ArrayHelper::merge(
31			parent::behaviors(),
32			[
33			'error' => [
34			'default' => 'save'
35			],
36			'redirect' => [
37			'only' => ['save'],
38			'actions' => [
39			'save' => [200]
40			]
41			],
42			'flash' => [
43			'actions' => [
44			'save' => [
45			200 => \Craft::t('saml-sp', "Settings successfully updated."),
46			401 => \Craft::t('saml-sp', "Failed to update settings.")
47			]
48			]
49			]
50			]
51			);
52			}
53
54			/**
55			* @return array
56			*/
57			protected function verbs(): array
58			{
59			return [
60			'save' => ['post', 'put']
61			];
62			}
63
64			/**
65			* @return mixed
66			* @throws \yii\base\InvalidConfigException
67			*/
68			public function actionSave()
69			{
70			$entityId = Craft::$app->request->getRequiredParam('entityId');
71			/** @var AbstractPlugin $plugin */
72			$plugin = $this->getSamlPlugin();
			0 ignored issues – show Unused Code introduced 2018-08-17 15:20 UTC by Report Bug Copy Issue Report `$plugin` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
73
74			$settings = [
75			'entityId' => $entityId
76			];
77
78			Craft::$app->plugins->savePluginSettings(
79			$this->getSamlPlugin(),
80			$settings
81			);
82			return $this->redirectToPostedUrl();
83			}
84
85			/**
86			* @return bool
87			* @throws \yii\web\ForbiddenHttpException
88			*/
89			public function checkUpdateAccess(): bool
90			{
91			return $this->checkAdminAccess();
92			}
93			}
94

flipboxfactory / saml-core

Push — develop ( eb3b7f...ef26fd )

AbstractSettingsController::actionSave() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like