Leonidas\Library\Admin\Abstracts\MaybeHandlesCsrfTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	maybeRenderTokenField()	0	3	2
A	getTokenManager()	0	3	1
A	renderTokenField()	0	3	1
A	setTokenManager()	0	5	1

<?php

namespace Leonidas\Library\Admin\Abstracts;

use Leonidas\Contracts\Auth\CsrfManagerInterface;

trait MaybeHandlesCsrfTrait
{
    /**
     * @var ?CsrfManagerInterface
     */
    protected $csrfManager = null;

    /**
     * Get the value of csrfToken
     *
     * @return CsrfManagerInterface
     */
    public function getTokenManager(): CsrfManagerInterface
    {
        return $this->csrfManager;

The expression return $this->csrfManager could return the type null which is incompatible with the type-hinted return Leonidas\Contracts\Auth\CsrfManagerInterface. Consider adding an additional type-check to rule them out.

    }

    /**
     * Set the value of csrfToken
     *
     * @param CsrfManagerInterface $csrfTokenManager
     *
     * @return $this
     */
    public function setTokenManager(CsrfManagerInterface $csrfTokenManager)
    {
        $this->csrfManager = $csrfTokenManager;

        return $this;
    }

    protected function renderTokenField(): string
    {
        return $this->csrfManager->renderField() . "\n";

The method renderField() does not exist on null.

    }

    protected function maybeRenderTokenField(): string
    {
        return isset($this->csrfManager) ? $this->renderTokenField() : '';
    }
}