Updatecart::preparePage() - Code Metrics - Inspection of "preparations for adding additional items with orde..." - HaaseIT/HCSF - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4f13d8...42e6bf )

by Marcus

created 2017-04-11 16:59 UTC

Updatecart::preparePage() F

↳ Parent: Updatecart

Complexity

Conditions	26
Paths	245

Size

Total Lines	111
Code Lines	72

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	111
rs	3.6068
cc	26
eloc	72
nc	245
nop	0

How to fix Long Method Complexity

<?php

/*
    HCSF - A multilingual CMS and Shopsystem
    Copyright (C) 2014  Marcus Haase - [email protected]

    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation, either version 3 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.

    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */

namespace HaaseIT\HCSF\Controller\Shop;

use HaaseIT\HCSF\HelperConfig;

/**
 * Class Updatecart
 * @package HaaseIT\HCSF\Controller\Shop
 */
class Updatecart extends Base
{
    /**
     *
     */
    public function preparePage()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        $this->P = new \HaaseIT\HCSF\CorePage($this->serviceManager);
        $this->P->cb_pagetype = 'content';

        if (
            (
                HelperConfig::$shop["show_pricesonlytologgedin"]
                && !\HaaseIT\HCSF\Customer\Helper::getUserData()
            )
            || !isset($_SERVER["HTTP_REFERER"])
        ) {
            $this->P->oPayload->cl_html = $this->serviceManager->get('textcats')->T("denied_default");
        } else {
            $iAmount = '';
            if (isset($_REQUEST["amount"])) {
                $iAmount = $_REQUEST["amount"];
            }

            if (empty($_REQUEST["itemno"]) || !is_numeric($iAmount)) {
                $this->replyToCartUpdate('noitemnooramount');
            } else {
                $iAmount = floor($iAmount);

                // Check if this item exists
                $aData = $this->serviceManager->get('oItem')->sortItems('', $_REQUEST["itemno"]);
                if (!isset($aData)) {
                    $this->replyToCartUpdate('itemnotfound');
                } else {
                    // are there additional items to this item, if so, check if they are valid, too.
                    if (!empty($_POST['additionalitems'])) {
                        $additionalitems = filter_input(INPUT_POST, 'additionalitems', FILTER_SANITIZE_SPECIAL_CHARS);

                        if (strpos($additionalitems, '~') !== false) {
                            $additionalitems = explode('~', $additionalitems);
                        } else {
                            $additionalitems = [$additionalitems];
                        }

                        $additionaldata = $this->serviceManager->get('oItem')->sortItems('', $additionalitems);

                        if (count($additionalitems) != count($additionaldata['item'])) {
                            $this->replyToCartUpdate('itemnotfound');
                        }
                    }

                    // build the key for this item for the shoppingcart
                    $sItemno = $aData["item"][$_REQUEST["itemno"]]['itm_no'];
                    $sCartKey = $sItemno;

                    if (isset(HelperConfig::$shop["custom_order_fields"])) {
                        foreach (HelperConfig::$shop["custom_order_fields"] as $sValue) {
                            if (isset($aData["item"][$sItemno]["itm_data"][$sValue])) {
                                $aOptions = [];
                                $TMP = explode('|', $aData["item"][$sItemno]["itm_data"][$sValue]);
                                foreach ($TMP as $sTMPValue) {
                                    if (trim($sTMPValue) != '') {
                                        $aOptions[] = $sTMPValue;
                                    }
                                }
                                unset($sTMP);

                                if (isset($_REQUEST[$sValue]) && in_array($_REQUEST[$sValue], $aOptions)) {
                                    $sCartKey .= '|' . $sValue . ':' . $_REQUEST[$sValue];
                                } else {
                                    $this->replyToCartUpdate('requiredfieldmissing');
                                }
                            }
                        }
                    }
                    // if this Items is not in cart and amount is 0, no need to do anything, return to referer
                    if (!isset($_SESSION["cart"][$sCartKey]) && $iAmount == 0) {
                        $this->replyToCartUpdate('noactiontaken');
                    }
                    $aItem = [
                        'amount' => $iAmount,
                        'price' => $this->serviceManager->get('oItem')->calcPrice($aData["item"][$sItemno]),
                        'rg' => $aData["item"][$sItemno]['itm_rg'],
                        'vat' => $aData["item"][$sItemno]['itm_vatid'],
                        'name' => $aData["item"][$sItemno]['itm_name'],
                        'img' => $aData["item"][$sItemno]['itm_img'],
                    ];
                    if (!empty($_POST['action']) && $_POST['action'] === 'add') {
                        if (isset($_SESSION["cart"][$sCartKey])) { // if this item is already in cart, add to amount
                            $_SESSION["cart"][$sCartKey]['amount'] += $iAmount;
                        } else {
                            $_SESSION["cart"][$sCartKey] = $aItem;
                        }
                        // todo: add additional items
                    } else {
                        if (isset($_SESSION["cart"][$sCartKey])) { // if this item is already in cart, update amount
                            if ($iAmount == 0) { // new amount == 0 -> remove from cart
                                unset($_SESSION["cart"][$sCartKey]);
                                if (count($_SESSION["cart"]) == 0) { // once the last cart item is unset, we no longer need cartpricesums
                                    unset($_SESSION["cartpricesums"]);
                                }
                                $this->replyToCartUpdate('removed', ['cartkey' => $sCartKey]);
                            } else { // update amount
                                $_SESSION["cart"][$sCartKey]["amount"] = $iAmount;
                                $this->replyToCartUpdate('updated', ['cartkey' => $sCartKey, 'amount' => $iAmount]);
                            }
                        } else { // if this item is not in the cart yet, add it
                            $_SESSION["cart"][$sCartKey] = $aItem;
                        }
                    }
                    $this->replyToCartUpdate('added', ['cartkey' => $sCartKey, 'amount' => $iAmount]);
                }
            }
            die();

        }
    }

    /**
     * @param string $sReply
     * @param array $aMore
     */
    private function replyToCartUpdate($sReply, $aMore = []) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
        if (isset($_REQUEST["ajax"])) {
            $aAR = [
                'cart' => $_SESSION["cart"],
                'reply' => $sReply,
                'cartsums' => \HaaseIT\HCSF\Shop\Helper::calculateCartItems($_SESSION["cart"]),
                'currency' => HelperConfig::$shop["waehrungssymbol"],
                'numberformat_decimals' => HelperConfig::$core['numberformat_decimals'],
                'numberformat_decimal_point' => HelperConfig::$core['numberformat_decimal_point'],
                'numberformat_thousands_seperator' => HelperConfig::$core['numberformat_thousands_seperator'],
            ];
            if (count($aMore)) {
                $aAR = array_merge($aAR, $aMore);
            }
            echo $this->serviceManager->get('twig')->render('shop/update-cart.twig', $aAR);
        } else {
            $aMSG["msg"] =  $sReply;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            if (count($aMore)) {
                $aMSG = array_merge($aMSG, $aMore);
            }
            header('Location: '.\HaaseIT\Toolbox\Tools::makeLinkHRefWithAddedGetVars($_SERVER["HTTP_REFERER"], $aMSG, true, false));
        }
        die();

    }

}

1			<?php
2
3			/*
4			HCSF - A multilingual CMS and Shopsystem
5			Copyright (C) 2014 Marcus Haase - [email protected]
6
7			This program is free software: you can redistribute it and/or modify
8			it under the terms of the GNU General Public License as published by
9			the Free Software Foundation, either version 3 of the License, or
10			(at your option) any later version.
11
12			This program is distributed in the hope that it will be useful,
13			but WITHOUT ANY WARRANTY; without even the implied warranty of
14			MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15			GNU General Public License for more details.
16
17			You should have received a copy of the GNU General Public License
18			along with this program. If not, see <http://www.gnu.org/licenses/>.
19			*/
20
21			namespace HaaseIT\HCSF\Controller\Shop;
22
23			use HaaseIT\HCSF\HelperConfig;
24
25			/**
26			* Class Updatecart
27			* @package HaaseIT\HCSF\Controller\Shop
28			*/
29			class Updatecart extends Base
30			{
31			/**
32			*
33			*/
34			public function preparePage()
			0 ignored issues – show Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `preparePage` uses the super-global variable `$_SERVER` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history... Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `preparePage` uses the super-global variable `$_REQUEST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history... Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `preparePage` uses the super-global variable `$_SESSION` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history... Coding Style introduced 2017-04-11 17:07 UTC by Report Bug Copy Issue Report `preparePage` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
35			{
36			$this->P = new \HaaseIT\HCSF\CorePage($this->serviceManager);
37			$this->P->cb_pagetype = 'content';
38
39			if (
40			(
41			HelperConfig::$shop["show_pricesonlytologgedin"]
42			&& !\HaaseIT\HCSF\Customer\Helper::getUserData()
43			)
44			\|\| !isset($_SERVER["HTTP_REFERER"])
45			) {
46			$this->P->oPayload->cl_html = $this->serviceManager->get('textcats')->T("denied_default");
47			} else {
48			$iAmount = '';
49			if (isset($_REQUEST["amount"])) {
50			$iAmount = $_REQUEST["amount"];
51			}
52
53			if (empty($_REQUEST["itemno"]) \|\| !is_numeric($iAmount)) {
54			$this->replyToCartUpdate('noitemnooramount');
55			} else {
56			$iAmount = floor($iAmount);
57
58			// Check if this item exists
59			$aData = $this->serviceManager->get('oItem')->sortItems('', $_REQUEST["itemno"]);
60			if (!isset($aData)) {
61			$this->replyToCartUpdate('itemnotfound');
62			} else {
63			// are there additional items to this item, if so, check if they are valid, too.
64			if (!empty($_POST['additionalitems'])) {
65			$additionalitems = filter_input(INPUT_POST, 'additionalitems', FILTER_SANITIZE_SPECIAL_CHARS);
66
67			if (strpos($additionalitems, '~') !== false) {
68			$additionalitems = explode('~', $additionalitems);
69			} else {
70			$additionalitems = [$additionalitems];
71			}
72
73			$additionaldata = $this->serviceManager->get('oItem')->sortItems('', $additionalitems);
74
75			if (count($additionalitems) != count($additionaldata['item'])) {
76			$this->replyToCartUpdate('itemnotfound');
77			}
78			}
79
80			// build the key for this item for the shoppingcart
81			$sItemno = $aData["item"][$_REQUEST["itemno"]]['itm_no'];
82			$sCartKey = $sItemno;
83
84			if (isset(HelperConfig::$shop["custom_order_fields"])) {
85			foreach (HelperConfig::$shop["custom_order_fields"] as $sValue) {
86			if (isset($aData["item"][$sItemno]["itm_data"][$sValue])) {
87			$aOptions = [];
88			$TMP = explode('\|', $aData["item"][$sItemno]["itm_data"][$sValue]);
89			foreach ($TMP as $sTMPValue) {
90			if (trim($sTMPValue) != '') {
91			$aOptions[] = $sTMPValue;
92			}
93			}
94			unset($sTMP);
95
96			if (isset($_REQUEST[$sValue]) && in_array($_REQUEST[$sValue], $aOptions)) {
97			$sCartKey .= '\|' . $sValue . ':' . $_REQUEST[$sValue];
98			} else {
99			$this->replyToCartUpdate('requiredfieldmissing');
100			}
101			}
102			}
103			}
104			// if this Items is not in cart and amount is 0, no need to do anything, return to referer
105			if (!isset($_SESSION["cart"][$sCartKey]) && $iAmount == 0) {
106			$this->replyToCartUpdate('noactiontaken');
107			}
108			$aItem = [
109			'amount' => $iAmount,
110			'price' => $this->serviceManager->get('oItem')->calcPrice($aData["item"][$sItemno]),
111			'rg' => $aData["item"][$sItemno]['itm_rg'],
112			'vat' => $aData["item"][$sItemno]['itm_vatid'],
113			'name' => $aData["item"][$sItemno]['itm_name'],
114			'img' => $aData["item"][$sItemno]['itm_img'],
115			];
116			if (!empty($_POST['action']) && $_POST['action'] === 'add') {
117			if (isset($_SESSION["cart"][$sCartKey])) { // if this item is already in cart, add to amount
118			$_SESSION["cart"][$sCartKey]['amount'] += $iAmount;
119			} else {
120			$_SESSION["cart"][$sCartKey] = $aItem;
121			}
122			// todo: add additional items
123			} else {
124			if (isset($_SESSION["cart"][$sCartKey])) { // if this item is already in cart, update amount
125			if ($iAmount == 0) { // new amount == 0 -> remove from cart
126			unset($_SESSION["cart"][$sCartKey]);
127			if (count($_SESSION["cart"]) == 0) { // once the last cart item is unset, we no longer need cartpricesums
128			unset($_SESSION["cartpricesums"]);
129			}
130			$this->replyToCartUpdate('removed', ['cartkey' => $sCartKey]);
131			} else { // update amount
132			$_SESSION["cart"][$sCartKey]["amount"] = $iAmount;
133			$this->replyToCartUpdate('updated', ['cartkey' => $sCartKey, 'amount' => $iAmount]);
134			}
135			} else { // if this item is not in the cart yet, add it
136			$_SESSION["cart"][$sCartKey] = $aItem;
137			}
138			}
139			$this->replyToCartUpdate('added', ['cartkey' => $sCartKey, 'amount' => $iAmount]);
140			}
141			}
142			die();
			0 ignored issues – show Coding Style Compatibility introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report The method `preparePage()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
143			}
144			}
145
146			/**
147			* @param string $sReply
148			* @param array $aMore
149			*/
150			private function replyToCartUpdate($sReply, $aMore = []) {
			0 ignored issues – show Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `replyToCartUpdate` uses the super-global variable `$_REQUEST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history... Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `replyToCartUpdate` uses the super-global variable `$_SESSION` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history... Coding Style introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `replyToCartUpdate` uses the super-global variable `$_SERVER` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
151			if (isset($_REQUEST["ajax"])) {
152			$aAR = [
153			'cart' => $_SESSION["cart"],
154			'reply' => $sReply,
155			'cartsums' => \HaaseIT\HCSF\Shop\Helper::calculateCartItems($_SESSION["cart"]),
156			'currency' => HelperConfig::$shop["waehrungssymbol"],
157			'numberformat_decimals' => HelperConfig::$core['numberformat_decimals'],
158			'numberformat_decimal_point' => HelperConfig::$core['numberformat_decimal_point'],
159			'numberformat_thousands_seperator' => HelperConfig::$core['numberformat_thousands_seperator'],
160			];
161			if (count($aMore)) {
162			$aAR = array_merge($aAR, $aMore);
163			}
164			echo $this->serviceManager->get('twig')->render('shop/update-cart.twig', $aAR);
165			} else {
166			$aMSG["msg"] = $sReply;
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report `$aMSG` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$aMSG = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
167			if (count($aMore)) {
168			$aMSG = array_merge($aMSG, $aMore);
169			}
170			header('Location: '.\HaaseIT\Toolbox\Tools::makeLinkHRefWithAddedGetVars($_SERVER["HTTP_REFERER"], $aMSG, true, false));
171			}
172			die();
			0 ignored issues – show Coding Style Compatibility introduced 2017-03-21 09:14 UTC by Report Bug Copy Issue Report The method `replyToCartUpdate()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
173			}
174
175			}

HaaseIT / HCSF

Push — master ( 4f13d8...42e6bf )

Updatecart::preparePage() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like