App\Http\Controllers\InviteController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Test Coverage

Coverage

42.42%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	7	1
A	upload()	0	19	2
A	store()	0	18	2
A	create()	0	4	1

<?php

namespace App\Http\Controllers;

use App\Http\Requests\InviteRequest;
use App\Invite;
use App\Notifications\InviteCompetitor;
use App\Tournament;
use App\User;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\URL;
use Illuminate\Support\Facades\View;
use Maatwebsite\Excel\Facades\Excel;

class InviteController extends Controller
{
    protected $emailBadFormat;
    protected $wrongEmail;


    /**
     * Display a listing of all invitations.
     *
     * @return View
     */
    public function index()
    {
        $invites = Auth::user()
            ->invites()
            ->with('tournament.owner')
            ->paginate(config('constants.PAGINATION'));
        return view('invitation.index', compact('invites'));

The expression return view('invitation.index', compact('invites')) returns the type Illuminate\Contracts\View\Factory|Illuminate\View\View which is incompatible with the documented return type Illuminate\Support\Facades\View.

    }

    /**
     * Display the UI for inviting competitors
     *
     * @param Tournament $tournament
     * @return View
     */
    public function create(Tournament $tournament)
    {
        // Should dd $tournament
        return view('invitation.show', compact('tournament'));

The expression return view('invitation.show', compact('tournament')) returns the type Illuminate\Contracts\View\Factory|Illuminate\View\View which is incompatible with the documented return type Illuminate\Support\Facades\View.

    }


    /**
     * Send an email to competitor and store invitation.
     *
     * @param InviteRequest|Request $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        //TODO check that recipient is list of emails
        $this->validate($request, [
            'recipients' => 'required'
        ]);

        $tournament = Tournament::where('slug', $request->tournamentSlug)->first();
        $recipients = json_decode($request->get("recipients"));
        foreach ($recipients as $recipient) {
            // Mail to Recipients
            $code = resolve(Invite::class)->generateTournamentInvite($recipient, $tournament);
            $user = new User();
            $user->email = $recipient;
            $user->notify(new InviteCompetitor($user, $tournament, $code, null));
        }
        flash()->success(trans('msg.invitation_sent'));

Are you sure the usage of flash() is correct as it seems to always return null.

        return redirect(URL::action('TournamentController@edit', $tournament->slug));

The expression return redirect(Illuminate\Support\Facades\URL::action('TournamentController@edit', $tournament->slug)) returns the type Illuminate\Http\RedirectResponse|Illuminate\Routing\Redirector which is incompatible with the documented return type Illuminate\Http\Response.

    }


    /**
     * Send an email to competitor and store invitation.
     *
     * @param InviteRequest|Request $request
     * @return \Illuminate\Http\Response
     */
    public function upload(Request $request)
    {
        $this->emailBadFormat = false;
        $file = $request->file('invites')->store('invites');
        $tournament = Tournament::where('slug', $request->tournamentSlug)->first();
        // Parse Csv File

        $reader = Excel::load("storage/app/" . $file, function ($reader) {

Are you sure $file of type false|string can be used in concatenation?

The parameter $reader is not used and could be removed.

        })->get();

        // Checking if malformed email
        resolve(Invite::class)->checkBadEmailsInExcel($reader, $tournament);
        if ($this->emailBadFormat == true) {

The condition $this->emailBadFormat == true is always false.

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            flash()->error(trans('msg.email_not_valid', ['email' => $this->wrongEmail]));
            return redirect(URL::action('InviteController@create', $tournament->slug));
        }
        Invite::sendInvites($reader, $tournament);
        flash()->success(trans('msg.invitation_sent'));

Are you sure the usage of flash() is correct as it seems to always return null.

        return redirect(URL::action('InviteController@create', $tournament->slug));

The expression return redirect(Illuminate\Support\Facades\URL::action('InviteController@create', $tournament->slug)) returns the type Illuminate\Http\RedirectResponse|Illuminate\Routing\Redirector which is incompatible with the documented return type Illuminate\Http\Response.

    }
}