Modelarium\Laravel\Directives\RenderableDirective::processModelFieldDirective()

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

<?php declare(strict_types=1);
    public static function processModelFieldDirective(
        ModelGenerator $generator,
        \GraphQL\Type\Definition\FieldDefinition $field,
        \GraphQL\Language\AST\DirectiveNode $directive
    ): void {
        foreach ($directive->arguments as $arg) {
            /**
             * @var \GraphQL\Language\AST\ArgumentNode $arg
             */

            $argName = $arg->name->value;
            $argValue = $arg->value->value; /** @phpstan-ignore-line */

Accessing value on the interface GraphQL\Language\AST\ValueNode suggest that you code against a concrete implementation. How about adding an instanceof check?

            $generator->fModel->appendRenderable($argName, $argValue);

The method appendRenderable() does not exist on Formularium\Model.

        }