Stu\Module\Ship\Lib\AstroEntryLib

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

8.33%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	cancelAstroFinalizing()	0	16	2
A	finish()	0	17	2

<?php

declare(strict_types=1);

namespace Stu\Module\Ship\Lib;

use RuntimeException;
use Stu\Component\Ship\AstronomicalMappingEnum;
use Stu\Component\Ship\ShipStateEnum;
use Stu\Orm\Repository\AstroEntryRepositoryInterface;

final class AstroEntryLib implements AstroEntryLibInterface
{
    private AstroEntryRepositoryInterface $astroEntryRepository;

    public function __construct(
        AstroEntryRepositoryInterface $astroEntryRepository
    ) {
        $this->astroEntryRepository = $astroEntryRepository;
    }

    public function cancelAstroFinalizing(ShipWrapperInterface $wrapper): void
    {
        $ship = $wrapper->get();

        $ship->setState(ShipStateEnum::SHIP_STATE_NONE);
        $astroLab = $wrapper->getAstroLaboratorySystemData();
        if ($astroLab === null) {
            throw new RuntimeException('this should not happen');
        }
        $astroLab->setAstroStartTurn(null)->update();

        $entry = $this->astroEntryRepository->getByShipLocation($ship, false);

        $entry->setState(AstronomicalMappingEnum::MEASURED);
        $entry->setAstroStartTurn(null);
        $this->astroEntryRepository->save($entry);

It seems like $entry can also be of type null; however, parameter $entry of Stu\Orm\Repository\AstroEntryRepositoryInterface::save() does only seem to accept Stu\Orm\Entity\AstronomicalEntryInterface, maybe add an additional type check?

    }

    public function finish(ShipWrapperInterface $wrapper): void
    {
        $ship = $wrapper->get();

        $ship->setState(ShipStateEnum::SHIP_STATE_NONE);

        $astroLab = $wrapper->getAstroLaboratorySystemData();
        if ($astroLab === null) {
            throw new RuntimeException('this should not happen');
        }
        $astroLab->setAstroStartTurn(null)->update();

        $entry = $this->astroEntryRepository->getByShipLocation($ship, false);

        $entry->setState(AstronomicalMappingEnum::DONE);
        $entry->setAstroStartTurn(null);
        $this->astroEntryRepository->save($entry);

It seems like $entry can also be of type null; however, parameter $entry of Stu\Orm\Repository\AstroEntryRepositoryInterface::save() does only seem to accept Stu\Orm\Entity\AstronomicalEntryInterface, maybe add an additional type check?

    }
}