Omatech\Mage\Core\Repositories\Users\UpdateUser

Complexity

Total Complexity

1

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	update()	0	23	1

<?php

namespace Omatech\Mage\Core\Repositories\Users;

use Omatech\Mage\Core\Domains\Users\Contracts\UpdateUserInterface;
use Omatech\Mage\Core\Domains\Users\Contracts\UserInterface;
use Omatech\Mage\Core\Events\Users\UserUpdated;
use Omatech\Mage\Core\Repositories\UserBaseRepository;

class UpdateUser extends UserBaseRepository implements UpdateUserInterface
{
    /**
     * @param UserInterface $user
     * @return bool
     */
    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;
    }
}