Repeat::getMax() - Code Metrics - Inspection of "#3 Now supports dynamic max, and allows multiple v..." - Level-2/Transphporm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9c7a65...e91ae6 )

by Tom

created 2016-03-04 14:45 UTC

Repeat::getMax() A

↳ Parent: Repeat

Complexity

Conditions	2
Paths	2

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	3
rs	10
cc	2
eloc	2
nc	2
nop	1

<?php
/* @description     Transformation Style Sheets - Revolutionising PHP templating    *
 * @author          Tom Butler [email protected]                                             *
 * @copyright       2015 Tom Butler <[email protected]> | https://r.je/                      *
 * @license         http://www.opensource.org/licenses/bsd-license.php  BSD License *
 * @version         1.0                                                             */
namespace Transphporm\Property;
class Repeat implements \Transphporm\Property {
	private $functionSet;
	private $elementData;

	public function __construct(\Transphporm\FunctionSet $functionSet, \Transphporm\Hook\ElementData $elementData) {
		$this->functionSet = $functionSet;
		$this->elementData = $elementData;		
	}

	public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
		if ($element->getAttribute('transphporm') === 'added') return $element->parentNode->removeChild($element);
		$rule = $rules['repeat'];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

		$max = $this->getMax($values);
		$count = 0;
		foreach ($values[0] as $key => $iteration) {
			$clone = $element->cloneNode(true);
			//Mark all but one of the nodes as having been added by transphporm, when the hook is run again, these are removed
			if ($count++ > 0) $clone->setAttribute('transphporm', 'added');
			if ($count > $max) break;
			
			$this->elementData->bind($clone, $iteration, 'iteration');
			$this->elementData->bind($clone, $key, 'key');
			$element->parentNode->insertBefore($clone, $element);

			//Re-run the hook on the new element, but use the iterated data
			//Don't run repeat on the clones element or it will loop forever
			unset($rules['repeat']);
			$this->createHook($rules, $pseudoMatcher, $properties)->run($clone);

		}
		//Remove the original element
		$element->parentNode->removeChild($element);
		return false;
	}

	private function getMax($values) {
		return isset($values[1]) ? $values[1][0] : PHP_INT_MAX;
	}

	private function createHook($newRules, $pseudoMatcher, $properties) {
		$hook = new \Transphporm\Hook\PropertyHook($newRules, $pseudoMatcher, new \Transphporm\Parser\Value($this->functionSet));
		foreach ($properties as $name => $property) $hook->registerProperty($name, $property);
		return $hook;
	}
}

1			<?php
2			/* @description Transformation Style Sheets - Revolutionising PHP templating *
3			* @author Tom Butler [email protected] *
4			* @copyright 2015 Tom Butler <[email protected]> \| https://r.je/ *
5			* @license http://www.opensource.org/licenses/bsd-license.php BSD License *
6			* @version 1.0 */
7			namespace Transphporm\Property;
8			class Repeat implements \Transphporm\Property {
9			private $functionSet;
10			private $elementData;
11
12			public function __construct(\Transphporm\FunctionSet $functionSet, \Transphporm\Hook\ElementData $elementData) {
13			$this->functionSet = $functionSet;
14			$this->elementData = $elementData;
15			}
16
17			public function run(array $values, \DomElement $element, array $rules, \Transphporm\Hook\PseudoMatcher $pseudoMatcher, array $properties = []) {
18			if ($element->getAttribute('transphporm') === 'added') return $element->parentNode->removeChild($element);
19			$rule = $rules['repeat'];
			0 ignored issues – show Unused Code introduced 2016-03-04 13:34 UTC by Report Bug Copy Issue Report `$rule` 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...
20
21			$max = $this->getMax($values);
22			$count = 0;
23			foreach ($values[0] as $key => $iteration) {
24			$clone = $element->cloneNode(true);
25			//Mark all but one of the nodes as having been added by transphporm, when the hook is run again, these are removed
26			if ($count++ > 0) $clone->setAttribute('transphporm', 'added');
27			if ($count > $max) break;
28
29			$this->elementData->bind($clone, $iteration, 'iteration');
30			$this->elementData->bind($clone, $key, 'key');
31			$element->parentNode->insertBefore($clone, $element);
32
33			//Re-run the hook on the new element, but use the iterated data
34			//Don't run repeat on the clones element or it will loop forever
35			unset($rules['repeat']);
36			$this->createHook($rules, $pseudoMatcher, $properties)->run($clone);
			0 ignored issues – show Compatibility introduced 2015-12-18 15:32 UTC by Report Bug Copy Issue Report `$clone` of type `object<DOMNode>` is not a sub-type of `object<DOMElement>`. It seems like you assume a child class of the class `DOMNode` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
37			}
38			//Remove the original element
39			$element->parentNode->removeChild($element);
40			return false;
41			}
42
43			private function getMax($values) {
44			return isset($values[1]) ? $values[1][0] : PHP_INT_MAX;
45			}
46
47			private function createHook($newRules, $pseudoMatcher, $properties) {
48			$hook = new \Transphporm\Hook\PropertyHook($newRules, $pseudoMatcher, new \Transphporm\Parser\Value($this->functionSet));
49			foreach ($properties as $name => $property) $hook->registerProperty($name, $property);
50			return $hook;
51			}
52			}

Level-2 / Transphporm

Push — master ( 9c7a65...e91ae6 )

Repeat::getMax() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like