Application\Api\Field\Mutation\SuggestDeletion

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	build()	0	26	2

<?php

declare(strict_types=1);

namespace Application\Api\Field\Mutation;

use Application\Api\Helper;
use Application\Enum\ChangeType;
use Application\Enum\Site;
use Application\Model\Card;
use Application\Model\Change;
use Application\Repository\ChangeRepository;
use Ecodev\Felix\Api\Field\FieldInterface;
use GraphQL\Type\Definition\Type;

class SuggestDeletion implements FieldInterface
{
    public static function build(): iterable
    {
        yield 'suggestDeletion' => fn () => [
            'type' => Type::nonNull(_types()->getOutput(Change::class)),
            'description' => 'Suggest the deletion of an existing image',
            'args' => [
                'id' => Type::nonNull(_types()->getId(Card::class)),
                'request' => Type::nonNull(Type::string()),
            ],
            'resolve' => function ($root, array $args): Change {
                /** @var Site $site */
                $site = $root['site'];

                $original = $args['id']->getEntity();

                /** @var ChangeRepository $changeRepository */
                $changeRepository = _em()->getRepository(Change::class);
                $change = $changeRepository->getOrCreate(ChangeType::Delete, $original, $args['request'], $site);

                Helper::throwIfDenied($change, 'create');

                if (!$change->getId()) {

The expression $change->getId() of type integer|null is loosely compared to false; this is ambiguous if the integer can be 0. You might want to explicitly use === null instead.

                    _em()->flush();
                }

                return $change;
            },
        ];
    }
}