MultipleDelete::run() - Code Metrics - DevGroup-ru/dotplant2 - Measure and Improve Code Quality continuously with Scrutinizer

MultipleDelete::run() A
last analyzed 2020-02-16 09:05 UTC

↳ Parent: MultipleDelete

Complexity

Conditions	4
Paths	2

Size

Total Lines

Duplication

Lines	6
Ratio	30 %

Importance

Changes

Metric	Value
dl	6
loc	20
rs	9.6
c	0
b	0
f	0
cc	4
nc	2
nop	0

<?php

namespace app\backend\actions;

use app;
use yii;
use yii\base\Action;
use yii\base\InvalidConfigException;

/**
 * Action for multiple deletion of models from backend grid
 * @package app\backend\actions
 */
class MultipleDelete extends Action
{
    /**
     * @var string Model name, ie. `Product::className()`
     */
    public $modelName = null;

    /**
     * @var bool Mark as deleted instead of calling `delete()`
     */
    public $markAsDeleted = false;

    /**
     * @var string Attribute that stores deleted state
     */
    public $deletedMarkAttribute = 'is_active';

    /**
     * @var mixed Deleted state value(ie. 1 for is_deleted or 0 for is_active)
     */
    public $deletedMarkValue = 0;

    /**
     * @var array Route to redirect after deletion
     */
    public $redirect = ['index'];

    public function init()
    {
        if (!isset($this->modelName)) {
            throw new InvalidConfigException("Model name should be set in controller actions");
        }
        if (!class_exists($this->modelName)) {
            throw new InvalidConfigException("Model class does not exists");
        }
    }

    /**
     * @inheritdoc
     */
    public function run()
    {
        $modelName = $this->modelName;
        $items = Yii::$app->request->post('items', []);
        if (!empty($items)) {
            /** @var \yii\db\ActiveRecord $modelName fake type for PHPStorm (: */
            $items = $modelName::find()->where(['in', 'id', $items])->all();
            foreach ($items as $item) {
                if ($this->markAsDeleted === true) {
                    $item->setAttribute($this->deletedMarkAttribute, $this->deletedMarkValue);
                    $item->save();
                } else {
                    $item->delete();
                }
            }
        }

        Yii::$app->session->setFlash('info', Yii::t('app', 'Objects deleted'));
        return $this->controller->redirect($this->redirect);
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\backend\actions;
4
5		use app;
6		use yii;
7		use yii\base\Action;
8		use yii\base\InvalidConfigException;
9
10		/**
11		* Action for multiple deletion of models from backend grid
12		* @package app\backend\actions
13		*/
14		class MultipleDelete extends Action
15		{
16		/**
17		* @var string Model name, ie. `Product::className()`
18		*/
19		public $modelName = null;
20
21		/**
22		* @var bool Mark as deleted instead of calling `delete()`
23		*/
24		public $markAsDeleted = false;
25
26		/**
27		* @var string Attribute that stores deleted state
28		*/
29		public $deletedMarkAttribute = 'is_active';
30
31		/**
32		* @var mixed Deleted state value(ie. 1 for is_deleted or 0 for is_active)
33		*/
34		public $deletedMarkValue = 0;
35
36		/**
37		* @var array Route to redirect after deletion
38		*/
39		public $redirect = ['index'];
40
41	View Code Duplication	public function init()
42		{
43		if (!isset($this->modelName)) {
44		throw new InvalidConfigException("Model name should be set in controller actions");
45		}
46		if (!class_exists($this->modelName)) {
47		throw new InvalidConfigException("Model class does not exists");
48		}
49		}
50
51		/**
52		* @inheritdoc
53		*/
54		public function run()
55		{
56		$modelName = $this->modelName;
57		$items = Yii::$app->request->post('items', []);
58		if (!empty($items)) {
59		/** @var \yii\db\ActiveRecord $modelName fake type for PHPStorm (: */
60		$items = $modelName::find()->where(['in', 'id', $items])->all();
61		foreach ($items as $item) {
62	View Code Duplication	if ($this->markAsDeleted === true) {
63		$item->setAttribute($this->deletedMarkAttribute, $this->deletedMarkValue);
64		$item->save();
65		} else {
66		$item->delete();
67		}
68		}
69		}
70
71		Yii::$app->session->setFlash('info', Yii::t('app', 'Objects deleted'));
72		return $this->controller->redirect($this->redirect);
		0 ignored issues – show Bug introduced 2018-06-05 13:06 UTC by Report Bug Copy Issue Report The method `redirect` 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...
73		}
74		}
75

DevGroup-ru / dotplant2

GitHub Access Token became invalid

MultipleDelete::run() A last analyzed 2020-02-16 09:05 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

MultipleDelete::run() A
last analyzed 2020-02-16 09:05 UTC