Thinktomorrow\Chief\App\Http\Controllers\Back\Users\DisableUserController::store()

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function store(Request $request, $id)
    {
        $this->authorize('disable-user');

        if (chiefAdmin()->id == $id) {

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            return redirect()->back()
                ->with('messages.error', 'U kan uzelf niet blokkeren.');
        }

        $user = User::findOrFail($id);

        app(DisableUser::class)->handle($user);

        return redirect()->route('chief.back.users.index')
            ->with('messages.success', 'De gebruikersaccount is geblokkeerd met onmiddellijke ingang.');
    }