Stu\Module\NPC\Action\CommodityCheat::handle()

Complexity

Conditions	8
Paths	7

Size

Total Lines	66
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(GameControllerInterface $game): void
    {
        $game->setView(ShowTools::VIEW_IDENTIFIER);
        $user = $game->getUser();

        // only Admins or NPC can trigger
        if (!$game->isAdmin() && !$game->isNpc()) {
            $game->addInformation(_('[b][color=#ff2626]Aktion nicht möglich, Spieler ist kein Admin/NPC![/color][/b]'));
            return;
        }

        if (!request::getVarByMethod(request::postvars(), 'shipid')) {
            $game->addInformation("Kein Schiff ausgewählt");
            return;
        } else {
            $shipId = request::postInt('shipid');
            $commodityId = request::postInt('commodityid');
            $amount = request::postInt('amount');
            $reason = request::postString('reason');


            $wrapper = $this->shipLoader->find($shipId);

            if ($wrapper === null) {
                throw new ShipDoesNotExistException(_('Ship does not exist!'));
            } else {
                $ship = $wrapper->get();
            }

            if ($amount < 1) {
                $game->addInformation("Anzahl muss größer als 0 sein");
                return;
            }

            if ($reason === '') {

The condition $reason === '' is always false.

                $game->addInformation("Grund fehlt");
                return;
            }

            $commodity = $this->commodityRepository->find($commodityId);

            if ($commodity === null) {
                $game->addInformation("Ungültige Ware");
                return;
            }

            $this->shipStorageManager->upperStorage(
                $ship,
                $commodity,
                $amount
            );

            $text = sprintf(
                '%s hat dem Schiff %s (%d) von Spieler %s (%d) %d %s hinzugefügt. Grund: %s',
                $user->getName(),
                $ship->getName(),
                $ship->getId(),
                $ship->getUser()->getName(),
                $ship->getUser()->getId(),
                $amount,
                $commodity->getName(),
                $reason

$reason of type false is incompatible with the type double|integer|string expected by parameter $values of sprintf().

            );

            $this->createEntry($text, $user->getId());
            $game->addInformation("Waren hinzugefügt");
        }