LaravelFreelancerNL\FluentAQL\Traits\NormalizesMiscellaneousFunctions

Complexity

Total Complexity

4

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	normalizeFirstDocument()	0	3	1
A	normalizeDocument()	0	16	3

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\FluentAQL\Traits;

use LaravelFreelancerNL\FluentAQL\QueryBuilder;

/**
 * Trait hasFunctions.
 *
 * AQL Function API calls.
 */
trait NormalizesMiscellaneousFunctions
{
    protected function normalizeDocument(QueryBuilder $queryBuilder): void
    {
        if ($this->parameters['id']  === null) {
            $this->parameters['id']  = $this->parameters['collection'];

The property parameters does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            unset($this->parameters['collection']);
        }

        if (isset($this->parameters['collection'])) {
            $this->parameters['collection'] = $queryBuilder->normalizeArgument(
                $this->parameters['collection'],
                ['Collection', 'Id', 'Query', 'Bind']
            );
        }
        $this->parameters['id'] = $queryBuilder->normalizeArgument(
            $this->parameters['id'],
            ['Id', 'Key', 'Query', 'List', 'Bind']
        );
    }

    protected function normalizeFirstDocument(QueryBuilder $queryBuilder): void
    {
        $this->normalizeAny($queryBuilder);

The method normalizeAny() does not exist on LaravelFreelancerNL\FluentAQL\Traits\NormalizesMiscellaneousFunctions. Did you maybe mean normalizeDocument()?

    }
}