Thinktomorrow\Chief\App\Http\Controllers\Back\Users\InviteController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	expired()	0	3	1
A	accept()	0	14	2
A	deny()	0	7	1
A	__construct()	0	3	1

<?php

namespace Thinktomorrow\Chief\App\Http\Controllers\Back\Users;

use Illuminate\Http\Request;
use Thinktomorrow\Chief\Admin\Users\Invites\Application\AcceptInvite;
use Thinktomorrow\Chief\Admin\Users\Invites\Application\DenyInvite;
use Thinktomorrow\Chief\Admin\Users\Invites\Invitation;
use Thinktomorrow\Chief\App\Http\Controllers\Controller;

class InviteController extends Controller
{
    public function __construct()
    {
        $this->middleware(['chief-validate-invite'])->except('expired');
    }

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     */
    public function expired()
    {
        return view('chief::admin.users.invite-expired');
    }

    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');
    }

    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     */
    public function deny(Request $request)
    {
        $invitation = Invitation::findByToken($request->token);

        app(DenyInvite::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\DenyInvite::handle() does only seem to accept Thinktomorrow\Chief\Admin\Users\Invites\Invitation, maybe add an additional type check?

        return view('chief::admin.users.invite-denied');
    }
}