EcommerceController::configureAction() - Code Metrics - Inspection of "change configurator system, add toggle for isset i..." - injitools/cms-Inji - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( bea1ad...15ea0b )

by Alexey

created 2017-10-21 13:50 UTC

EcommerceController::configureAction() B

↳ Parent: EcommerceController

Complexity

Conditions	4
Paths	4

Size

Total Lines	180
Code Lines	120

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	120
nc	4
nop	0
dl	0
loc	180
rs	8.1935
c	0
b	0
f	0

How to fix Long Method

<?php

/**
 * Ecommerce admin controller
 *
 * @author Alexey Krupskiy <[email protected]>
 * @link http://inji.ru/
 * @copyright 2015 Alexey Krupskiy
 * @license https://github.com/injitools/cms-Inji/blob/master/LICENSE
 */
class EcommerceController extends adminController {

    public function dashboardAction() {
        $this->view->setTitle('Онлайн магазин');
        $this->view->page();
    }

    public function configureAction() {
        $options = [
            [
                'type' => 'checkbox',
                'name' => 'view_empty_warehouse',
                'label' => 'Продавать отсутствующие товары',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['sell_over_warehouse'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'view_empty_image',
                'label' => 'Показывать товары без изображения',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['view_empty_image'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'sell_empty_warehouse',
                'label' => 'Продавать без остатоков на складе',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['sell_empty_warehouse'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'sell_over_warehouse',
                'label' => 'Продавать сверх остатоков на складе',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['sell_over_warehouse'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'label' => 'Показывать товары с нулевой ценой',
                'name' => 'show_zero_price',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['show_zero_price'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'show_without_price',
                'label' => 'Показывать товары без цен',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['show_without_price'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'filtersInLast',
                'label' => 'Показывать Фильтры только текущей и дочерних категорий',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['filtersInLast'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'isset_sort',
                'label' => 'Возможность сортировки по наличию',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['isset_sort'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'list_all',
                'label' => 'Возможность вывести все товары',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['list_all'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'cartAddToggle',
                'label' => 'Удаление из корзины при повторном добавлении товара',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['cartAddToggle'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'defaultNeedDelivery',
                'label' => 'По умолчанию для всех товаров доставка осуществляется',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['defaultNeedDelivery'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'checkbox',
                'name' => 'catalogPresentPage',
                'label' => 'Заглавная страница магазина',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['catalogPresentPage'], false)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                ]
            ],
            [
                'type' => 'text',
                'name' => 'notify_mail',
                'label' => 'E-mail оповещений о новых заказах',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['notify_mail'], '')
                ]
            ],
            [
                'type' => 'select',
                'name' => 'defaultCategoryView',
                'label' => 'Стандартный вид категории',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryView'], 'itemList'),
                    'values' => App::$cur->ecommerce->viewsCategoryList()
                ]
            ],
            [
                'type' => 'text',
                'name' => 'orderPrefix',
                'label' => 'Префикс для номеров заказов',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['orderPrefix'], '')
                ]
            ],
            [
                'type' => 'number',
                'name' => 'default_limit',
                'label' => 'Товаров на страницу',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['default_limit'], 18)
                ]
            ],
            [
                'type' => 'text',
                'name' => 'defaultCategoryTemplate',
                'label' => 'Шаблон магазина',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryTemplate'], 'current')
                ]
            ],
        ];
        if (App::$cur->money) {
            $options[] = [
                'type' => 'select',
                'name' => 'defaultCurrency',
                'label' => 'Валюта по умолчанию',
                'options' => [
                    'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryView'], ''),
                    'values' => ['' => 'Выберите'] + \Money\Currency::getList()
                ]
            ];
        }
        if (!empty($_POST['config'])) {
            $config = App::$cur->ecommerce->config;
            foreach ($options as $option) {
                $config[$option['name']] = Tools::defValue($_POST['config'][$option['name']], false);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }
            Config::save('module', $config, 'Ecommerce');
            Tools::redirect('/admin/ecommerce/configure', 'Настройки были изменены', 'success');
        }

        $managers = [
            'Ecommerce\Delivery',
            'Ecommerce\PayType',
            'Ecommerce\Warehouse',
            'Ecommerce\Unit',
            'Ecommerce\Card',
            'Ecommerce\Discount',
            'Ecommerce\Cart\Stage',
            'Ecommerce\Item\Type',
            'Ecommerce\Item\Option',
            'Ecommerce\Item\Offer\Option',
            'Ecommerce\Item\Offer\Price\Type',
            'Ecommerce\UserAdds\Field',
            'Ecommerce\Cart\Status',
            'Ecommerce\Delivery\Field',
            'Ecommerce\Delivery\Provider',
        ];
        $this->view->setTitle('Настройки магазина');
        $this->view->page(['data' => compact('managers', 'options')]);
    }

    public function reBlockIndexAction() {
        set_time_limit(0);
        $carts = Cart::getList();
        foreach ($carts as $cart) {
            $cart->save();
        }
        Tools::redirect('/admin/ecommerce/configure', 'Данные о блокировках обновлены');
    }

    public function reSearchIndexAction($i = 0) {
        set_time_limit(0);
        Model::$logging = false;
        $count = 100;
        $items = Ecommerce\Item::getList(['start' => $i * $count, 'limit' => $count]);
        if (!$items) {
            Tools::redirect('/admin/ecommerce/configure', 'Поисковый индекс обновлен');
        } else {
            $i++;
            foreach ($items as $key => $item) {
                $item->save();
                unset($items[$key]);
                unset($item);
            }
            echo 'Происходит процесс индексации: проиндексировано ' . $i * $count;
            Tools::redirect('/admin/ecommerce/reSearchIndex/' . $i);
        }
    }

    public function newOrdersSubscribeAction() {
        $this->Notifications->subscribe('Ecommerce-orders');
    }

    public function closeCartAction($cartId = 0) {
        $cart = Ecommerce\Cart::get((int) $cartId);
        $result = new Server\Result();
        if ($cart && $cart->cart_status_id != 5) {
            $cart->cart_status_id = 5;
            $cart->save();
            $result->successMsg = 'Заказ был завершен';
            $result->send();
        }
        $result->success = false;
        $result->content = 'Такая корзина не найдена';
        $result->send();
    }

}

1			<?php
2
3			/**
4			* Ecommerce admin controller
5			*
6			* @author Alexey Krupskiy <[email protected]>
7			* @link http://inji.ru/
8			* @copyright 2015 Alexey Krupskiy
9			* @license https://github.com/injitools/cms-Inji/blob/master/LICENSE
10			*/
11			class EcommerceController extends adminController {
12
13			public function dashboardAction() {
14			$this->view->setTitle('Онлайн магазин');
15			$this->view->page();
16			}
17
18			public function configureAction() {
19			$options = [
20			[
21			'type' => 'checkbox',
22			'name' => 'view_empty_warehouse',
23			'label' => 'Продавать отсутствующие товары',
24			'options' => [
25			'value' => Tools::defValue(App::$cur->ecommerce->config['sell_over_warehouse'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
26			]
27			],
28			[
29			'type' => 'checkbox',
30			'name' => 'view_empty_image',
31			'label' => 'Показывать товары без изображения',
32			'options' => [
33			'value' => Tools::defValue(App::$cur->ecommerce->config['view_empty_image'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
34			]
35			],
36			[
37			'type' => 'checkbox',
38			'name' => 'sell_empty_warehouse',
39			'label' => 'Продавать без остатоков на складе',
40			'options' => [
41			'value' => Tools::defValue(App::$cur->ecommerce->config['sell_empty_warehouse'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
42			]
43			],
44			[
45			'type' => 'checkbox',
46			'name' => 'sell_over_warehouse',
47			'label' => 'Продавать сверх остатоков на складе',
48			'options' => [
49			'value' => Tools::defValue(App::$cur->ecommerce->config['sell_over_warehouse'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
50			]
51			],
52			[
53			'type' => 'checkbox',
54			'label' => 'Показывать товары с нулевой ценой',
55			'name' => 'show_zero_price',
56			'options' => [
57			'value' => Tools::defValue(App::$cur->ecommerce->config['show_zero_price'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
58			]
59			],
60			[
61			'type' => 'checkbox',
62			'name' => 'show_without_price',
63			'label' => 'Показывать товары без цен',
64			'options' => [
65			'value' => Tools::defValue(App::$cur->ecommerce->config['show_without_price'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
66			]
67			],
68			[
69			'type' => 'checkbox',
70			'name' => 'filtersInLast',
71			'label' => 'Показывать Фильтры только текущей и дочерних категорий',
72			'options' => [
73			'value' => Tools::defValue(App::$cur->ecommerce->config['filtersInLast'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
74			]
75			],
76			[
77			'type' => 'checkbox',
78			'name' => 'isset_sort',
79			'label' => 'Возможность сортировки по наличию',
80			'options' => [
81			'value' => Tools::defValue(App::$cur->ecommerce->config['isset_sort'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
82			]
83			],
84			[
85			'type' => 'checkbox',
86			'name' => 'list_all',
87			'label' => 'Возможность вывести все товары',
88			'options' => [
89			'value' => Tools::defValue(App::$cur->ecommerce->config['list_all'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
90			]
91			],
92			[
93			'type' => 'checkbox',
94			'name' => 'cartAddToggle',
95			'label' => 'Удаление из корзины при повторном добавлении товара',
96			'options' => [
97			'value' => Tools::defValue(App::$cur->ecommerce->config['cartAddToggle'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
98			]
99			],
100			[
101			'type' => 'checkbox',
102			'name' => 'defaultNeedDelivery',
103			'label' => 'По умолчанию для всех товаров доставка осуществляется',
104			'options' => [
105			'value' => Tools::defValue(App::$cur->ecommerce->config['defaultNeedDelivery'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
106			]
107			],
108			[
109			'type' => 'checkbox',
110			'name' => 'catalogPresentPage',
111			'label' => 'Заглавная страница магазина',
112			'options' => [
113			'value' => Tools::defValue(App::$cur->ecommerce->config['catalogPresentPage'], false)
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
114			]
115			],
116			[
117			'type' => 'text',
118			'name' => 'notify_mail',
119			'label' => 'E-mail оповещений о новых заказах',
120			'options' => [
121			'value' => Tools::defValue(App::$cur->ecommerce->config['notify_mail'], '')
122			]
123			],
124			[
125			'type' => 'select',
126			'name' => 'defaultCategoryView',
127			'label' => 'Стандартный вид категории',
128			'options' => [
129			'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryView'], 'itemList'),
130			'values' => App::$cur->ecommerce->viewsCategoryList()
131			]
132			],
133			[
134			'type' => 'text',
135			'name' => 'orderPrefix',
136			'label' => 'Префикс для номеров заказов',
137			'options' => [
138			'value' => Tools::defValue(App::$cur->ecommerce->config['orderPrefix'], '')
139			]
140			],
141			[
142			'type' => 'number',
143			'name' => 'default_limit',
144			'label' => 'Товаров на страницу',
145			'options' => [
146			'value' => Tools::defValue(App::$cur->ecommerce->config['default_limit'], 18)
147			]
148			],
149			[
150			'type' => 'text',
151			'name' => 'defaultCategoryTemplate',
152			'label' => 'Шаблон магазина',
153			'options' => [
154			'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryTemplate'], 'current')
155			]
156			],
157			];
158			if (App::$cur->money) {
159			$options[] = [
160			'type' => 'select',
161			'name' => 'defaultCurrency',
162			'label' => 'Валюта по умолчанию',
163			'options' => [
164			'value' => Tools::defValue(App::$cur->ecommerce->config['defaultCategoryView'], ''),
165			'values' => ['' => 'Выберите'] + \Money\Currency::getList()
166			]
167			];
168			}
169			if (!empty($_POST['config'])) {
170			$config = App::$cur->ecommerce->config;
171			foreach ($options as $option) {
172			$config[$option['name']] = Tools::defValue($_POST['config'][$option['name']], false);
			0 ignored issues – show Documentation introduced 2017-10-21 13:54 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `string`. 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...
173			}
174			Config::save('module', $config, 'Ecommerce');
175			Tools::redirect('/admin/ecommerce/configure', 'Настройки были изменены', 'success');
176			}
177
178			$managers = [
179			'Ecommerce\Delivery',
180			'Ecommerce\PayType',
181			'Ecommerce\Warehouse',
182			'Ecommerce\Unit',
183			'Ecommerce\Card',
184			'Ecommerce\Discount',
185			'Ecommerce\Cart\Stage',
186			'Ecommerce\Item\Type',
187			'Ecommerce\Item\Option',
188			'Ecommerce\Item\Offer\Option',
189			'Ecommerce\Item\Offer\Price\Type',
190			'Ecommerce\UserAdds\Field',
191			'Ecommerce\Cart\Status',
192			'Ecommerce\Delivery\Field',
193			'Ecommerce\Delivery\Provider',
194			];
195			$this->view->setTitle('Настройки магазина');
196			$this->view->page(['data' => compact('managers', 'options')]);
197			}
198
199			public function reBlockIndexAction() {
200			set_time_limit(0);
201			$carts = Cart::getList();
202			foreach ($carts as $cart) {
203			$cart->save();
204			}
205			Tools::redirect('/admin/ecommerce/configure', 'Данные о блокировках обновлены');
206			}
207
208			public function reSearchIndexAction($i = 0) {
209			set_time_limit(0);
210			Model::$logging = false;
211			$count = 100;
212			$items = Ecommerce\Item::getList(['start' => $i * $count, 'limit' => $count]);
213			if (!$items) {
214			Tools::redirect('/admin/ecommerce/configure', 'Поисковый индекс обновлен');
215			} else {
216			$i++;
217			foreach ($items as $key => $item) {
218			$item->save();
219			unset($items[$key]);
220			unset($item);
221			}
222			echo 'Происходит процесс индексации: проиндексировано ' . $i * $count;
223			Tools::redirect('/admin/ecommerce/reSearchIndex/' . $i);
224			}
225			}
226
227			public function newOrdersSubscribeAction() {
228			$this->Notifications->subscribe('Ecommerce-orders');
229			}
230
231			public function closeCartAction($cartId = 0) {
232			$cart = Ecommerce\Cart::get((int) $cartId);
233			$result = new Server\Result();
234			if ($cart && $cart->cart_status_id != 5) {
235			$cart->cart_status_id = 5;
236			$cart->save();
237			$result->successMsg = 'Заказ был завершен';
238			$result->send();
239			}
240			$result->success = false;
241			$result->content = 'Такая корзина не найдена';
242			$result->send();
243			}
244
245			}

injitools / cms-Inji

Push — master ( bea1ad...15ea0b )

EcommerceController::configureAction() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like