Application\Api\Field\Mutation\ExportAccountingReport::build()

Complexity

Conditions	2
Paths	1

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	2.2793

Importance

Changes

0

<?php
    public static function build(): array
    {
        return [
            'name' => 'exportAccountingReport',
            'type' => Type::nonNull(Type::string()),
            'description' => 'Prepare an accounting report and return the URL to download it',
            'args' => [
                'date' => _types()->get('Date'),
                'showBudget' => Type::nonNull(Type::boolean()),
            ],
            'resolve' => function ($root, array $args, SessionInterface $session): string {
                global $container;

                /** @var AccountingReport $exporter */
                $exporter = $container->get(AccountingReport::class);

                if ($args['date']) {
                    $exporter->setDate($args['date']);
                }
                $exporter->showBudget($args['showBudget']);

                // Select root accounts
                /** @var AccountRepository $accountRepository */
                $accountRepository = _em()->getRepository(Account::class);
                $rootAccountsQuery = $accountRepository->getRootAccountsQuery();

                return $exporter->export($rootAccountsQuery->getResult());

It seems like $rootAccountsQuery->getResult() can also be of type integer; however, parameter $items of Application\Service\Exporter\AbstractExporter::export() does only seem to accept iterable, maybe add an additional type check?

            },