Omatech\Mage\Core\Repositories\Roles\UpdateRole::update()

Complexity

Conditions	1
Paths	1

Size

Total Lines

18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	1

Importance

Changes

0

<?php
    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;
    }
}