Session - Code Metrics - flipboxfactory/saml-idp - Measure and Improve Code Quality continuously with Scrutinizer

Session A
last analyzed 2020-12-15 16:30 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
wmc	5
lcom	0
cbo	2
dl	0
loc	67
ccs	21
cts	21
cp	1
rs	10
c	0
b	0
f	0

5 Methods

Rating	Name	Size	Complexity
A	setAuthnRequest()	8	1
A	getAuthnRequest()	6	1
A	setRelayState()	8	1
A	getRelayState()	6	1
A	remove()	11	1

<?php
/**
 * Created by PhpStorm.
 * User: dsmrt
 * Date: 1/26/18
 * Time: 3:00 PM
 */

namespace flipbox\saml\idp\services;

use SAML2\AuthnRequest;

class Session extends \flipbox\saml\core\services\Session
{

    const AUTHNREQUEST_KEY = 'authnrequest.message';
    const RELAY_STATE_KEY  = 'relaystate';

    /**
     * @param AuthnRequest $message
     * @return $this
     */
    public function setAuthnRequest(AuthnRequest $message)
    {
        \Craft::$app->getSession()->set(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            static::AUTHNREQUEST_KEY,
            $message
        );
        return $this;
    }

    /**
     * @return AuthnRequest|null
     */
    public function getAuthnRequest()
    {
        return \Craft::$app->getSession()->get(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            static::AUTHNREQUEST_KEY
        );
    }

    /**
     * @param string $relayState
     * @return $this
     */
    public function setRelayState(string $relayState)
    {
        \Craft::$app->getSession()->set(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            static::RELAY_STATE_KEY,
            $relayState
        );
        return $this;
    }

    /**
     * @return string|null
     */
    public function getRelayState()
    {
        return \Craft::$app->getSession()->get(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            static::RELAY_STATE_KEY
        );
    }

    /**
     * @return array
     */
    public function remove()
    {
        return [
            \Craft::$app->getSession()->remove(
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
                static::AUTHNREQUEST_KEY
            ),
            \Craft::$app->getSession()->remove(
                static::RELAY_STATE_KEY
            ),
        ];
    }
}


GitHub Access Token became invalid

Session A
last analyzed 2020-12-15 16:30 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

5 Methods

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1		<?php
2		/**
3		* Created by PhpStorm.
4		* User: dsmrt
5		* Date: 1/26/18
6		* Time: 3:00 PM
7		*/
8
9		namespace flipbox\saml\idp\services;
10
11		use SAML2\AuthnRequest;
12
13		class Session extends \flipbox\saml\core\services\Session
14		{
15
16		const AUTHNREQUEST_KEY = 'authnrequest.message';
17		const RELAY_STATE_KEY = 'relaystate';
18
19		/**
20		* @param AuthnRequest $message
21		* @return $this
22		*/
23	2	public function setAuthnRequest(AuthnRequest $message)
24		{
25	2	\Craft::$app->getSession()->set(
		0 ignored issues – show Bug introduced 2020-10-30 17:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
26	2	static::AUTHNREQUEST_KEY,
27	2	$message
28		);
29	2	return $this;
30		}
31
32		/**
33		* @return AuthnRequest\|null
34		*/
35	2	public function getAuthnRequest()
36		{
37	2	return \Craft::$app->getSession()->get(
		0 ignored issues – show Bug introduced 2020-10-30 17:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
38	2	static::AUTHNREQUEST_KEY
39		);
40		}
41
42		/**
43		* @param string $relayState
44		* @return $this
45		*/
46	2	public function setRelayState(string $relayState)
47		{
48	2	\Craft::$app->getSession()->set(
		0 ignored issues – show Bug introduced 2020-10-30 17:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
49	2	static::RELAY_STATE_KEY,
50	2	$relayState
51		);
52	2	return $this;
53		}
54
55		/**
56		* @return string\|null
57		*/
58	2	public function getRelayState()
59		{
60	2	return \Craft::$app->getSession()->get(
		0 ignored issues – show Bug introduced 2020-10-30 17:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
61	2	static::RELAY_STATE_KEY
62		);
63		}
64
65		/**
66		* @return array
67		*/
68	2	public function remove()
69		{
70		return [
71	2	\Craft::$app->getSession()->remove(
		0 ignored issues – show Bug introduced 2020-10-30 17:42 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
72	2	static::AUTHNREQUEST_KEY
73		),
74	2	\Craft::$app->getSession()->remove(
75	2	static::RELAY_STATE_KEY
76		),
77		];
78		}
79		}
80

flipboxfactory / saml-idp

GitHub Access Token became invalid

Session A last analyzed 2020-12-15 16:30 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

5 Methods

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

Session A
last analyzed 2020-12-15 16:30 UTC