Domain\CategoryStructure\Structure\Tree\TreeManager

Complexity

Total Complexity

2

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	assignParent()	0	12	1

<?php
/**
 * (c) Tomasz Kunicki <kunicki.tomasz@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace Domain\CategoryStructure\Structure\Tree;

use Domain\Category\Entity\CategoryInterface;
use Domain\CategoryStructure\Repository\CategoryTreeRepositoryInterface;

/**
 * Class TreeManager
 *
 * @package Domain\Category\Structure\Tree
 */
class TreeManager
{
    /** @var  CategoryTreeRepositoryInterface */
    private $categoryTreeRepository;

    /**
     * TreeManager constructor.
     *
     * @param CategoryTreeRepositoryInterface $categoryTreeRepository
     */
    public function __construct(CategoryTreeRepositoryInterface $categoryTreeRepository)
    {
        $this->categoryTreeRepository = $categoryTreeRepository;
    }

    /**
     * @param CategoryInterface $parent
     * @param CategoryInterface $newChild
     */
    public function assignParent(CategoryInterface $parent, CategoryInterface $newChild)

The parameter $parent is not used and could be removed.

The parameter $newChild is not used and could be removed.

    {
//        $oldParent = $this->categoryTreeRepository->getParent($newChild);

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

//
//        if (!is_null($oldParent)) {
//            $oldParent->removeChild($newChild);
//            $this->categoryTreeRepository->update($oldParent);
//
//        }
//        $parent->addChild($newChild);
//        $this->categoryTreeRepository->update($parent);
    }
}