Stu\Module\Tick\Colony\Component\ProceedStorage

Complexity

Total Complexity

12

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

4.76%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	work()	0	59	11
A	__construct()	0	5	1

<?php

namespace Stu\Module\Tick\Colony\Component;

use Stu\Lib\Information\InformationInterface;
use Stu\Lib\Transfer\Storage\StorageManagerInterface;
use Stu\Module\Commodity\Lib\CommodityCacheInterface;
use Stu\Orm\Entity\ColonyInterface;
use Stu\Orm\Repository\ColonyDepositMiningRepositoryInterface;

class ProceedStorage implements ColonyTickComponentInterface
{
    public function __construct(
        private readonly ColonyDepositMiningRepositoryInterface $colonyDepositMiningRepository,
        private readonly CommodityCacheInterface $commodityCache,
        private readonly StorageManagerInterface $storageManager
    ) {}

    public function work(ColonyInterface $colony, array &$production, InformationInterface $information): void
    {
        $sum = $colony->getStorageSum();

        //DECREASE
        foreach ($production as $commodityId => $obj) {
            $amount = $obj->getProduction();
            $commodity = $this->commodityCache->get($commodityId);

            if ($amount < 0) {
                $amount = abs($amount);

                if ($commodity->isSaveable()) {
                    // STANDARD
                    $this->storageManager->lowerStorage(
                        $colony,
                        $this->commodityCache->get($commodityId),
                        $amount

It seems like $amount can also be of type double; however, parameter $amount of Stu\Lib\Transfer\Storage\StorageManagerInterface::lowerStorage() does only seem to accept integer, maybe add an additional type check?

                    );
                    $sum -= $amount;
                } else {
                    // EFFECTS
                    $depositMining = $this->colonyDepositMiningRepository->getCurrentUserDepositMinings($colony)[$commodityId];

                    $depositMining->setAmountLeft($depositMining->getAmountLeft() - $amount);

$depositMining->getAmountLeft() - $amount of type double is incompatible with the type integer expected by parameter $amountLeft of Stu\Orm\Entity\ColonyDepositMiningInterface::setAmountLeft().

                    $this->colonyDepositMiningRepository->save($depositMining);
                }
            }
        }

        foreach ($production as $commodityId => $obj) {

            $commodity = $this->commodityCache->get($commodityId);
            if ($obj->getProduction() <= 0 || !$commodity->isSaveable()) {
                continue;
            }
            if ($sum >= $colony->getMaxStorage()) {
                if ($colony->getUser()->isStorageNotification()) {
                    $information->addInformation('Das Lager der Kolonie ist voll');
                }
                break;
            }
            if ($sum + $obj->getProduction() > $colony->getMaxStorage()) {
                $this->storageManager->upperStorage(
                    $colony,
                    $commodity,
                    $colony->getMaxStorage() - $sum
                );
                if ($colony->getUser()->isStorageNotification()) {
                    $information->addInformation('Das Lager der Kolonie ist voll');
                }
                break;
            }
            $this->storageManager->upperStorage(
                $colony,
                $commodity,
                $obj->getProduction()
            );
            $sum += $obj->getProduction();
        }
    }
}