SuggestCreation::build() - Code Metrics - unil-lettres/dilps-tiresias - Measure and Improve Code Quality continuously with Scrutinizer

SuggestCreation::build() A
last analyzed 2025-08-05 08:14 UTC

↳ Parent: SuggestCreation

Complexity

Conditions	2
Paths	1

Size

Total Lines	26
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	17
CRAP Score	2

Importance

Changes

Metric	Value
eloc	15
c	0
b	0
f	0
dl	0
loc	26
ccs	17
cts	17
cp	1
rs	9.7666
cc	2
nc	1
nop	0
crap	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 SuggestCreation implements FieldInterface
{
    public static function build(): iterable
    {
        yield 'suggestCreation' => fn () => [
            'type' => Type::nonNull(_types()->getOutput(Change::class)),
            'description' => 'Suggest the creation of a new 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'];

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

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

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

                if (!$change->getId()) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
                    _em()->flush();
                }

                return $change;
            },
        ];
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Application\Api\Field\Mutation;
6
7		use Application\Api\Helper;
8		use Application\Enum\ChangeType;
9		use Application\Enum\Site;
10		use Application\Model\Card;
11		use Application\Model\Change;
12		use Application\Repository\ChangeRepository;
13		use Ecodev\Felix\Api\Field\FieldInterface;
14		use GraphQL\Type\Definition\Type;
15
16		class SuggestCreation implements FieldInterface
17		{
18	1	public static function build(): iterable
19		{
20	1	yield 'suggestCreation' => fn () => [
21	1	'type' => Type::nonNull(_types()->getOutput(Change::class)),
22	1	'description' => 'Suggest the creation of a new image',
23	1	'args' => [
24	1	'id' => Type::nonNull(_types()->getId(Card::class)),
25	1	'request' => Type::nonNull(Type::string()),
26	1	],
27	1	'resolve' => function ($root, array $args): Change {
28		/** @var Site $site */
29	1	$site = $root['site'];
30
31	1	$suggestion = $args['id']->getEntity();
32
33		/** @var ChangeRepository $changeRepository */
34	1	$changeRepository = _em()->getRepository(Change::class);
35	1	$change = $changeRepository->getOrCreate(ChangeType::Create, $suggestion, $args['request'], $site);
36
37	1	Helper::throwIfDenied($change, 'create');
38
39	1	if (!$change->getId()) {
		0 ignored issues – show Bug Best Practice introduced 2022-02-21 09:14 UTC by Report Bug Copy Issue Report 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. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
40	1	_em()->flush();
41		}
42
43	1	return $change;
44	1	},
45	1	];
46		}
47		}
48

unil-lettres / dilps-tiresias

SuggestCreation::build() A last analyzed 2025-08-05 08:14 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

SuggestCreation::build() A
last analyzed 2025-08-05 08:14 UTC