Passed
Pull Request — master (#6127)
by Angel Fernando Quiroz
08:01
created

MigrationMoodleEventSubscriber::getExtraField()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 8
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 5
c 1
b 0
f 0
nc 1
nop 0
dl 0
loc 8
rs 10
1
<?php
2
3
/* For licensing terms, see /license.txt */
4
5
declare(strict_types=1);
6
7
use Chamilo\CoreBundle\Entity\ExtraField;
0 ignored issues
show
Bug introduced by
This use statement conflicts with another class in this namespace, ExtraField. Consider defining an alias.

Let?s assume that you have a directory layout like this:

.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

and let?s assume the following content of Bar.php:

// Bar.php
namespace OtherDir;

use SomeDir\Foo; // This now conflicts the class OtherDir\Foo

If both files OtherDir/Foo.php and SomeDir/Foo.php are loaded in the same runtime, you will see a PHP error such as the following:

PHP Fatal error:  Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php

However, as OtherDir/Foo.php does not necessarily have to be loaded and the error is only triggered if it is loaded before OtherDir/Bar.php, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias:

// Bar.php
namespace OtherDir;

use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore.
Loading history...
8
use Chamilo\CoreBundle\Entity\ExtraFieldValues;
9
use Chamilo\CoreBundle\Event\CheckLoginCredentialsEvent;
10
use Chamilo\CoreBundle\Event\Events;
11
use Doctrine\ORM\Exception\NotSupported;
12
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
13
use Symfony\Component\Security\Core\Exception\AccessDeniedException;
14
15
class MigrationMoodleEventSubscriber implements EventSubscriberInterface
16
{
17
    /**
18
     * @inheritDoc
19
     */
20
    public static function getSubscribedEvents(): array
21
    {
22
        return [
23
            Events::CHECK_LOGIN_CREDENTIALS => 'onCheckLoginCredentials',
24
        ];
25
    }
26
27
    /**
28
     * @throws NotSupported
29
     */
30
    public function onCheckLoginCredentials(CheckLoginCredentialsEvent $event): void
31
    {
32
        $userData = $event->getUser();
33
        $credentials = $event->getCredentials();
34
35
        $extraField = $this->getExtraField();
36
37
        if (empty($extraField)) {
38
            return;
39
        }
40
41
        $fieldValue = $this->getExtraFieldValue($extraField, $userData);
42
43
        if (empty($fieldValue)) {
44
            return;
45
        }
46
47
        $isPasswordVerified = password_verify(
48
            $credentials['password'],
49
            $fieldValue->getFieldValue()
50
        );
51
52
        if (!$isPasswordVerified) {
53
            throw new AccessDeniedException();
54
        }
55
    }
56
57
    /**
58
     * @throws NotSupported
59
     */
60
    private function getExtraField(): ?ExtraField
61
    {
62
        return Database::getManager()
63
            ->getRepository(ExtraField::class)
64
            ->findOneBy(
65
                [
66
                    'variable' => 'moodle_password',
67
                    'extraFieldType' => ExtraField::USER_FIELD_TYPE,
68
                ]
69
            );
70
    }
71
72
    /**
73
     * @throws NotSupported
74
     */
75
    private function getExtraFieldValue(ExtraField $extraField, array $userData): ?ExtraFieldValues
76
    {
77
        return Database::getManager()
78
            ->getRepository(ExtraFieldValues::class)
79
            ->findOneBy(['field' => $extraField, 'itemId' => $userData['id']]);
80
    }
81
}
82