Yiisoft\Http\Header\AcceptHeader

Complexity

Total Complexity

14

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
C	collect()	0	42	12
A	__construct()	0	6	2

<?php

declare(strict_types=1);

namespace Yiisoft\Http\Header;

use InvalidArgumentException;
use Yiisoft\Http\Header\Value\Accept\Accept;
use Yiisoft\Http\Header\Internal\BaseHeaderValue;

final class AcceptHeader extends Header
{
    // todo: comparing
    protected const DEFAULT_VALUE_CLASS = Accept::class;

    public function __construct(string $nameOrClass)
    {
        parent::__construct($nameOrClass);
        if (!is_a($this->headerClass, Accept::class, true)) {
            throw new InvalidArgumentException(
                sprintf("%s class is not an instance of %s", $this->headerClass, Accept::class)
            );
        }
    }

    /**
     * Add value in order
     */
    protected function collect(BaseHeaderValue $value): void
    {
        if (count($this->collection) === 0) {
            $this->collection[] = $value;
            return;
        }
        for ($pos = array_key_last($this->collection); $pos >= 0; --$pos) {
            $item = $this->collection[$pos];
            $result = (float)$item->getQuality() <=> (float)$value->getQuality();

The method getQuality() does not exist on Yiisoft\Http\Header\Internal\BaseHeaderValue. It seems like you code against a sub-type of Yiisoft\Http\Header\Internal\BaseHeaderValue such as Yiisoft\Http\Header\Internal\WithParamsHeaderValue.

            if ($result > 0) {
                break;
            } elseif ($result === 0) {
                $separator = $this->headerClass::VALUE_SEPARATOR;
                if ($separator !== '') {
                    $itemTypes = array_reverse(explode($separator, $item->getValue()));
                    $valueTypes = array_reverse(explode($separator, $value->getValue()));
                } else {
                    $itemTypes = [$item->getValue()];
                    $valueTypes = [$value->getValue()];
                }
                $result = count($itemTypes) <=> count($valueTypes);
                if ($result > 0) {
                    break;
                } elseif ($result === 0) {
                    foreach ($itemTypes as $part => $itemType) {
                        if ($itemType === '*' xor $valueTypes[$part] === '*') {
                            if ($itemType !== '*') {
                                break 2;
                            } else {
                                $this->collection[$pos + 1] = $item;
                                continue 2;
                            }
                        }
                    }
                    if (count($item->getParams()) >= count($value->getParams())) {

The method getParams() does not exist on Yiisoft\Http\Header\Internal\BaseHeaderValue. It seems like you code against a sub-type of Yiisoft\Http\Header\Internal\BaseHeaderValue such as Yiisoft\Http\Header\Internal\WithParamsHeaderValue.

                        break;
                    }
                }
            }
            $this->collection[$pos + 1] = $item;
        }
        $this->collection[$pos + 1] = $value;
    }
}