samtny / league-frontend

app/Http/Controllers/ResultSubmissionsController.php

<?php

namespace App\Http\Controllers;

use App\Association;
use App\Result;
use App\TeamResult;
use App\ResultSubmission;
use Illuminate\Http\Request;

class ResultSubmissionsController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index(Association $association)
    {
        return view('result_submissions.approve', ['association' => $association]);

The expression return view('result_submissions.approve', array('association' => $association)) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Show the form for creating a new resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function create()
    {
        //
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)

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

    {
        //
    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)

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

    {
        //
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function edit($id)

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

    {
        //
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)
    {
        $submission = ResultSubmission::find($id);

        if ($request->delete == 'delete') {
            $submission->delete();
        }
        else {
            $result = Result::where('match_id', $submission->match_id)->first();

            if (empty($result)) {
                $result = new Result();
                $result->match_id = $submission->match_id;
            }

            $result->home_team_id = $submission->match->home_team_id;

The property home_team_id does not seem to exist on App\PLMatch. Are you sure there is no database migration missing?

            $result->away_team_id = $submission->match->away_team_id;

The property away_team_id does not seem to exist on App\PLMatch. Are you sure there is no database migration missing?

            $result->home_team_score = $submission->home_team_score;
            $result->away_team_score = $submission->away_team_score;

            $result->save();

            // home team result:
            $team_result = TeamResult::where('schedule_id', $submission->match->schedule_id)

The property schedule_id does not exist on App\PLMatch. Did you mean schedule?

                ->where('match_id', $submission->match_id)
                ->where('team_id', $result->home_team_id)
                ->first();

            if (empty($team_result)) {
                $team_result = new TeamResult();
                $team_result->schedule_id = $submission->match->schedule_id;
                $team_result->match_id = $submission->match->id;
                $team_result->team_id = $result->home_team_id;
            }

            $team_result->points = $result->home_team_score;
            $team_result->win = $result->home_team_id == $submission->win_team_id ? 1 : 0;
            $team_result->loss = $result->home_team_id != $submission->win_team_id ? 1 : 0;
            $team_result->tie = 0;
            $team_result->save();

            // away team result;
            $team_result = TeamResult::where('schedule_id', $submission->match->schedule_id)
                ->where('match_id', $submission->match_id)
                ->where('team_id', $result->away_team_id)
                ->first();

            if (empty($team_result)) {
                $team_result = new TeamResult();
                $team_result->schedule_id = $submission->match->schedule_id;
                $team_result->match_id = $submission->match->id;
                $team_result->team_id = $result->away_team_id;
            }

            $team_result->points = $result->away_team_score;
            $team_result->win = $result->away_team_id == $submission->win_team_id ? 1 : 0;
            $team_result->loss = $result->away_team_id != $submission->win_team_id ? 1 : 0;
            $team_result->tie = 0;
            $team_result->save();

            $submission->approved = TRUE;
            $submission->save();
        }

        $url = $request->url;

        if (!empty($url)) {
            return redirect($url)->with('success', 'Data saved successfully!');

The expression return redirect($url)->with('success', 'Data saved successfully!') also could return the type Illuminate\Http\RedirectResponse|Illuminate\Routing\Redirector which is incompatible with the documented return type Illuminate\Http\Response.

        }

        return redirect()->route('user', ['id' => \Auth::user()->id]);

The expression return redirect()->route('user', array('id' => Auth::user()->id)) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)

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

    {
        //
    }
}