Chamilo\CoreBundle\Helpers\ScimHelper

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	saveExternalId()	0	6	1
A	__construct()	0	5	1
A	getExtraField()	0	5	1
A	createToken()	0	6	2
A	findUser()	0	15	3
A	getExternalId()	0	9	1

<?php

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

declare(strict_types=1);

namespace Chamilo\CoreBundle\Helpers;

use Chamilo\CoreBundle\Entity\ExtraField;
use Chamilo\CoreBundle\Entity\User;
use Chamilo\CoreBundle\Repository\ExtraFieldRepository;
use Chamilo\CoreBundle\Repository\ExtraFieldValuesRepository;
use Chamilo\CoreBundle\Repository\Node\UserRepository;
use Doctrine\ORM\NonUniqueResultException;
use Random\RandomException;

readonly class ScimHelper
{
    public const SCIM_FIELD = 'scim_external_id';

    public function __construct(
        private ExtraFieldRepository $extraFieldRepo,
        private ExtraFieldValuesRepository $extraFieldValuesRepo,
        private UserRepository $userRepo,
    ) {}

    public static function createToken(): string
    {
        try {
            return bin2hex(random_bytes(64));
        } catch (RandomException) {
            return '';
        }
    }

    private function getExtraField(): ExtraField
    {
        return $this->extraFieldRepo->findByVariable(

The expression return $this->extraFieldRepo->findByVariable(Chamilo\CoreBundle\Entity\ExtraField::USER_FIELD_TYPE, self::SCIM_FIELD) could return the type null which is incompatible with the type-hinted return Chamilo\CoreBundle\Entity\ExtraField. Consider adding an additional type-check to rule them out.

            ExtraField::USER_FIELD_TYPE,
            self::SCIM_FIELD
        );
    }

    public function findUser(string $externalId): ?User
    {
        $field = $this->getExtraField();

        try {
            $fieldValue = $this->extraFieldValuesRepo->findByVariableAndValue($field, $externalId);
        } catch (NonUniqueResultException) {
            return null;
        }

        if (!$fieldValue) {
            return null;
        }

        return $this->userRepo->find($fieldValue->getItemId());
    }

    public function getExternalId(User $user): ?string
    {
        $fieldValue = $this->extraFieldValuesRepo->getValueByVariableAndItem(
            self::SCIM_FIELD,
            $user->getId(),
            ExtraField::USER_FIELD_TYPE
        );

        return $fieldValue?->getFieldValue();
    }

    public function saveExternalId(string $externalId, User $user): void
    {
        $this->extraFieldValuesRepo->updateItemData(
            $this->getExtraField(),
            $user,
            $externalId
        );
    }
}