AzureAuthenticatorHelper::registerUser() - Code Metrics - Inspection of "Bump nanoid from 3.3.7 to 3.3.8" - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — dependabot/npm_and_yarn/nanoid... ( aaf2c9...c4aa90 )

unknown

created 2025-01-09 17:57 UTC

AzureAuthenticatorHelper::registerUser() A

↳ Parent: AzureAuthenticatorHelper

Complexity

Conditions	3
Paths	3

Size

Total Lines	66
Code Lines	45

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	45
nc	3
nop	2
dl	0
loc	66
rs	9.2
c	1
b	0
f	0

How to fix Long Method

<?php

/* For licensing terms, see /license.txt */

declare(strict_types=1);

namespace Chamilo\CoreBundle\ServiceHelper;

use Chamilo\CoreBundle\Entity\ExtraField;
use Chamilo\CoreBundle\Entity\ExtraFieldValues;
use Chamilo\CoreBundle\Entity\User;
use Chamilo\CoreBundle\Repository\ExtraFieldRepository;
use Chamilo\CoreBundle\Repository\ExtraFieldValuesRepository;
use Chamilo\CoreBundle\Repository\Node\UserRepository;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\NonUniqueResultException;
use Symfony\Component\HttpKernel\Exception\UnauthorizedHttpException;

readonly class AzureAuthenticatorHelper
{
    public const EXTRA_FIELD_ORGANISATION_EMAIL = 'organisationemail';
    public const EXTRA_FIELD_AZURE_ID = 'azure_id';
    public const EXTRA_FIELD_AZURE_UID = 'azure_uid';

    public function __construct(
        private ExtraFieldValuesRepository $extraFieldValuesRepo,
        private ExtraFieldRepository $extraFieldRepo,
        private UserRepository $userRepository,
        private EntityManagerInterface $entityManager,
        private AccessUrlHelper $urlHelper,
    ) {}

    /**
     * @throws NonUniqueResultException
     */
    public function registerUser(array $azureUserInfo, string $azureUidKey = 'objectId'): User
    {
        if (empty($azureUserInfo)) {
            throw new UnauthorizedHttpException('User info not found.');
        }

        [
            $firstNme,
            $lastName,
            $username,
            $email,
            $phone,
            $authSource,
            $active,
            $extra,
        ] = $this->formatUserData($azureUserInfo, $azureUidKey);

        $userId = $this->getUserIdByVerificationOrder($azureUserInfo, $azureUidKey);

        if (empty($userId)) {
            $user = (new User())
                ->setCreatorId($this->userRepository->getRootUser()->getId())
            ;
        } else {
            $user = $this->userRepository->find($userId);
        }

        $user
            ->setFirstname($firstNme)
            ->setLastname($lastName)
            ->setEmail($email)
            ->setUsername($username)
            ->setPlainPassword('azure')
            ->setStatus(STUDENT)
            ->setAuthSource($authSource)
            ->setPhone($phone)
            ->setActive($active)
            ->setRoleFromStatus(STUDENT)
        ;

        $this->userRepository->updateUser($user);

        $url = $this->urlHelper->getCurrent();
        $url->addUser($user);

        $this->entityManager->flush();

        $this->extraFieldValuesRepo->updateItemData(
            $this->getOrganizationEmailField(),
            $user,
            $extra['extra_'.self::EXTRA_FIELD_ORGANISATION_EMAIL]
        );

        $this->extraFieldValuesRepo->updateItemData(
            $this->getAzureIdField(),
            $user,
            $extra['extra_'.self::EXTRA_FIELD_AZURE_ID]
        );

        $this->extraFieldValuesRepo->updateItemData(
            $this->getAzureUidField(),
            $user,
            $extra['extra_'.self::EXTRA_FIELD_AZURE_UID]
        );

        return $user;
    }

    private function getOrganizationEmailField()
    {
        return $this->extraFieldRepo->findByVariable(
            ExtraField::USER_FIELD_TYPE,
            self::EXTRA_FIELD_ORGANISATION_EMAIL
        );
    }

    private function getAzureIdField()
    {
        return $this->extraFieldRepo->findByVariable(
            ExtraField::USER_FIELD_TYPE,
            self::EXTRA_FIELD_AZURE_ID
        );
    }

    private function getAzureUidField()
    {
        return $this->extraFieldRepo->findByVariable(
            ExtraField::USER_FIELD_TYPE,
            self::EXTRA_FIELD_AZURE_UID
        );
    }

    /**
     * @throws NonUniqueResultException
     */
    public function getUserIdByVerificationOrder(array $azureUserData, string $azureUidKey = 'objectId'): ?int
    {
        $selectedOrder = $this->getExistingUserVerificationOrder();

        $organisationEmailField = $this->getOrganizationEmailField();
        $azureIdField = $this->getAzureIdField();
        $azureUidField = $this->getAzureUidField();

        /** @var array<int, ExtraFieldValues> $positionsAndFields */
        $positionsAndFields = [
            1 => $this->extraFieldValuesRepo->findByVariableAndValue($organisationEmailField, $azureUserData['mail']),
            2 => $this->extraFieldValuesRepo->findByVariableAndValue($azureIdField, $azureUserData['mailNickname']),
            3 => $this->extraFieldValuesRepo->findByVariableAndValue($azureUidField, $azureUserData[$azureUidKey]),
        ];

        foreach ($selectedOrder as $position) {
            if (!empty($positionsAndFields[$position])) {
                return $positionsAndFields[$position]->getItemId();
            }
        }

        return null;
    }

    public function getExistingUserVerificationOrder(): array
    {
        return [1, 2, 3];
    }

    private function formatUserData(
        array $azureUserData,
        string $azureUidKey
    ): array {
        $phone = null;

        if (isset($azureUserData['telephoneNumber'])) {
            $phone = $azureUserData['telephoneNumber'];
        } elseif (isset($azureUserData['businessPhones'][0])) {
            $phone = $azureUserData['businessPhones'][0];
        } elseif (isset($azureUserData['mobilePhone'])) {
            $phone = $azureUserData['mobilePhone'];
        }

        // If the option is set to create users, create it
        $firstNme = $azureUserData['givenName'];
        $lastName = $azureUserData['surname'];
        $email = $azureUserData['mail'];
        $username = $azureUserData['userPrincipalName'];
        $authSource = 'azure';
        $active = ($azureUserData['accountEnabled'] ? 1 : 0);
        $extra = [
            'extra_'.self::EXTRA_FIELD_ORGANISATION_EMAIL => $azureUserData['mail'],
            'extra_'.self::EXTRA_FIELD_AZURE_ID => $azureUserData['mailNickname'],
            'extra_'.self::EXTRA_FIELD_AZURE_UID => $azureUserData[$azureUidKey],
        ];

        return [
            $firstNme,
            $lastName,
            $username,
            $email,
            $phone,
            $authSource,
            $active,
            $extra,
        ];
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			declare(strict_types=1);
6
7			namespace Chamilo\CoreBundle\ServiceHelper;
8
9			use Chamilo\CoreBundle\Entity\ExtraField;
10			use Chamilo\CoreBundle\Entity\ExtraFieldValues;
11			use Chamilo\CoreBundle\Entity\User;
12			use Chamilo\CoreBundle\Repository\ExtraFieldRepository;
13			use Chamilo\CoreBundle\Repository\ExtraFieldValuesRepository;
14			use Chamilo\CoreBundle\Repository\Node\UserRepository;
15			use Doctrine\ORM\EntityManagerInterface;
16			use Doctrine\ORM\NonUniqueResultException;
17			use Symfony\Component\HttpKernel\Exception\UnauthorizedHttpException;
18
19			readonly class AzureAuthenticatorHelper
20			{
21			public const EXTRA_FIELD_ORGANISATION_EMAIL = 'organisationemail';
22			public const EXTRA_FIELD_AZURE_ID = 'azure_id';
23			public const EXTRA_FIELD_AZURE_UID = 'azure_uid';
24
25			public function __construct(
26			private ExtraFieldValuesRepository $extraFieldValuesRepo,
27			private ExtraFieldRepository $extraFieldRepo,
28			private UserRepository $userRepository,
29			private EntityManagerInterface $entityManager,
30			private AccessUrlHelper $urlHelper,
31			) {}
32
33			/**
34			* @throws NonUniqueResultException
35			*/
36			public function registerUser(array $azureUserInfo, string $azureUidKey = 'objectId'): User
37			{
38			if (empty($azureUserInfo)) {
39			throw new UnauthorizedHttpException('User info not found.');
40			}
41
42			[
43			$firstNme,
44			$lastName,
45			$username,
46			$email,
47			$phone,
48			$authSource,
49			$active,
50			$extra,
51			] = $this->formatUserData($azureUserInfo, $azureUidKey);
52
53			$userId = $this->getUserIdByVerificationOrder($azureUserInfo, $azureUidKey);
54
55			if (empty($userId)) {
56			$user = (new User())
57			->setCreatorId($this->userRepository->getRootUser()->getId())
58			;
59			} else {
60			$user = $this->userRepository->find($userId);
61			}
62
63			$user
64			->setFirstname($firstNme)
65			->setLastname($lastName)
66			->setEmail($email)
67			->setUsername($username)
68			->setPlainPassword('azure')
69			->setStatus(STUDENT)
70			->setAuthSource($authSource)
71			->setPhone($phone)
72			->setActive($active)
73			->setRoleFromStatus(STUDENT)
74			;
75
76			$this->userRepository->updateUser($user);
77
78			$url = $this->urlHelper->getCurrent();
79			$url->addUser($user);
80
81			$this->entityManager->flush();
82
83			$this->extraFieldValuesRepo->updateItemData(
84			$this->getOrganizationEmailField(),
85			$user,
86			$extra['extra_'.self::EXTRA_FIELD_ORGANISATION_EMAIL]
87			);
88
89			$this->extraFieldValuesRepo->updateItemData(
90			$this->getAzureIdField(),
91			$user,
92			$extra['extra_'.self::EXTRA_FIELD_AZURE_ID]
93			);
94
95			$this->extraFieldValuesRepo->updateItemData(
96			$this->getAzureUidField(),
97			$user,
98			$extra['extra_'.self::EXTRA_FIELD_AZURE_UID]
99			);
100
101			return $user;
102			}
103
104			private function getOrganizationEmailField()
105			{
106			return $this->extraFieldRepo->findByVariable(
107			ExtraField::USER_FIELD_TYPE,
108			self::EXTRA_FIELD_ORGANISATION_EMAIL
109			);
110			}
111
112			private function getAzureIdField()
113			{
114			return $this->extraFieldRepo->findByVariable(
115			ExtraField::USER_FIELD_TYPE,
116			self::EXTRA_FIELD_AZURE_ID
117			);
118			}
119
120			private function getAzureUidField()
121			{
122			return $this->extraFieldRepo->findByVariable(
123			ExtraField::USER_FIELD_TYPE,
124			self::EXTRA_FIELD_AZURE_UID
125			);
126			}
127
128			/**
129			* @throws NonUniqueResultException
130			*/
131			public function getUserIdByVerificationOrder(array $azureUserData, string $azureUidKey = 'objectId'): ?int
132			{
133			$selectedOrder = $this->getExistingUserVerificationOrder();
134
135			$organisationEmailField = $this->getOrganizationEmailField();
136			$azureIdField = $this->getAzureIdField();
137			$azureUidField = $this->getAzureUidField();
138
139			/** @var array<int, ExtraFieldValues> $positionsAndFields */
140			$positionsAndFields = [
141			1 => $this->extraFieldValuesRepo->findByVariableAndValue($organisationEmailField, $azureUserData['mail']),
142			2 => $this->extraFieldValuesRepo->findByVariableAndValue($azureIdField, $azureUserData['mailNickname']),
143			3 => $this->extraFieldValuesRepo->findByVariableAndValue($azureUidField, $azureUserData[$azureUidKey]),
144			];
145
146			foreach ($selectedOrder as $position) {
147			if (!empty($positionsAndFields[$position])) {
148			return $positionsAndFields[$position]->getItemId();
149			}
150			}
151
152			return null;
153			}
154
155			public function getExistingUserVerificationOrder(): array
156			{
157			return [1, 2, 3];
158			}
159
160			private function formatUserData(
161			array $azureUserData,
162			string $azureUidKey
163			): array {
164			$phone = null;
165
166			if (isset($azureUserData['telephoneNumber'])) {
167			$phone = $azureUserData['telephoneNumber'];
168			} elseif (isset($azureUserData['businessPhones'][0])) {
169			$phone = $azureUserData['businessPhones'][0];
170			} elseif (isset($azureUserData['mobilePhone'])) {
171			$phone = $azureUserData['mobilePhone'];
172			}
173
174			// If the option is set to create users, create it
175			$firstNme = $azureUserData['givenName'];
176			$lastName = $azureUserData['surname'];
177			$email = $azureUserData['mail'];
178			$username = $azureUserData['userPrincipalName'];
179			$authSource = 'azure';
180			$active = ($azureUserData['accountEnabled'] ? 1 : 0);
181			$extra = [
182			'extra_'.self::EXTRA_FIELD_ORGANISATION_EMAIL => $azureUserData['mail'],
183			'extra_'.self::EXTRA_FIELD_AZURE_ID => $azureUserData['mailNickname'],
184			'extra_'.self::EXTRA_FIELD_AZURE_UID => $azureUserData[$azureUidKey],
185			];
186
187			return [
188			$firstNme,
189			$lastName,
190			$username,
191			$email,
192			$phone,
193			$authSource,
194			$active,
195			$extra,
196			];
197			}
198			}
199

chamilo / chamilo-lms

Push — dependabot/npm_and_yarn/nanoid... ( aaf2c9...c4aa90 )

AzureAuthenticatorHelper::registerUser() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like