flipbox\saml\core\fields\AbstractExternalIdentity

Complexity

Total Complexity

7

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	displayName()	0	4	1
A	hasContentColumn()	0	4	1
A	normalizeValue()	0	8	2
A	getInputHtml()	0	4	1
A	getStaticHtml()	0	18	2

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 */

namespace flipbox\saml\core\fields;

use craft\base\ElementInterface;
use craft\base\Field;
use craft\elements\User;
use craft\helpers\UrlHelper;
use flipbox\saml\core\records\ProviderIdentityInterface;
use yii\db\Query;

abstract class AbstractExternalIdentity extends Field implements \flipbox\saml\core\EnsureSAMLPlugin
{

    public static function displayName(): string
    {
        return 'SAML External Identity';
    }

    /**
     * @inheritdoc
     */
    public static function hasContentColumn(): bool
    {
        return false;
    }

    public function normalizeValue($value, ElementInterface $element = null)
    {
        if (! ($element instanceof User)) {
            return null;
        }
        /** @var User $element */
        return $this->getPlugin()->getProviderIdentity()->findByUser($element);
    }

    public function getInputHtml($value, ElementInterface $element = null): string
    {
        return $this->getStaticHtml($value, $element);

It seems like $element defined by parameter $element on line 41 can be null; however, flipbox\saml\core\fields\AbstractExternalIdentity::getStaticHtml() does not accept null, maybe add an additional type check?

    }

    /**
     * @param $value
     * @param ElementInterface $element
     * @return string
     * @throws \Twig_Error_Loader
     * @throws \yii\base\Exception
     */
    public function getStaticHtml($value, ElementInterface $element): string
    {
        if (! ($value instanceof Query)) {
            return '';
        }
        $handle = $this->getPlugin()->getHandle();

        return \Craft::$app->getView()->renderTemplate(
            $handle . '/_cp/fields/external-id',
            [
                'identities' => $value,
                'element' => $element,
                'baseProviderUrl' => UrlHelper::cpUrl(
                    $handle . '/metadata'
                ),
            ]
        );
    }
}