IndexAnnotation - Code Metrics - Inspection of "Version bump" - Maslosoft/Mangan - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2b5327...235414 )

by Peter

created 2017-12-17 22:25 UTC

IndexAnnotation A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

Importance

Changes

Metric	Value
wmc	4
lcom	0
cbo	4
dl	0
loc	49
ccs	0
cts	2
cp	0
rs	10
c	0
b	0
f	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	init()	0	27	4

<?php

/**
 * This software package is licensed under AGPL or Commercial license.
 *
 * @package maslosoft/mangan
 * @licence AGPL or Commercial
 * @copyright Copyright (c) Piotr Masełkowski <[email protected]>
 * @copyright Copyright (c) Maslosoft
 * @copyright Copyright (c) Others as mentioned in code
 * @link https://maslosoft.com/mangan/
 */

namespace Maslosoft\Mangan\Annotations\Indexes;

use Maslosoft\Addendum\Helpers\ParamsExpander;
use Maslosoft\Mangan\Meta\IndexMeta;
use Maslosoft\Mangan\Meta\ManganPropertyAnnotation;
use Maslosoft\Mangan\Sort;

/**
 * IndexAnnotation
 *
 * @author Piotr Maselkowski <pmaselkowski at gmail.com>
 */
class IndexAnnotation extends ManganPropertyAnnotation
{

	const Ns = __NAMESPACE__;

	public $value;

	/**
	 * This can be either:
	 *
	 * * Empty - for simple ascending index
	 * * `Sort::SortAsc` - for simple ascending index
	 * * `Sort::SortDesc` - for simple descending index
	 * * `array` - for any other keys specification
	 *
	 * @var mixed
	 */
	public $keys;
	public $options;

	public function init()
	{
		$data = (object)ParamsExpander::expand($this, ['keys', 'options']);

		$entity = $this->getEntity();
		$name = $entity->name;
		if(empty($data->keys))
		{
			$keys[$name] = Sort::SortAsc;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

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

    // do something with $myArray
}
		}
		else
		{
			if(!is_array($data->keys))
			{
				$keys[$name] = $data->keys;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

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

    // do something with $myArray
}
			}
			else
			{
				$keys = $data->keys;
			}
		}
		if(empty($data->options))
		{
			$data->options = [];
		}
		$entity->index[] = new IndexMeta($keys, $data->options);
	}

}


1			<?php
2
3			/**
4			* This software package is licensed under AGPL or Commercial license.
5			*
6			* @package maslosoft/mangan
7			* @licence AGPL or Commercial
8			* @copyright Copyright (c) Piotr Masełkowski <[email protected]>
9			* @copyright Copyright (c) Maslosoft
10			* @copyright Copyright (c) Others as mentioned in code
11			* @link https://maslosoft.com/mangan/
12			*/
13
14			namespace Maslosoft\Mangan\Annotations\Indexes;
15
16			use Maslosoft\Addendum\Helpers\ParamsExpander;
17			use Maslosoft\Mangan\Meta\IndexMeta;
18			use Maslosoft\Mangan\Meta\ManganPropertyAnnotation;
19			use Maslosoft\Mangan\Sort;
20
21			/**
22			* IndexAnnotation
23			*
24			* @author Piotr Maselkowski <pmaselkowski at gmail.com>
25			*/
26			class IndexAnnotation extends ManganPropertyAnnotation
27			{
28
29			const Ns = __NAMESPACE__;
30
31			public $value;
32
33			/**
34			* This can be either:
35			*
36			* * Empty - for simple ascending index
37			* * `Sort::SortAsc` - for simple ascending index
38			* * `Sort::SortDesc` - for simple descending index
39			* * `array` - for any other keys specification
40			*
41			* @var mixed
42			*/
43			public $keys;
44			public $options;
45
46			public function init()
47			{
48			$data = (object)ParamsExpander::expand($this, ['keys', 'options']);
49
50			$entity = $this->getEntity();
51			$name = $entity->name;
52			if(empty($data->keys))
53			{
54			$keys[$name] = Sort::SortAsc;
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-12-17 22:35 UTC by Report Bug Copy Issue Report `$keys` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$keys = 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...
55			}
56			else
57			{
58			if(!is_array($data->keys))
59			{
60			$keys[$name] = $data->keys;
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-12-17 22:35 UTC by Report Bug Copy Issue Report `$keys` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$keys = 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...
61			}
62			else
63			{
64			$keys = $data->keys;
65			}
66			}
67			if(empty($data->options))
68			{
69			$data->options = [];
70			}
71			$entity->index[] = new IndexMeta($keys, $data->options);
72			}
73
74			}
75

Maslosoft / Mangan

Push — master ( 2b5327...235414 )

IndexAnnotation A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

1 Method

Duplication Side-by-Side

Filter issues like