AddToCartAction::afterRun() - Code Metrics - Inspection of "Changed redirects order to display the error messa..." - hiqdev/yii2-cart - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c6d8a3...7d6821 )

by Dmitry

created 2020-03-04 15:03 UTC

AddToCartAction::afterRun() B

↳ Parent: AddToCartAction

Complexity

Conditions	6
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

Metric	Value
dl	0
loc	23
ccs	0
cts	15
cp	0
rs	8.9297
c	0
b	0
f	0
cc	6
nc	8
nop	0
crap	42

<?php

/*
 * Cart module for Yii2
 *
 * @link      https://github.com/hiqdev/yii2-cart
 * @package   yii2-cart
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\yii2\cart\actions;

use hiqdev\yii2\cart\NotPurchasableException;
use hiqdev\hiart\Collection;
use hiqdev\yii2\cart\CartPositionInterface;
use hiqdev\yii2\cart\Module as CartModule;
use Yii;

class AddToCartAction extends \yii\base\Action
{
    /**
     * @var CartPositionInterface The class for new product
     */
    public $productClass;

    /**
     * @var boolean whether the action expects bulk models load using `selection`
     */
    public $bulkLoad = false;

    /**
     * @var bool whether client should be redirected to the cart in case of success item adding
     */
    public $redirectToCart = false;

    /**
     * @var bool whether any errors occurred during save
     */
    protected $hasErrors = false;

    /**
     * Returns the cart module.
     * @return CartModule
     */
    public function getCartModule()
    {
        return CartModule::getInstance();
    }

    public function run()
    {
        $data = null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $request = Yii::$app->request;
        /** @var CartPositionInterface $model */
        $model = Yii::createObject($this->productClass);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $collection = new Collection(); // TODO: drop dependency
        $collection->setModel($model);

        if (!$this->bulkLoad) {
            $data = [$request->post() ?: $request->get()];
            $collection->load($data);
        } else {
            $collection->load();
        }

        $positions = [];
        foreach ($collection->models as $position) {
            /** @var CartPositionInterface $position */
            if (!$position->validate()) {

                $this->hasErrors = true;
                $error = $collection->getFirstError();
                if (empty($error)) {
                    $error = Yii::t('cart', 'Failed to add item to the cart');
                }
                Yii::$app->session->addFlash('warning', $error);
                Yii::warning('Failed to add item to cart', 'cart');

                continue;
            }

            $positions[] = $position;
        }

        try {
            $this->putPositionsToCart($positions);
        } catch (NotPurchasableException $exception) {
            $this->hasErrors = true;
            Yii::$app->getSession()->setFlash('error', $exception->getMessage());
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.
}
            $exception->resolve();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        }
    }

    protected function putPositionsToCart($positions)
    {
        $cart = $this->getCartModule()->getCart();
        $cart->putPositions($positions);
    }

    protected function afterRun()
    {
        $request = Yii::$app->request;

        if ($request->isAjax) {
            Yii::$app->end();
        }

        if ($this->redirectToCart && !$this->hasErrors) {
            return $this->controller->redirect('@cart');
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.
}
        }

        if (isset($request->referrer)) {
            if ($this->hasErrors) {
                return $this->controller->redirect('@cart');
            }

            Yii::$app->getSession()->setFlash('success', Yii::t('cart', 'Item has been added to cart'));
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.
}
            return $this->controller->redirect($request->referrer);
        }

        return $this->controller->goHome();
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			/*
4			* Cart module for Yii2
5			*
6			* @link https://github.com/hiqdev/yii2-cart
7			* @package yii2-cart
8			* @license BSD-3-Clause
9			* @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
10			*/
11
12			namespace hiqdev\yii2\cart\actions;
13
14			use hiqdev\yii2\cart\NotPurchasableException;
15			use hiqdev\hiart\Collection;
16			use hiqdev\yii2\cart\CartPositionInterface;
17			use hiqdev\yii2\cart\Module as CartModule;
18			use Yii;
19
20			class AddToCartAction extends \yii\base\Action
21			{
22			/**
23			* @var CartPositionInterface The class for new product
24			*/
25			public $productClass;
26
27			/**
28			* @var boolean whether the action expects bulk models load using `selection`
29			*/
30			public $bulkLoad = false;
31
32			/**
33			* @var bool whether client should be redirected to the cart in case of success item adding
34			*/
35			public $redirectToCart = false;
36
37			/**
38			* @var bool whether any errors occurred during save
39			*/
40			protected $hasErrors = false;
41
42			/**
43			* Returns the cart module.
44			* @return CartModule
45			*/
46			public function getCartModule()
47			{
48			return CartModule::getInstance();
49			}
50
51			public function run()
52			{
53			$data = null;
			0 ignored issues – show Unused Code introduced 2016-08-07 20:00 UTC by Report Bug Copy Issue Report `$data` 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...
54			$request = Yii::$app->request;
55			/** @var CartPositionInterface $model */
56			$model = Yii::createObject($this->productClass);
			0 ignored issues – show Documentation introduced 2016-08-07 20:00 UTC by Report Bug Copy Issue Report `$this->productClass` is of type `object<hiqdev\yii2\cart\CartPositionInterface>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
57			$collection = new Collection(); // TODO: drop dependency
58			$collection->setModel($model);
59
60			if (!$this->bulkLoad) {
61			$data = [$request->post() ?: $request->get()];
62			$collection->load($data);
63			} else {
64			$collection->load();
65			}
66
67			$positions = [];
68			foreach ($collection->models as $position) {
69			/** @var CartPositionInterface $position */
70			if (!$position->validate()) {
			0 ignored issues – show Bug introduced 2016-08-07 20:00 UTC by Report Bug Copy Issue Report The method `validate()` does not seem to exist on `object<hiqdev\yii2\cart\CartPositionInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
71			$this->hasErrors = true;
72			$error = $collection->getFirstError();
73			if (empty($error)) {
74			$error = Yii::t('cart', 'Failed to add item to the cart');
75			}
76			Yii::$app->session->addFlash('warning', $error);
77			Yii::warning('Failed to add item to cart', 'cart');
78
79			continue;
80			}
81
82			$positions[] = $position;
83			}
84
85			try {
86			$this->putPositionsToCart($positions);
87			} catch (NotPurchasableException $exception) {
88			$this->hasErrors = true;
89			Yii::$app->getSession()->setFlash('error', $exception->getMessage());
			0 ignored issues – show Bug introduced 2019-06-27 13:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. 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...
90			$exception->resolve();
			0 ignored issues – show Unused Code introduced 2019-08-06 14:08 UTC by Report Bug Copy Issue Report The call to the method `hiqdev\yii2\cart\NotPurc...bleException::resolve()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
91			}
92			}
93
94			protected function putPositionsToCart($positions)
95			{
96			$cart = $this->getCartModule()->getCart();
97			$cart->putPositions($positions);
98			}
99
100			protected function afterRun()
101			{
102			$request = Yii::$app->request;
103
104			if ($request->isAjax) {
105			Yii::$app->end();
106			}
107
108			if ($this->redirectToCart && !$this->hasErrors) {
109			return $this->controller->redirect('@cart');
			0 ignored issues – show Bug introduced 2017-12-07 16:39 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...
110			}
111
112			if (isset($request->referrer)) {
113			if ($this->hasErrors) {
114			return $this->controller->redirect('@cart');
115			}
116
117			Yii::$app->getSession()->setFlash('success', Yii::t('cart', 'Item has been added to cart'));
			0 ignored issues – show Bug introduced 2018-04-19 15:02 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. 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...
118			return $this->controller->redirect($request->referrer);
119			}
120
121			return $this->controller->goHome();
			0 ignored issues – show Bug introduced 2017-12-07 16:39 UTC by Report Bug Copy Issue Report The method `goHome` 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...
122			}
123			}
124

hiqdev / yii2-cart

Push — master ( c6d8a3...7d6821 )

AddToCartAction::afterRun() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like