Thinktomorrow\Chief\App\Http\Controllers\Back\Users\InviteController::accept()

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function accept(Request $request)
    {
        $invitation = Invitation::findByToken($request->token);

        app(AcceptInvite::class)->handle($invitation);

It seems like $invitation can also be of type Illuminate\Database\Eloquent\Relations\HasManyThrough and Illuminate\Database\Eloquent\Relations\HasOneThrough and null; however, parameter $invitation of Thinktomorrow\Chief\Admin\Users\Invites\Application\AcceptInvite::handle() does only seem to accept Thinktomorrow\Chief\Admin\Users\Invites\Invitation, maybe add an additional type check?

        // Log user into the system and proceed to start page
        auth()->guard('chief')->login($invitation->invitee);

The property invitee does not seem to exist on Illuminate\Database\Eloquent\Relations\HasManyThrough.

The property invitee does not seem to exist on Illuminate\Database\Eloquent\Relations\HasOneThrough.

        if (is_null($invitation->invitee->password)) {

The condition is_null($invitation->invitee->password) is always false.

            return redirect()->route('chief.back.password.edit');
        }

        return redirect()->route('chief.back.dashboard.getting-started');
    }