AbstractExternalIdentity::getStaticHtml() - Code Metrics - flipboxfactory/saml-core - Measure and Improve Code Quality continuously with Scrutinizer

AbstractExternalIdentity::getStaticHtml() A
last analyzed 2021-04-19 18:56 UTC

↳ Parent: AbstractExternalIdentity

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.6666
c	0
b	0
f	0
nc	2
cc	2
nop	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);
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
    }

    /**
     * @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'
                ),
            ]
        );
    }
}


1			<?php
2
3			/**
4			* @copyright Copyright (c) Flipbox Digital Limited
5			*/
6
7			namespace flipbox\saml\core\fields;
8
9			use craft\base\ElementInterface;
10			use craft\base\Field;
11			use craft\elements\User;
12			use craft\helpers\UrlHelper;
13			use flipbox\saml\core\records\ProviderIdentityInterface;
14			use yii\db\Query;
15
16			abstract class AbstractExternalIdentity extends Field implements \flipbox\saml\core\EnsureSAMLPlugin
17			{
18
19			public static function displayName(): string
20			{
21			return 'SAML External Identity';
22			}
23
24			/**
25			* @inheritdoc
26			*/
27			public static function hasContentColumn(): bool
28			{
29			return false;
30			}
31
32			public function normalizeValue($value, ElementInterface $element = null)
33			{
34			if (! ($element instanceof User)) {
35			return null;
36			}
37			/** @var User $element */
38			return $this->getPlugin()->getProviderIdentity()->findByUser($element);
39			}
40
41			public function getInputHtml($value, ElementInterface $element = null): string
42			{
43			return $this->getStaticHtml($value, $element);
			0 ignored issues – show Bug introduced 2018-08-17 15:20 UTC by Report Bug Copy Issue Report It seems like `$element` defined by parameter `$element` on line `41` can be `null`; however, `flipbox\saml\core\fields...entity::getStaticHtml()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
44			}
45
46			/**
47			* @param $value
48			* @param ElementInterface $element
49			* @return string
50			* @throws \Twig_Error_Loader
51			* @throws \yii\base\Exception
52			*/
53			public function getStaticHtml($value, ElementInterface $element): string
54			{
55			if (! ($value instanceof Query)) {
56			return '';
57			}
58			$handle = $this->getPlugin()->getHandle();
59
60			return \Craft::$app->getView()->renderTemplate(
61			$handle . '/_cp/fields/external-id',
62			[
63			'identities' => $value,
64			'element' => $element,
65			'baseProviderUrl' => UrlHelper::cpUrl(
66			$handle . '/metadata'
67			),
68			]
69			);
70			}
71			}
72

flipboxfactory / saml-core

AbstractExternalIdentity::getStaticHtml() A last analyzed 2021-04-19 18:56 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

AbstractExternalIdentity::getStaticHtml() A
last analyzed 2021-04-19 18:56 UTC