CasUser::__construct() - Code Metrics - Inspection of "tests: Remove now obsolete test." - ecphp/cas-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 2c4349...5c635f )

by Pol

created 2021-02-18 14:24 UTC

CasUser::__construct() A

↳ Parent: CasUser

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	1
dl	0
loc	3
ccs	2
cts	2
cp	1
crap	1
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace EcPhp\CasBundle\Security\Core\User;

final class CasUser implements CasUserInterface
{
    /**
     * The user storage.
     *
     * @var array<mixed>
     */
    private array $storage;

    /**
     * CasUser constructor.
     *
     * @param array<mixed> $data
     */
    public function __construct(array $data)
    {
        $this->storage = $data;
    }

    /**
     * {@inheritdoc}
     */
    public function eraseCredentials(): void
    {
    }

    /**
     * {@inheritdoc}
     */
    public function get(string $key, $default = null)
    {
        return $this->getStorage()[$key] ?? $default;
    }

    /**
     * {@inheritdoc}
     */
    public function getAttribute(string $key, $default = null)
    {
        return $this->getStorage()['attributes'][$key] ?? $default;
    }

    /**
     * {@inheritdoc}
     */
    public function getAttributes(): array
    {
        return $this->get('attributes', []);
    }

    /**
     * {@inheritdoc}
     */
    public function getPassword(): ?string
    {
        return null;
    }

    /**
     * {@inheritdoc}
     */
    public function getPgt(): ?string
    {
        return $this->get('proxyGrantingTicket');
    }

    /**
     * {@inheritdoc}
     */
    public function getRoles(): array
    {
        return ['ROLE_CAS_AUTHENTICATED'];
    }

    /**
     * {@inheritdoc}
     */
    public function getSalt(): ?string
    {
        return null;
    }

    /**
     * {@inheritdoc}
     */
    public function getUsername(): string
    {
        return $this->get('user');

    }

    /**
     * Get the storage.
     *
     * @return array<mixed>
     */
    private function getStorage(): array
    {
        return $this->storage;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace EcPhp\CasBundle\Security\Core\User;
6
7		final class CasUser implements CasUserInterface
8		{
9		/**
10		* The user storage.
11		*
12		* @var array<mixed>
13		*/
14		private array $storage;
15
16		/**
17		* CasUser constructor.
18		*
19		* @param array<mixed> $data
20		*/
21	9	public function __construct(array $data)
22		{
23	9	$this->storage = $data;
24	9	}
25
26		/**
27		* {@inheritdoc}
28		*/
29		public function eraseCredentials(): void
30		{
31		}
32
33		/**
34		* {@inheritdoc}
35		*/
36	3	public function get(string $key, $default = null)
37		{
38	3	return $this->getStorage()[$key] ?? $default;
39		}
40
41		/**
42		* {@inheritdoc}
43		*/
44	1	public function getAttribute(string $key, $default = null)
45		{
46	1	return $this->getStorage()['attributes'][$key] ?? $default;
47		}
48
49		/**
50		* {@inheritdoc}
51		*/
52	1	public function getAttributes(): array
53		{
54	1	return $this->get('attributes', []);
55		}
56
57		/**
58		* {@inheritdoc}
59		*/
60		public function getPassword(): ?string
61		{
62		return null;
63		}
64
65		/**
66		* {@inheritdoc}
67		*/
68	1	public function getPgt(): ?string
69		{
70	1	return $this->get('proxyGrantingTicket');
71		}
72
73		/**
74		* {@inheritdoc}
75		*/
76	1	public function getRoles(): array
77		{
78	1	return ['ROLE_CAS_AUTHENTICATED'];
79		}
80
81		/**
82		* {@inheritdoc}
83		*/
84		public function getSalt(): ?string
85		{
86		return null;
87		}
88
89		/**
90		* {@inheritdoc}
91		*/
92	1	public function getUsername(): string
93		{
94	1	return $this->get('user');
		0 ignored issues – show Bug Best Practice introduced 2020-01-30 10:21 UTC by Report Bug Copy Issue Report The expression `return $this->get('user')` could return the type `null` which is incompatible with the type-hinted return `string`. Consider adding an additional type-check to rule them out. Loading history...
95		}
96
97		/**
98		* Get the storage.
99		*
100		* @return array<mixed>
101		*/
102	4	private function getStorage(): array
103		{
104	4	return $this->storage;
105		}
106		}
107

ecphp / cas-bundle

Push — master ( 2c4349...5c635f )

CasUser::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like