Leonidas\Library\Core\Asset\AbstractInlineAsset

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getCode()	0	3	1
A	shouldBeLoaded()	0	3	1
A	__construct()	0	6	1
A	getPolicy()	0	3	1
A	getHandle()	0	3	1

<?php

namespace Leonidas\Library\Core\Asset;

use Leonidas\Library\Core\Http\Policy\NoPolicy;
use Psr\Http\Message\ServerRequestInterface;
use WebTheory\HttpPolicy\ServerRequestPolicyInterface;

class AbstractInlineAsset
{
    protected string $handle;

    protected string $code;

    protected ?ServerRequestPolicyInterface $policy = null;

    public function __construct(string $handle, string $code, ?ServerRequestPolicyInterface $policy = null)
    {
        $this->handle = $handle;
        $this->code = $code;

        $this->policy = $policy ?? new NoPolicy();
    }

    public function getHandle(): string
    {
        return $this->handle;
    }

    public function getCode(): string
    {
        return $this->code;
    }

    public function getPolicy(): ServerRequestPolicyInterface
    {
        return $this->policy;

The expression return $this->policy could return the type null which is incompatible with the type-hinted return WebTheory\HttpPolicy\ServerRequestPolicyInterface. Consider adding an additional type-check to rule them out.

    }

    public function shouldBeLoaded(ServerRequestInterface $request): bool
    {
        return $this->policy->approvesRequest($request);

The method approvesRequest() does not exist on null.

    }
}