Application\Api\Field\Mutation\UpdateTransaction

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

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

<?php

declare(strict_types=1);

namespace Application\Api\Field\Mutation;

use Application\Api\Field\FieldInterface;
use Application\Api\Helper;
use Application\Model\Transaction;
use Application\Model\TransactionLine;
use GraphQL\Type\Definition\Type;
use Zend\Expressive\Session\SessionInterface;

abstract class UpdateTransaction implements FieldInterface
{
    public static function build(): array
    {
        return [
            'name' => 'updateTransaction',
            'type' => Type::nonNull(_types()->getOutput(Transaction::class)),
            'description' => 'Update a transaction, and optionally replace all its transaction lines if given any',
            'args' => [
                'id' => Type::nonNull(_types()->getId(Transaction::class)),
                'input' => Type::nonNull(_types()->getPartialInput(Transaction::class)),
                'lines' => Type::listOf(Type::nonNull(_types()->getInput(TransactionLine::class))),
            ],
            'resolve' => function ($root, array $args, SessionInterface $session): Transaction {

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

                // Do it
                $transaction = $args['id']->getEntity();
                $input = $args['input'];
                Helper::hydrate($transaction, $input);

                // Check ACL
                Helper::throwIfDenied($transaction, 'update');
                $lines = $args['lines'];

                if ($lines !== null) {
                    _em()->getRepository(Transaction::class)->hydrateLinesAndFlush($transaction, $lines);
                } else {
                    _em()->flush();
                }

                return $transaction;

The expression return $transaction could return the type Application\Model\AbstractModel which includes types incompatible with the type-hinted return Application\Model\Transaction. Consider adding an additional type-check to rule them out.

            },
        ];
    }
}