BatchAction::run() - Code Metrics - rkit/bootstrap-yii2 - Measure and Improve Code Quality continuously with Scrutinizer

BatchAction::run() A
last analyzed 2018-06-26 11:20 UTC

↳ Parent: BatchAction

Complexity

Conditions	5
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	26
rs	9.1928
c	0
b	0
f	0
cc	5
nc	3
nop	0

<?php

namespace app\modules\admin\actions;

use Yii;
use yii\base\Action;

class BatchAction extends Action
{
    /**
     * @var string $modelClass
     */
    public $modelClass;
    /**
     * @var array $actions
     */
    public $actions;

    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->actions[$operation])) {
                $attrubutes = $this->actions[$operation];
                foreach ($models as $model) {
                    switch ($operation) {
                        case 'delete':
                            $model->delete();
                            break;
                        default:
                            $model->setAttributes($attrubutes, false);
                            $model->save(false);
                            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\actions;
4
5			use Yii;
6			use yii\base\Action;
7
8			class BatchAction extends Action
9			{
10			/**
11			* @var string $modelClass
12			*/
13			public $modelClass;
14			/**
15			* @var array $actions
16			*/
17			public $actions;
18
19			public function run()
20			{
21			$model = new $this->modelClass;
22
23			if (($ids = Yii::$app->request->post('selection')) !== null) {
24			$models = $model::findAll($ids);
25			$operation = Yii::$app->request->post('operation');
26
27			if (isset($this->actions[$operation])) {
28			$attrubutes = $this->actions[$operation];
29			foreach ($models as $model) {
30			switch ($operation) {
31			case 'delete':
32			$model->delete();
33			break;
34			default:
35			$model->setAttributes($attrubutes, false);
36			$model->save(false);
37			break;
38			}
39			}
40			}
41			}
42
43			return $this->controller->asJson(true);
			0 ignored issues – show Bug introduced 2017-11-07 10:30 UTC by Report Bug Copy Issue Report The method `asJson` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\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

BatchAction::run() A last analyzed 2018-06-26 11:20 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

BatchAction::run() A
last analyzed 2018-06-26 11:20 UTC