OperationsAction::run() - Code Metrics - Inspection of "Upd .travis" - rkit/bootstrap-yii2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c7544d...5e7707 )

by Igor

created 2017-04-27 17:03 UTC

OperationsAction::run() B

↳ Parent: OperationsAction

Complexity

Conditions	5
Paths	3

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	5

Importance

Changes

Metric	Value
dl	0
loc	25
ccs	14
cts	14
cp	1
rs	8.439
c	0
b	0
f	0
cc	5
eloc	16
nc	3
nop	0
crap	5

<?php

namespace app\modules\admin\controllers\common;

use Yii;
use yii\base\Action;
use yii\web\Response;

class OperationsAction extends Action
{
    /**
     * @var string $modelClass
     */
    public $modelClass;
    /**
     * @var array $operations
     */
    public $operations;

    public function run()
    {
        $model = new $this->modelClass;

        if (($ids = Yii::$app->request->post('selection')) !== null) {
            $models = $model::findAll($ids);
            $operation = Yii::$app->request->post('operation');

            if (isset($this->operations[$operation])) {
                $attrubutes = $this->operations[$operation];
                foreach ($models as $model) {
                    switch ($operation) {
                        case 'delete':
                            $model->delete();
                            break;
                        default:
                            $model->updateAttributes($attrubutes);
                            break;
                    }
                }
            }
        }

        return $this->controller->asJson(true);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }
}


1		<?php
2
3		namespace app\modules\admin\controllers\common;
4
5		use Yii;
6		use yii\base\Action;
7		use yii\web\Response;
8
9		class OperationsAction extends Action
10		{
11		/**
12		* @var string $modelClass
13		*/
14		public $modelClass;
15		/**
16		* @var array $operations
17		*/
18		public $operations;
19
20	4	public function run()
21		{
22	4	$model = new $this->modelClass;
23
24	4	if (($ids = Yii::$app->request->post('selection')) !== null) {
25	4	$models = $model::findAll($ids);
26	4	$operation = Yii::$app->request->post('operation');
27
28	4	if (isset($this->operations[$operation])) {
29	4	$attrubutes = $this->operations[$operation];
30	4	foreach ($models as $model) {
31		switch ($operation) {
32	4	case 'delete':
33	2	$model->delete();
34	2	break;
35		default:
36	2	$model->updateAttributes($attrubutes);
37	4	break;
38		}
39		}
40		}
41		}
42
43	4	return $this->controller->asJson(true);
		0 ignored issues – show Bug introduced 2017-04-25 07:22 UTC by Report Bug Copy Issue Report The method `asJson` does only exist in `yii\web\Controller`, but not in `yii\base\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
44		}
45		}
46

rkit / bootstrap-yii2

Push — master ( c7544d...5e7707 )

OperationsAction::run() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like