php recursive function with reference

Question

I have recursive function with reference param. It goes through the navigation, checking if user is allowed to see current node. But when it goes to the sub-nodes ( $nodeItem['nodes'] ), it works good inside its instance, but doesn't return reduced array. So that's the problem. When it goes to subnodes , where I shouldn't have access, it unsets them in its instance, but when it returns array and I am back in main navigation nodes array, all subnodes are back. What could possibly be wrong?

private function getNodesByRole(&$nodesArray)
{
    foreach ($nodesArray as $nodeKey => $nodeItem) {
        $link = trim($nodeItem['href'], self::SEPARATOR_PRESENTER_LINK);

        if (substr_count($link, self::SEPARATOR_PRESENTER_LINK) == 2) {
            $resource = substr($link, 0, strrpos($link, self::SEPARATOR_PRESENTER_LINK));
            $privilege = substr($link, strrpos($link, self::SEPARATOR_PRESENTER_LINK) + 1);

            if ($this->user->isAllowed($resource, $privilege) === false) {
                unset($nodesArray[$nodeKey]);
                dump($nodeItem);
                dump($nodesArray);
            } else if (!empty($nodeItem['nodes'])) {
                $this->getNodesByRole($nodeItem['nodes']);
            }

        } else {
            if (!empty($nodeItem['nodes'])) {
                $this->getNodesByRole($nodeItem['nodes']);
            }
        }

        if (isset($nodeItem['nodes']) && count($nodeItem['nodes']) == 0) {
            unset($nodesArray[$nodeKey]);
            continue;
        }
    }
    return $nodesArray;
}

zenwraight · Accepted Answer · 2017-05-15 13:54:03Z

1

You need to set the returned array to the current states array, like this

private function getNodesByRole(&$nodesArray)
{
foreach ($nodesArray as $nodeKey => $nodeItem) {
    $link = trim($nodeItem['href'], self::SEPARATOR_PRESENTER_LINK);

    if (substr_count($link, self::SEPARATOR_PRESENTER_LINK) == 2) {
        $resource = substr($link, 0, strrpos($link, self::SEPARATOR_PRESENTER_LINK));
        $privilege = substr($link, strrpos($link, self::SEPARATOR_PRESENTER_LINK) + 1);

        if ($this->user->isAllowed($resource, $privilege) === false) {
            unset($nodesArray[$nodeKey]);
            dump($nodeItem);
            dump($nodesArray);
        } else if (!empty($nodeItem['nodes'])) {
            $nodesArray = $this->getNodesByRole($nodeItem['nodes']);
        }

    } else {
        if (!empty($nodeItem['nodes'])) {
            $nodesArray = $this->getNodesByRole($nodeItem['nodes']);
        }
    }

    if (isset($nodeItem['nodes']) && count($nodeItem['nodes']) == 0) {
        unset($nodesArray[$nodeKey]);
        continue;
    }
}
return $nodesArray;
}

Hope it helps!

answered May 15, 2017 at 13:54

zenwraight

1,9881 gold badge12 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

HS1993 Over a year ago

I just had to edit, that return is not saving to $nodesArray but to $nodesArray[$nodeKey]['nodes'] and it works great, thank you!!

Collectives™ on Stack Overflow

php recursive function with reference

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related