Inspection of "Remove check that might be redundant in some cases" - nazar-pc/CleverStyle-Framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( abcfbd...cc0839 )

by Nazar

created 2016-01-01 23:50 UTC

Status

Indentation +13 added lines, -13 removed lines patch added patch discarded remove patch

@@ -1,19 +1,19 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 /**
- * Provides next events:
- *  Shop/Cart/calculate
- *  [
- *   'items'    => &$items,   // Array of array elements [id => item_id, units => units, price => total_price]
- *   'shipping' => &$shipping // Array in form [type => shipping_type_id, price => shipping_type_price]
- *  ]
- */
+	 * Provides next events:
+	 *  Shop/Cart/calculate
+	 *  [
+	 *   'items'    => &$items,   // Array of array elements [id => item_id, units => units, price => total_price]
+	 *   'shipping' => &$shipping // Array in form [type => shipping_type_id, price => shipping_type_price]
+	 *  ]
+	 */
 
 namespace cs\modules\Shop;
 use

Please login to merge, or discard this patch.

components/modules/Shop/api/payment_methods.get.php 1 patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Page;

Please login to merge, or discard this patch.

components/modules/Shop/categories_.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -113,7 +113,7 @@
 block discarded – undo
 if (!$items) {
 	return;
 }
-$items_total     = $Items->search(
+$items_total = $Items->search(
 	[
 		'listed'      => 1,
 		'category'    => $current_category,

Please login to merge, or discard this patch.

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Config,

Please login to merge, or discard this patch.

components/modules/Shop/events/enabled.php 1 patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package    Shop
- * @attribute  modules
- * @author     Nazar Mokrynskyi <[email protected]>
- * @copyright  Copyright (c) 2015, Nazar Mokrynskyi
- * @license    MIT License, see license.txt
- */
+	 * @package    Shop
+	 * @attribute  modules
+	 * @author     Nazar Mokrynskyi <[email protected]>
+	 * @copyright  Copyright (c) 2015, Nazar Mokrynskyi
+	 * @license    MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Config,

Please login to merge, or discard this patch.

components/modules/Shop/events/enabled/admin.php 1 patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @attribute modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @attribute modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Event,

Please login to merge, or discard this patch.

components/modules/Shop/items_.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@
 block discarded – undo
 	':',
 	array_slice(Route::instance()->path, -1)[0]
 );
-$item       = $Items->get_for_user(array_pop($item));
+$item = $Items->get_for_user(array_pop($item));
 $Page->title($item['title']);
 $Page->Description = description($item['description']);
 $Page->canonical_url(

Please login to merge, or discard this patch.

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Config,

Please login to merge, or discard this patch.

components/modules/Shop/orders_.php 2 patches

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -18,7 +18,7 @@  discard block
 block discarded – undo
 $L              = new Prefix('shop_');
 $Language       = Language::instance();
 $Page           = Page::instance();
-$Session           = Session::instance();
+$Session = Session::instance();
 $Categories     = Categories::instance();
 $Items          = Items::instance();
 $Orders         = Orders::instance();
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
 $page           = @$_GET['page'] ?: 1;
 $count          = @$_GET['count'] ?: Config::instance()->module('Shop')->items_per_page;
 if ($Session->user()) {
-	$orders       = $Orders->get($Orders->search(
+	$orders = $Orders->get($Orders->search(
 		[
 			'user' => $Session->get_user()
 		] + (array)$_GET,

Please login to merge, or discard this patch.

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Config,

Please login to merge, or discard this patch.

components/modules/Shop/pay.php 1 patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Shop
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Shop
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2014-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Shop;
 use
 	cs\Config,

Please login to merge, or discard this patch.

components/modules/Static_pages/Categories.php 2 patches

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -20,7 +20,7 @@
 block discarded – undo
 	use
 		CRUD,
 		Singleton;
-	protected $data_model          = [
+	protected $data_model = [
 		'id'     => 'int',
 		'title'  => 'ml:text',
 		'path'   => 'ml:text',

Please login to merge, or discard this patch.

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -1,11 +1,11 @@
 block discarded – undo
 <?php
 /**
- * @package   Static Pages
- * @category  modules
- * @author    Nazar Mokrynskyi <[email protected]>
- * @copyright Copyright (c) 2011-2015, Nazar Mokrynskyi
- * @license   MIT License, see license.txt
- */
+	 * @package   Static Pages
+	 * @category  modules
+	 * @author    Nazar Mokrynskyi <[email protected]>
+	 * @copyright Copyright (c) 2011-2015, Nazar Mokrynskyi
+	 * @license   MIT License, see license.txt
+	 */
 namespace cs\modules\Static_pages;
 use
 	h,

Please login to merge, or discard this patch.

		@@ -113,7 +113,7 @@
		block discarded – undo
113	113	if (!$items) {
114	114	return;
115	115	}
116		-$items_total = $Items->search(
	116	+$items_total = $Items->search(
117	117	[
118	118	'listed' => 1,
119	119	'category' => $current_category,

		@@ -1,19 +1,19 @@
		block discarded – undo
1	1	<?php
2	2	/**
3		- * @package Shop
4		- * @category modules
5		- * @author Nazar Mokrynskyi <[email protected]>
6		- * @copyright Copyright (c) 2015, Nazar Mokrynskyi
7		- * @license MIT License, see license.txt
8		- */
	3	+ * @package Shop
	4	+ * @category modules
	5	+ * @author Nazar Mokrynskyi <[email protected]>
	6	+ * @copyright Copyright (c) 2015, Nazar Mokrynskyi
	7	+ * @license MIT License, see license.txt
	8	+ */
9	9	/**
10		- * Provides next events:
11		- * Shop/Cart/calculate
12		- * [
13		- * 'items' => &$items, // Array of array elements [id => item_id, units => units, price => total_price]
14		- * 'shipping' => &$shipping // Array in form [type => shipping_type_id, price => shipping_type_price]
15		- * ]
16		- */
	10	+ * Provides next events:
	11	+ * Shop/Cart/calculate
	12	+ * [
	13	+ * 'items' => &$items, // Array of array elements [id => item_id, units => units, price => total_price]
	14	+ * 'shipping' => &$shipping // Array in form [type => shipping_type_id, price => shipping_type_price]
	15	+ * ]
	16	+ */
17	17
18	18	namespace cs\modules\Shop;
19	19	use

		@@ -1,11 +1,11 @@
		block discarded – undo
1	1	<?php
2	2	/**
3		- * @package Shop
4		- * @category modules
5		- * @author Nazar Mokrynskyi <[email protected]>
6		- * @copyright Copyright (c) 2015, Nazar Mokrynskyi
7		- * @license MIT License, see license.txt
8		- */
	3	+ * @package Shop
	4	+ * @category modules
	5	+ * @author Nazar Mokrynskyi <[email protected]>
	6	+ * @copyright Copyright (c) 2015, Nazar Mokrynskyi
	7	+ * @license MIT License, see license.txt
	8	+ */
9	9	namespace cs\modules\Shop;
10	10	use
11	11	cs\Page;

		@@ -23,7 +23,7 @@
		block discarded – undo
23	23	':',
24	24	array_slice(Route::instance()->path, -1)[0]
25	25	);
26		-$item = $Items->get_for_user(array_pop($item));
	26	+$item = $Items->get_for_user(array_pop($item));
27	27	$Page->title($item['title']);
28	28	$Page->Description = description($item['description']);
29	29	$Page->canonical_url(

		@@ -18,7 +18,7 @@ discard block
		block discarded – undo
18	18	$L = new Prefix('shop_');
19	19	$Language = Language::instance();
20	20	$Page = Page::instance();
21		-$Session = Session::instance();
	21	+$Session = Session::instance();
22	22	$Categories = Categories::instance();
23	23	$Items = Items::instance();
24	24	$Orders = Orders::instance();
		@@ -27,7 +27,7 @@ discard block
		block discarded – undo
27	27	$page = @$_GET['page'] ?: 1;
28	28	$count = @$_GET['count'] ?: Config::instance()->module('Shop')->items_per_page;
29	29	if ($Session->user()) {
30		- $orders = $Orders->get($Orders->search(
	30	+ $orders = $Orders->get($Orders->search(
31	31	[
32	32	'user' => $Session->get_user()
33	33	] + (array)$_GET,

		@@ -20,7 +20,7 @@
		block discarded – undo
20	20	use
21	21	CRUD,
22	22	Singleton;
23		- protected $data_model = [
	23	+ protected $data_model = [
24	24	'id' => 'int',
25	25	'title' => 'ml:text',
26	26	'path' => 'ml:text',

nazar-pc / CleverStyle-Framework

Push — master ( abcfbd...cc0839 )

Status

Category

Indentation +13 added lines, -13 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Indentation +6 added lines, -6 removed lines patch added patch discarded remove patch