Stu\Module\Colony\Lib\Gui\Component\BuildmenuProvider

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setTemplateVariables()	0	17	2

<?php

namespace Stu\Module\Colony\Lib\Gui\Component;

use Stu\Component\Building\BuildMenuEnum;
use Stu\Lib\Colony\PlanetFieldHostInterface;
use Stu\Module\Control\GameControllerInterface;
use Stu\Orm\Entity\ColonyInterface;
use Stu\Orm\Repository\BuildingRepositoryInterface;

final class BuildmenuProvider implements GuiComponentProviderInterface
{
    private BuildingRepositoryInterface $buildingRepository;

    public function __construct(
        BuildingRepositoryInterface $buildingRepository,
    ) {
        $this->buildingRepository = $buildingRepository;
    }

    /** @param ColonyInterface&PlanetFieldHostInterface $host */
    public function setTemplateVariables(
        PlanetFieldHostInterface $host,
        GameControllerInterface $game
    ): void {

        foreach (BuildMenuEnum::BUILDMENU_IDS as $id) {
            $menus[$id]['buildings'] = $this->buildingRepository->getByColonyAndUserAndBuildMenu(
                $host,
                $game->getUser()->getId(),
                $id,
                0
            );

            $menus[$id]['name'] = BuildMenuEnum::getDescription($id);
        }

        $game->setTemplateVar('BUILD_MENUS', $menus);

The variable $menus seems to be defined by a foreach iteration on line 27. Are you sure the iterator is never empty, otherwise this variable is not defined?

    }
}