Xoopsmodules\instruction\Tree::_makePagesUserTree()

Complexity

Conditions	16
Paths	582

Size

Total Lines	101
Code Lines	41

Duplication

Lines	5
Ratio	4.95 %

Importance

Changes

0

<?php namespace Xoopsmodules\instruction;
    public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
    {

        // Сохраняем значение предыдущей страницы
        //static $stat_prevpages;

        if ($key > 0) {

            // ID страницы
            $pageid = $this->tree[$key]['obj']->getVar('pageid');
            // Название страницы
            $pagetitle = $this->tree[$key]['obj']->getVar('title');
            // Тип страницы
            $pagetype = $this->tree[$key]['obj']->getVar('type');

            // Дочернии категории
            $page_childs = $this->getAllChild($pageid);
            // Число дочерних страниц
            $num_childs = count($page_childs);

            // Генерируем класс
            // InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
            $class = [];
            // Данный класс должен быть у любого узла
            $class[] = 'InstrTreeNode';
            // Если узел нулевого уровня, добавляем InstrTreeIsRoot
            if (0 === $level) {
                $class[] = 'InstrTreeIsRoot';
            }
            // Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
            // Если у узла нет потомков - InstrTreeExpandLeaf
            if (0 == $num_childs) {
                $class[] = 'InstrTreeExpandLeaf';
                // Если у искомого элемента есть потомки - открываем список
            } elseif ($currpageid == $pageid) {
                $class[] = 'InstrTreeExpandOpen';
                // Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
            } elseif (array_key_exists($currpageid, $page_childs)) {
                $class[] = 'InstrTreeExpandOpen';
                //
            } else {
                $class[] = 'InstrTreeExpandClosed';
            }

            // Данный класс нужно добавлять последнему узлу в каждом уровне

            if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
                $class[] = 'InstrTreeIsLast';
            }

            //$class[] = 'InstrTreeIsLast';

63% of this comment could be valid code. Did you maybe forget this after debugging?

            // Test
            //$ret .= '<div id="' . $pageid . '">';

            // Создаём запись
            $ret .= '<li class="' . implode(' ', $class) . '">';
            //
            $ret .= '<div class="InstrTreeExpand"></div>';
            //
            $ret .= '<div class="InstrTreeContent">';

            // Если это лист дерева
            if (0 == $pagetype) {
                $ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
                //
            } elseif ($currpageid == $pageid) {
                $ret .= $pagetitle;
                //
            } else {
                $ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
            }

            $ret .= '</div>';

            // Если есть потомки
            if ($num_childs > 0) {
                $ret .= '<ul class="InstrTreeContainer">';
            }

            // Инкримент уровня
            $level++;
        }

        // Рекурсия
        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {

This code seems to be duplicated across your project.

            foreach ($this->tree[$key]['child'] as $childkey) {
                $this->_makePagesUserTree($childkey, $ret, $currpageid, $lastpageids, $level);
            }
        }

        // Test
        if ($key > 0) {
            // Если есть потомки
            if ($num_childs > 0) {

The variable $num_childs does not seem to be defined for all execution paths leading up to this point.

                $ret .= '</ul>';
            }
            // Конец текущей записи
            $ret .= '</li>';
        }
    }