NavigationTree::activeNodes() - Code Metrics - Inspection of "Replace getNodeById() by getNodeAt()" - webeweb/bootstrap-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0295ab...9b2687 )

by WEBEWEB

created 2018-07-11 09:15 UTC

NavigationTree::activeNodes() B

↳ Parent: NavigationTree

Complexity

Conditions	7
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	45
rs	8.2666
c	0
b	0
f	0
cc	7
nc	8
nop	3

<?php

/**
 * This file is part of the bootstrap-bundle package.
 *
 * (c) 2017 WEBEWEB
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace WBW\Bundle\BootstrapBundle\Navigation;

/**
 * Navigation tree.
 *
 * @author webeweb <https://github.com/webeweb/>
 * @package WBW\Bundle\BootstrapBundle\Navigation
 */
class NavigationTree extends AbstractNavigationNode {

    /**
     * Constructor.
     *
     * @param $name The name.
     */
    public function __construct($name) {
        parent::__construct($name);
    }

    /**
     * Actives the nodes.
     *
     * @param string $url The active URL.
     * @param array $nodes The nodes.
     * @param integer $level The node level.
     * @return boolean Returns true in case of success, false othewise.
     */
    public function activeNodes($url, array $nodes = [], $level = 0) {

        // Initialize.
        $parent  = false;
        $current = false;
$myVar = 'Value';
$higher = false;

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

        // Handle each node.
        foreach ($nodes as $n) {

            // Check the node.
            if (false === ($n instanceOf AbstractNavigationNode)) {
                continue;
            }

            // Determines if the current node matches the URL.
            if ($url === $n->getRoute()) {
                $current = true;
                $delete  = true;
            } else {
                $current = $this->activeNodes($url, $n->getNodes(), $level + 1);
            }

            // Handle next node.
            if (false === $current) {
                continue;
            }

            // Mark the node as active.
            $n->setActive(true);

            // Remove the icon only on the last level.
            if (true === $delete && 0 < $level) {
                $n->setIcon(null);
            }

            // Reset.
            $parent  = true;
            $current = false;
$myVar = 'Value';
$higher = false;

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

        // Return.
        return $parent;
    }

    /**
     * Get the breadcrumbs.
     *
     * @param AbstractNavigationNode $node The navigation node.
     * @return AsbtractNavigationNode[] Returns the navigation nodes.
     */
    public function getBreadcrumbs(AbstractNavigationNode $node = null) {

        // Create the breadcrumbs.
        $breadcrumbs = [];

        // Correct the parameter if necessary.
        if (null === $node) {
            $node = $this;
        }

        // Check the instance.
        if (true === ($node instanceof NavigationNode || $node instanceof BreadcrumbNode) && true === $node->getActive()) {
            $breadcrumbs[] = $node;
        }

        // Handle each node.
        foreach ($node->getNodes() as $current) {
            if (false === ($current instanceof AbstractNavigationNode)) {
                continue;
            }
            $breadcrumbs = array_merge($breadcrumbs, $this->getBreadcrumbs($current));
        }

        // Return the breadcrumbs.
        return $breadcrumbs;
    }

}


1			<?php
2
3			/**
4			* This file is part of the bootstrap-bundle package.
5			*
6			* (c) 2017 WEBEWEB
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace WBW\Bundle\BootstrapBundle\Navigation;
13
14			/**
15			* Navigation tree.
16			*
17			* @author webeweb <https://github.com/webeweb/>
18			* @package WBW\Bundle\BootstrapBundle\Navigation
19			*/
20			class NavigationTree extends AbstractNavigationNode {
21
22			/**
23			* Constructor.
24			*
25			* @param $name The name.
26			*/
27			public function __construct($name) {
28			parent::__construct($name);
29			}
30
31			/**
32			* Actives the nodes.
33			*
34			* @param string $url The active URL.
35			* @param array $nodes The nodes.
36			* @param integer $level The node level.
37			* @return boolean Returns true in case of success, false othewise.
38			*/
39			public function activeNodes($url, array $nodes = [], $level = 0) {
40
41			// Initialize.
42			$parent = false;
43			$current = false;
			0 ignored issues – show Unused Code introduced 2018-07-11 09:17 UTC by Report Bug Copy Issue Report `$current` 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...
44			$delete = false;
45
46			// Handle each node.
47			foreach ($nodes as $n) {
48
49			// Check the node.
50			if (false === ($n instanceOf AbstractNavigationNode)) {
51			continue;
52			}
53
54			// Determines if the current node matches the URL.
55			if ($url === $n->getRoute()) {
56			$current = true;
57			$delete = true;
58			} else {
59			$current = $this->activeNodes($url, $n->getNodes(), $level + 1);
60			}
61
62			// Handle next node.
63			if (false === $current) {
64			continue;
65			}
66
67			// Mark the node as active.
68			$n->setActive(true);
69
70			// Remove the icon only on the last level.
71			if (true === $delete && 0 < $level) {
72			$n->setIcon(null);
73			}
74
75			// Reset.
76			$parent = true;
77			$current = false;
			0 ignored issues – show Unused Code introduced 2018-07-11 09:17 UTC by Report Bug Copy Issue Report `$current` 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...
78			$delete = false;
79			}
80
81			// Return.
82			return $parent;
83			}
84
85			/**
86			* Get the breadcrumbs.
87			*
88			* @param AbstractNavigationNode $node The navigation node.
89			* @return AsbtractNavigationNode[] Returns the navigation nodes.
90			*/
91			public function getBreadcrumbs(AbstractNavigationNode $node = null) {
92
93			// Create the breadcrumbs.
94			$breadcrumbs = [];
95
96			// Correct the parameter if necessary.
97			if (null === $node) {
98			$node = $this;
99			}
100
101			// Check the instance.
102			if (true === ($node instanceof NavigationNode \|\| $node instanceof BreadcrumbNode) && true === $node->getActive()) {
103			$breadcrumbs[] = $node;
104			}
105
106			// Handle each node.
107			foreach ($node->getNodes() as $current) {
108			if (false === ($current instanceof AbstractNavigationNode)) {
109			continue;
110			}
111			$breadcrumbs = array_merge($breadcrumbs, $this->getBreadcrumbs($current));
112			}
113
114			// Return the breadcrumbs.
115			return $breadcrumbs;
116			}
117
118			}
119

webeweb / bootstrap-bundle

Push — master ( 0295ab...9b2687 )

NavigationTree::activeNodes() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like