O2System\Framework\Models\Sql\Traits\HierarchicalTrait

Complexity

Total Complexity

21

Size/Duplication

Total Lines	210
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getParents()	0	21	4
A	getNumChilds()	0	11	2
A	hasChilds()	0	11	3
A	hasParent()	0	11	3
A	getChilds()	0	17	4
A	rebuildTree()	0	54	5

<?php
/**
 * This file is part of the O2System PHP Framework package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @author         Steeve Andrian Salim
 * @copyright      Copyright (c) Steeve Andrian Salim
 */

// ------------------------------------------------------------------------

namespace O2System\Framework\Models\Sql\Traits;

// ------------------------------------------------------------------------

/**
 * Class HierarchicalTrait
 *
 * @package O2System\Framework\Models\Sql\Traits
 */
trait HierarchicalTrait
{
    /**
     * Parent Key Field
     *
     * @access  public
     * @type    string
     */
    public $parentKey = 'id_parent';

    /**
     * Hierarchical Enabled Flag
     *
     * @access  protected
     * @type    bool
     */
    protected $hierarchical = true;

    /**
     * Rebuild Tree
     *
     * Rebuild self hierarchical table
     *
     * @access public
     *
     * @param string $table Working database table
     *
     * @return int  Right column value
     */
    protected function rebuildTree($idParent = 0, $left = 1, $depth = 0)
    {
        ini_set('xdebug.max_nesting_level', 10000);
        ini_set('memory_limit', '-1');

        if ($result = $this->qb
            ->table($this->table)
            ->select('id')
            ->where('id_parent', $idParent)
            ->orderBy('id')
            ->get()) {

            $right = $left + 1;

            $i = 0;
            foreach ($result as $row) {
                if ($i == 0) {
                    $this->qb
                        ->from($this->table)
                        ->where('id', $row->id)
                        ->update($update = [
                            'record_left'  => $left,
                            'record_right' => $right,
                            'record_depth' => $depth,
                        ]);
                } else {
                    $this->qb
                        ->from($this->table)
                        ->where('id', $row->id)
                        ->update($update = [
                            'record_left'  => $left = $right + 1,
                            'record_right' => $right = $left + 1,
                            'record_depth' => $depth,
                        ]);
                }

                $update[ 'id' ] = $row->id;

                if ($this->hasChilds($row->id)) {
                    $right = $this->rebuildTree($row->id, $right, $depth + 1);
                    $this->qb
                        ->from($this->table)
                        ->where('id', $row->id)
                        ->update($update = [
                            'record_right' => $right,
                        ]);
                    $update[ 'id' ] = $row->id;
                }

                $i++;
            }
        }

        return $right + 1;

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

    }

    // ------------------------------------------------------------------------

    /**
     * Has Childs
     *
     * Check if there is a child rows
     *
     * @param string $table Working database table
     *
     * @access public
     * @return bool
     */
    protected function hasChilds($idParent)
    {
        if ($result = $this->qb
            ->table($this->table)
            ->select('id')
            ->where('id_parent', $idParent)
            ->get()) {
            return (bool)($result->count() == 0 ? false : true);
        }

        return false;
    }

    // ------------------------------------------------------------------------

    /**
     * Find Parents
     *
     * Retreive parents of a record
     *
     * @param numeric $id Record ID

The type O2System\Framework\Models\Sql\Traits\numeric was not found. Maybe you did not declare it correctly or list all dependencies?

     *
     * @access public
     * @return array
     */
    protected function getParents($id, &$parents = [])
    {
        if ($result = $this->qb
            ->from($this->table)
            ->whereIn('id', $this->qb
                ->subQuery()
                ->from($this->table)
                ->select('id_parent')
                ->where('id', $id)
            )
            ->get()) {
            if ($result->count()) {
                $parents[] = $row = $result->first();

                if ($this->hasParent($row->id_parent)) {
                    $this->getParents($row->id, $parents);
                }
            }
        }

        return array_reverse($parents);
    }

    protected function hasParent($idParent)
    {
        if ($result = $this->qb
            ->table($this->table)
            ->select('id')
            ->where('id', $idParent)
            ->get()) {
            return (bool)($result->count() == 0 ? false : true);
        }

        return false;
    }
    // ------------------------------------------------------------------------

    /**
     * Find Childs
     *
     * Retreive all childs
     *
     * @param numeric $idParent Parent ID
     *
     * @access public
     * @return array
     */
    protected function getChilds($idParent)
    {
        $childs = [];

        if($result = $this->qb
            ->table($this->table)
            ->where('id_parent', $idParent)
            ->get()) {
            foreach ($result as $row) {
                $childs[] = $row;
                if ($this->hasChild($row->id)) {

The method hasChild() does not exist on O2System\Framework\Models\Sql\Traits\HierarchicalTrait. Did you maybe mean hasChilds()?

                    $row->childs = $this->getChilds($row->id);
                }
            }
        }

        return $childs;
    }
    // ------------------------------------------------------------------------

    /**
     * Count Childs
     *
     * Num childs of a record
     *
     * @param numeric $idParent Record Parent ID
     *
     * @access public
     * @return bool
     */
    protected function getNumChilds($idParent)
    {
        if($result = $this->qb
            ->table($this->table)
            ->select('id')
            ->where('id_parent', $idParent)
            ->get()) {
            return $result->count();
        }

        return 0;

The expression return 0 returns the type integer which is incompatible with the documented return type boolean.

    }
}