Issues (3627)

UserBundle/DataFixtures/ORM/LoadUserData.php (1 issue)

1
<?php
2
3
/*
4
 * @copyright   2014 Mautic Contributors. All rights reserved
5
 * @author      Mautic
6
 *
7
 * @link        http://mautic.org
8
 *
9
 * @license     GNU/GPLv3 http://www.gnu.org/licenses/gpl-3.0.html
10
 */
11
12
namespace Mautic\UserBundle\DataFixtures\ORM;
13
14
use Doctrine\Bundle\FixturesBundle\FixtureGroupInterface;
15
use Doctrine\Common\DataFixtures\AbstractFixture;
16
use Doctrine\Common\DataFixtures\OrderedFixtureInterface;
17
use Doctrine\Common\Persistence\ObjectManager;
18
use Mautic\UserBundle\Entity\User;
19
use Symfony\Component\Security\Core\Encoder\EncoderFactoryInterface;
20
21
class LoadUserData extends AbstractFixture implements OrderedFixtureInterface, FixtureGroupInterface
22
{
23
    /**
24
     * {@inheritdoc}
25
     */
26
    public static function getGroups(): array
27
    {
28
        return ['group_mautic_install_data'];
29
    }
30
31
    /**
32
     * @var EncoderFactoryInterface
33
     */
34
    private $encoder;
35
36
    /**
37
     * {@inheritdoc}
38
     */
39
    public function __construct(EncoderFactoryInterface $encoder)
40
    {
41
        $this->encoder = $encoder;
42
    }
43
44
    public function load(ObjectManager $manager)
45
    {
46
        $user = new User();
47
        $user->setFirstName('Admin');
48
        $user->setLastName('User');
49
        $user->setUsername('admin');
50
        $user->setEmail('[email protected]');
51
        $encoder = $this->encoder->getEncoder($user);
52
        $user->setPassword($encoder->encodePassword('mautic', $user->getSalt()));
0 ignored issues
show
Are you sure the usage of $user->getSalt() targeting Mautic\UserBundle\Entity\User::getSalt() seems to always return null.

This check looks for function or method calls that always return null and whose return value is used.

class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

The method getObject() can return nothing but null, so it makes no sense to use the return value.

The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes.

Loading history...
53
        $user->setRole($this->getReference('admin-role'));
54
        $manager->persist($user);
55
        $manager->flush();
56
57
        $this->addReference('admin-user', $user);
58
59
        $user = new User();
60
        $user->setFirstName('Sales');
61
        $user->setLastName('User');
62
        $user->setUsername('sales');
63
        $user->setEmail('[email protected]');
64
        $encoder = $this->encoder->getEncoder($user);
65
        $user->setPassword($encoder->encodePassword('mautic', $user->getSalt()));
66
        $user->setRole($this->getReference('sales-role'));
67
        $manager->persist($user);
68
        $manager->flush();
69
70
        $this->addReference('sales-user', $user);
71
    }
72
73
    /**
74
     * {@inheritdoc}
75
     */
76
    public function getOrder()
77
    {
78
        return 2;
79
    }
80
}
81