ynloultratech / graphql-bundle

src/Mutation/DeleteBatchNode.php

<?php
/*******************************************************************************
 *  This file is part of the GraphQL Bundle package.
 *
 *  (c) YnloUltratech <support@ynloultratech.com>
 *
 *  For the full copyright and license information, please view the LICENSE
 *  file that was distributed with this source code.
 ******************************************************************************/

namespace Ynlo\GraphQLBundle\Mutation;

use GraphQL\Error\UserError;
use Symfony\Component\Form\FormEvent;
use Ynlo\GraphQLBundle\Exception\Controlled\NotFoundError;
use Ynlo\GraphQLBundle\Model\NodeInterface;
use Ynlo\GraphQLBundle\Validator\ConstraintViolationList;

/**
 * Class DeleteBatchNodeMutation
 */
class DeleteBatchNode extends AbstractMutationResolver
{
    /**
     * @var array
     */
    protected $deletedRecords = [];

    /**
     * {@inheritdoc}
     */
    public function process(&$data)
    {
        foreach ($data['ids'] as $item) {
            $this->preDelete($item);
            foreach ($this->extensions as $extension) {
                $extension->preDelete($item, $this, $this->context);
            }

            $this->deletedRecords[] = clone $item; //clone required to avoid node without id after delete
            $this->getManager()->remove($item);
        }

        $this->getManager()->flush();

        foreach ($data['ids'] as $item) {
            $this->postDelete($item);
            foreach ($this->extensions as $extension) {
                $extension->postDelete($item, $this, $this->context);
            }
        }
    }

    /**
     * {@inheritdoc}
     */
    public function returnPayload($data, ConstraintViolationList $violations, $inputSource)
    {
        $class = $this->getPayloadClass();

        return new $class($this->deletedRecords, $inputSource['clientMutationId'] ?? null);
    }

    /**
     * {@inheritDoc}
     */
    public function onSubmit(FormEvent $event)
    {
        $node = $this->context->getDefinition()->getNode();
        $class = $this->context->getEndpoint()->getClassForType($node);

It seems like $node can also be of type null; however, parameter $type of Ynlo\GraphQLBundle\Definition\Registry\Endpoint::getClassForType() does only seem to accept string, maybe add an additional type check?

        if ($event->getForm()->get('ids') && is_array($event->getForm()->get('ids')->getData())) {
            foreach ($event->getForm()->get('ids')->getData() as $node) {
                if (!$node instanceof NodeInterface || !$node->getId() || !is_a($node, $class)) {
                    throw new NotFoundError();
                }
            }
        } else {
            throw new UserError('Batch error, invalid data');
        }
    }

    /**
     * @param NodeInterface $node
     */
    protected function preDelete(NodeInterface $node)

The parameter $node is not used and could be removed.

    {
        //override
    }

    /**
     * @param NodeInterface $node
     */
    protected function postDelete(NodeInterface $node)

The parameter $node is not used and could be removed.

    {
        //override
    }
}