Leonidas\Library\Admin\Component\Page\Traits\NestedPageTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getParentSlug()	0	3	1
A	defineParentFile()	0	5	2
A	getParentFileResolver()	0	3	1
A	getSubmenuFileResolver()	0	3	1
A	defineSubmenuFile()	0	5	2

<?php

namespace Leonidas\Library\Admin\Component\Page\Traits;

use Leonidas\Contracts\Admin\ParentFileResolverInterface;
use Leonidas\Contracts\Admin\SubmenuFileResolverInterface;
use Psr\Http\Message\ServerRequestInterface;

trait NestedPageTrait
{
    protected string $parentSlug;

    protected ?ParentFileResolverInterface $parentFileResolver;

    protected ?SubmenuFileResolverInterface $submenuFileResolver;

    public function __construct(
        string $parentSlug,
        ?ParentFileResolverInterface $parentFileResolver = null,
        ?SubmenuFileResolverInterface $subMenuFileResolver = null
    ) {
        $this->parentSlug = $parentSlug;
        $this->parentFileResolver = $parentFileResolver;
        $this->submenuFileResolver = $subMenuFileResolver;
    }

    public function getParentSlug(): string
    {
        return $this->parentSlug;
    }

    public function getSubmenuFileResolver(): ?SubmenuFileResolverInterface
    {
        return $this->submenuFileResolver;
    }

    public function getParentFileResolver(): ?ParentFileResolverInterface
    {
        return $this->parentFileResolver;
    }

    public function defineSubmenuFile(ServerRequestInterface $request): string
    {
        return ($resolver = $this->getSubmenuFileResolver())

The expression return $resolver = $this->getSubmenuFileResolver() ? $resolver->resolveSubmenuFile($request) : null could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

            ? $resolver->resolveSubmenuFile($request)
            : null;
    }

    public function defineParentFile(ServerRequestInterface $request): string
    {
        return ($resolver = $this->getParentFileResolver())

The expression return $resolver = $this->getParentFileResolver() ? $resolver->resolveParentFile($request) : null could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

            ? $resolver->resolveParentFile($request)
            : null;
    }
}