unil-lettres / dilps-tiresias

server/Application/Api/Field/Mutation/AcceptChange.php

<?php

declare(strict_types=1);

namespace Application\Api\Field\Mutation;

use Application\Api\Helper;
use Application\Enum\ChangeType;
use Application\Model\Card;
use Application\Model\Change;
use Application\Model\User;
use Doctrine\ORM\Event\PreUpdateEventArgs;
use Ecodev\Felix\Api\Field\FieldInterface;
use Exception;
use GraphQL\Type\Definition\Type;

class AcceptChange implements FieldInterface
{
    public static function build(): iterable
    {
        yield 'acceptChange' => fn () => [
            'type' => _types()->getOutput(Card::class),
            'description' => 'Accept the change and return the modified Image, unless if it has been deleted',
            'args' => [
                'id' => Type::nonNull(_types()->getId(Change::class)),
            ],
            'resolve' => function ($root, array $args): ?Card {
                /** @var Change $change */
                $change = $args['id']->getEntity();
                Helper::throwIfDenied($change, 'update');

                $image = null;
                switch ($change->getType()) {
                    case ChangeType::Create:
                        $image = $change->getSuggestion();
                        $image->setOwner(User::getCurrent());

                        $changeSet = ['fakeProperty' => ['foo' => 'bar']];
                        $fakeEvent = new PreUpdateEventArgs($image, _em(), $changeSet);

It seems like $image can also be of type null; however, parameter $entity of Doctrine\ORM\Event\PreUpdateEventArgs::__construct() does only seem to accept object, maybe add an additional type check?

                        $image->timestampUpdate($fakeEvent);

                        break;
                    case ChangeType::Update:
                        $image = $change->getOriginal();
                        $change->getSuggestion()->copyInto($image);

It seems like $image can also be of type null; however, parameter $original of Application\Model\Card::copyInto() does only seem to accept Application\Model\Card, maybe add an additional type check?

                        _em()->remove($change->getSuggestion());

It seems like $change->getSuggestion() can also be of type null; however, parameter $object of Doctrine\ORM\EntityManager::remove() does only seem to accept object, maybe add an additional type check?

                        break;
                    case ChangeType::Delete:
                        $original = $change->getOriginal();

                        // Trigger proxy loading, so the image on disk will be able to be deleted after the flush
                        $original->getPath();

                        _em()->remove($original);

                        break;
                    default:
                        throw new Exception('Unsupported change type: ' . $change->getType()->value);
                }

                _em()->remove($change);
                _em()->flush();

                return $image;
            },
        ];
    }
}