Omatech\Mage\Core\Repositories\Roles\UpdateRole

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

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

<?php

namespace Omatech\Mage\Core\Repositories\Roles;

use Omatech\Mage\Core\Domains\Roles\Contracts\RoleInterface;
use Omatech\Mage\Core\Domains\Roles\Contracts\UpdateRoleInterface;
use Omatech\Mage\Core\Events\Roles\RoleUpdated;
use Omatech\Mage\Core\Repositories\RoleBaseRepository;

class UpdateRole extends RoleBaseRepository implements UpdateRoleInterface
{
    /**
     * @param RoleInterface $role
     * @return bool
     */
    public function update(RoleInterface $role): bool
    {
        $updated = $this->query()->find($role->getId());

        $updated->fill([
                'name'       => $role->getName(),
                'guard_name' => $role->getGuardName(),
            ])->save();

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

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

$role of type object<Omatech\Mage\Core\Domains\Roles\Contracts\RoleInterface> is not a sub-type of object<Omatech\Mage\Core\Domains\Roles\Role>. It seems like you assume a concrete implementation of the interface Omatech\Mage\Core\Domains\Roles\Contracts\RoleInterface to be always present.

        event(new RoleUpdated($role, count($updated->getChanges()) >= 1));

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