Modules\User\Services\UserService

Complexity

Total Complexity

10

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Test Coverage

Coverage

76.92%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	all()	0	3	1
A	setRoles()	0	6	2
A	newUser()	0	5	1
A	delete()	0	3	1
A	update()	0	5	1
A	find()	0	7	2
A	create()	0	6	1
A	findByIdentityId()	0	3	1

<?php
/**
 * Created by PhpStorm.
 * User: arthur
 * Date: 04.10.18
 * Time: 16:17.
 */

namespace Modules\User\Services;

use Modules\Authorization\Entities\Role;
use Modules\User\Contracts\UserServiceContract;
use Modules\User\Entities\User;

class UserService implements UserServiceContract
{
    public function all()
    {
        return User::all();
    }

    public function find($id): ?User
    {
        if ($id instanceof User) {
            return $id;
        }

        return User::find($id);

The expression return Modules\User\Entities\User::find($id) could return the type Illuminate\Database\Eloquent\Collection which is incompatible with the type-hinted return Modules\User\Entities\User|null. Consider adding an additional type-check to rule them out.

    }

    public function findByIdentityId($id): ?User
    {
        return User::cache()->findBy('identity_id', $id);

The expression return Modules\User\Entities\User::cache()->findBy('identity_id', $id) could return the type Eloquent which includes types incompatible with the type-hinted return Modules\User\Entities\User|null. Consider adding an additional type-check to rule them out.

    }

    public function update($id, $data): ?User
    {
        $user = $this->find($id);
        $user->update($data);
        return $user;
    }

    public function create($data): User
    {
        $user = User::create($data);
        $user->assignRole(Role::USER);

        return $user;
    }

    public function delete($id): bool
    {
        return User::destroy($id);

The expression return Modules\User\Entities\User::destroy($id) returns the type integer which is incompatible with the type-hinted return boolean.

    }

    public function newUser($data): User
    {
        $user = new User($data);
        $user->assignRole(Role::USER);
        return $user;
    }

    public function setRoles($id, array $roles): void
    {
        if (!in_array(Role::USER, $roles)) {
            $roles[] = Role::USER;
        }
        $this->find($id)->syncRoles($roles);
    }
}