Omatech\Mage\Core\Repositories\Users\UpdateUser::update()

Complexity

Conditions	1
Paths	1

Size

Total Lines

23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	16
CRAP Score	1

Importance

Changes

0

<?php
    public function update(UserInterface $user): bool
    {
        $updated = $this->query()->find($user->getId());

        $updated->fill([
                'name'              => $user->getName(),
                'language'          => $user->getLanguage(),
                'email'             => $user->getEmail(),
                'email_verified_at' => $user->getEmailVerifiedAt(),
                'password'          => $user->getPassword(),
                'remember_token'    => $user->getRememberToken(),
            ])->save();

        $user->setCreatedAt($updated->created_at);
        $user->setUpdatedAt($updated->updated_at);

        $this->syncPermissions($updated, $user);

$user of type object<Omatech\Mage\Core\Domains\Users\Contracts\UserInterface> is not a sub-type of object<Omatech\Mage\Core\Domains\Users\User>. It seems like you assume a concrete implementation of the interface Omatech\Mage\Core\Domains\Users\Contracts\UserInterface to be always present.

        $this->syncRoles($updated, $user);

$user of type object<Omatech\Mage\Core\Domains\Users\Contracts\UserInterface> is not a sub-type of object<Omatech\Mage\Core\Domains\Users\User>. It seems like you assume a concrete implementation of the interface Omatech\Mage\Core\Domains\Users\Contracts\UserInterface to be always present.

        event(new UserUpdated($user, count($updated->getChanges()) >= 1));

        return count($updated->getChanges()) >= 1;
    }
}