App\Http\Controllers\enso\core\Administration\UserGroup\Update

Complexity

Total Complexity

1

Size/Duplication

Total Lines	10
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	8	1

<?php

namespace App\Http\Controllers\enso\core\Administration\UserGroup;

use Illuminate\Routing\Controller;
use LaravelEnso\Core\Http\Requests\ValidateUserGroupRequest;
use LaravelEnso\Core\Models\UserGroup;

class Update extends Controller
{
    public function __invoke(ValidateUserGroupRequest $request, UserGroup $userGroup)
    {
        $userGroup->updateWithRoles(
            $request->validatedExcept('roles'),
            $request->get('roles')

It seems like $request->get('roles') can also be of type null; however, parameter $roles of LaravelEnso\Core\Models\UserGroup::updateWithRoles() does only seem to accept array, maybe add an additional type check?

        );

        return ['message' => __('The user group was successfully updated')];
    }
}