Stu\Module\Ship\Action\DeactivateRPGModule\DeactivateRPGModule

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	performSessionCheck()	0	3	1
A	__construct()	0	8	1
A	handle()	0	22	2

<?php

declare(strict_types=1);

namespace Stu\Module\Ship\Action\DeactivateRPGModule;

use request;
use Stu\Component\Ship\System\ShipSystemTypeEnum;
use Stu\Module\Control\ActionControllerInterface;
use Stu\Module\Control\GameControllerInterface;
use Stu\Module\Ship\Lib\ActivatorDeactivatorHelperInterface;
use Stu\Module\Ship\View\ShowShip\ShowShip;
use Stu\Module\Ship\Lib\ShipLoaderInterface;
use Stu\Orm\Repository\ShipRepositoryInterface;

final class DeactivateRPGModule implements ActionControllerInterface
{
    public const ACTION_IDENTIFIER = 'B_DEACTIVATE_RPG_Module';

    private ActivatorDeactivatorHelperInterface $helper;

    private ShipLoaderInterface $shipLoader;

    private ShipRepositoryInterface $shipRepository;

    public function __construct(
        ShipLoaderInterface $shipLoader,
        ShipRepositoryInterface $shipRepository,
        ActivatorDeactivatorHelperInterface $helper
    ) {
        $this->shipLoader = $shipLoader;
        $this->shipRepository = $shipRepository;
        $this->helper = $helper;
    }

    public function handle(GameControllerInterface $game): void
    {
        $game->setView(ShowShip::VIEW_IDENTIFIER);

        $userId = $game->getUser()->getId();

        $this->helper->deactivate(request::indInt('id'), ShipSystemTypeEnum::SYSTEM_RPG_MODULE, $game, true);

        $ship = $this->shipLoader->getByIdAndUser(
            request::indInt('id'),
            $userId
        );

        if ($ship->isDisabled() == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $ship->setDisabled(false);
        }

        $ship->setCanBeDisabled(false);

        $this->shipRepository->save($ship);

        $game->addInformation("Das RPG Modul wurde deaktiviert");
    }
    public function performSessionCheck(): bool
    {
        return true;
    }
}