jeremykenedy / laravel-material-design

app/Http/Controllers/TasksController.php

<?php

namespace App\Http\Controllers;

use App\Models\Task;
use Auth;
use Illuminate\Http\Request;
use Illuminate\View\View;
use Redirect;

class TasksController extends Controller
{
    protected $rules = [
        'name'          => 'required|max:60',
        'description'   => 'max:155',
        'completed'     => 'numeric',
    ];

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('auth');
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $user = Auth::user();
        $data = [
            'tasks'           => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->get(),

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

            'tasksInComplete' => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->where('completed', '0')->get(),
            'tasksComplete'   => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->where('completed', '1')->get(),
        ];

        return view('tasks.index', $data);

The expression return view('tasks.index', $data) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index_all()
    {
        $user = Auth::user();
        $data = [
            'tasks' => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->get(),

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

        ];

        return view('tasks.filtered', $data);

The expression return view('tasks.filtered', $data) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index_incomplete()
    {
        $user = Auth::user();
        $data = [
            'tasks' => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->where('completed', '0')->get(),

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

        ];

        return view('tasks.filtered', $data);

The expression return view('tasks.filtered', $data) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index_complete()
    {
        $user = Auth::user();

        return view('tasks.filtered', [

The expression return view('tasks.filtered', array('tasks' => App\Models\Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->where('completed', '1')->get())) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

            'tasks' => Task::orderBy('created_at', 'asc')->where('user_id', $user->id)->where('completed', '1')->get(),

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

        ]);
    }

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

The expression return view('tasks.create') returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Store a newly created resource in storage.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\Response
     */
    public function store(Request $request)
    {
        $this->validate($request, $this->rules);

        $user = Auth::user();
        $task = $request->all();
        $task['user_id'] = $user->id;

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

        Task::create($task);

        return redirect('/tasks')->with('status', 'Task created');

The expression return redirect('/tasks')->with('status', 'Task created') returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * 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)
    {
        $task = Task::findOrFail($id);

        return view('tasks.edit')->withTask($task);

The expression return view('tasks.edit')->withTask($task) returns the type Illuminate\View\View which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Update the specified resource in storage.
     *
     * @param \Illuminate\Http\Request $request
     * @param int                      $id
     *
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)
    {
        $this->validate($request, $this->rules);

        $task = Task::findOrFail($id);
        $task->name = $request->input('name');
        $task->description = $request->input('description');
        $task->completed = $request->input('completed');

        if ($task->completed == '1') {
            $return_msg = 'Task Completed !!!';
        } else {
            $task->completed = 0;
            $return_msg = 'Task Updated';
        }

        $task->save();

        return Redirect::back()->with('status', $return_msg);

The expression return Redirect::back()->with('status', $return_msg) 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)
    {
        Task::findOrFail($id)->delete();

        return redirect('/tasks')->with('status', 'Task Deleted');

The expression return redirect('/tasks')->with('status', 'Task Deleted') returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

    }

    /**
     * Return current users tasks using View::Composer.
     *
     * @param  \App\Providers\ComposerServiceProvider.php
     * @param obj $view

The type App\Http\Controllers\obj was not found. Maybe you did not declare it correctly or list all dependencies?

     *
     * @return \Illuminate\View\View
     */
    public function getAllTasks(View $view)
    {
        $user = Auth::user();
        $queryTasks = Task::orderBy('created_at', 'asc');
        $Alltasks = $queryTasks->where('user_id', $user->id)->get();

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

        $view->with('allTasks', $Alltasks);
    }

    /**
     * Return current users INCOMPLETE tasks using View::Composer.
     *
     * @param  \App\Providers\ComposerServiceProvider.php
     * @param obj $view
     *
     * @return \Illuminate\View\View
     */
    public function getIncompleteTasks(View $view)
    {
        $user = Auth::user();
        $queryTasks = Task::orderBy('created_at', 'asc');
        $tasksInComplete = $queryTasks->where([
            ['user_id', '=', $user->id],

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

            ['completed', '=', '0'],
        ])->get();

        $view->with('incompleteTasks', $tasksInComplete);
    }

    /**
     * Return current users COMPLETE tasks using View::Composer.
     *
     * @param  \App\Providers\ComposerServiceProvider.php
     * @param obj $view
     *
     * @return \Illuminate\View\View
     */
    public function getCompleteTasks(View $view)
    {
        $user = Auth::user();
        $queryTasks = Task::orderBy('created_at', 'asc');
        $tasksCompleted = $queryTasks->where([
            ['user_id', '=', $user->id],

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

            ['completed', '=', '1'],
        ])->get();
        $view->with('completeTasks', $tasksCompleted);
    }
}