AuthorizationController::actionAuthorize() - Code Metrics - Inspection of "2.3.0 prep" - flipboxfactory/patron - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 424357...d1d6a1 )

by Nate

created 2020-10-28 15:48 UTC

AuthorizationController::actionAuthorize() A

↳ Parent: AuthorizationController

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	18
ccs	0
cts	16
cp	0
rs	9.6666
c	0
b	0
f	0
cc	2
nc	2
nop	1
crap	6

<?php

namespace flipbox\patron\cp\controllers;

use Craft;
use flipbox\patron\actions\authorization\Authorize;

class AuthorizationController extends AbstractController
{
    /**
     * @inheritdoc
     */
    protected function verbs(): array
    {
        return [
            'authorize' => ['get']
        ];
    }

    /**
     * @param int|null $id
     * @return mixed
     * @throws \yii\base\InvalidConfigException
     */
    public function actionAuthorize(int $id = null)
    {
        if (null === $id) {
            $id = (int)Craft::$app->getRequest()->getBodyParam('id');
        }

        $action = Craft::createObject([
            'class' => Authorize::class,
            'checkAccess' => [$this, 'checkAdminAccess']
        ], [
            'authorize',
            $this
        ]);

        return $action->runWithParams([
            'id' => $id
        ]);
    }

    /**
     * @inheritDoc
     */
    public function checkAdminAccess(): bool
    {
        $this->requireAdmin(false);
        return true;
    }
}


1			<?php
2
3			namespace flipbox\patron\cp\controllers;
4
5			use Craft;
6			use flipbox\patron\actions\authorization\Authorize;
7
8			class AuthorizationController extends AbstractController
9			{
10			/**
11			* @inheritdoc
12			*/
13			protected function verbs(): array
14			{
15			return [
16			'authorize' => ['get']
17			];
18			}
19
20			/**
21			* @param int\|null $id
22			* @return mixed
23			* @throws \yii\base\InvalidConfigException
24			*/
25			public function actionAuthorize(int $id = null)
26			{
27			if (null === $id) {
28			$id = (int)Craft::$app->getRequest()->getBodyParam('id');
29			}
30
31			$action = Craft::createObject([
32			'class' => Authorize::class,
33			'checkAccess' => [$this, 'checkAdminAccess']
34			], [
35			'authorize',
36			$this
37			]);
38
39			return $action->runWithParams([
40			'id' => $id
41			]);
42			}
43
44			/**
45			* @inheritDoc
46			*/
47			public function checkAdminAccess(): bool
48			{
49			$this->requireAdmin(false);
50			return true;
51			}
52			}
53

flipboxfactory / patron

Push — master ( 424357...d1d6a1 )

AuthorizationController::actionAuthorize() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like