App\Http\Controllers\ApiTaskController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	3	1
A	show()	0	4	1
A	destroy()	0	7	1
A	store()	0	11	1
A	update()	0	10	1

<?php

namespace App\Http\Controllers;

use App\Http\Requests\DestroyTask;
use App\Http\Requests\ListTask;
use App\Http\Requests\ShowTask;
use App\Http\Requests\StoreTask;
use App\Http\Requests\UpdateTask;
use App\Task;
use Illuminate\Http\Request;

/**
 * Class ApiTaskController.
 */
class ApiTaskController extends Controller
{
    /**
     * @param ListTask $request
     *
     * @return \Illuminate\Database\Eloquent\Collection|static[]
     */
    public function index(ListTask $request)

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

    {
        return Task::all();
    }

    /**
     * @param Task $task
     *
     * @return Task
     */
    public function show(ShowTask $request, Task $task)

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

    {
        //return $this->Object->show($task); això era per retornar el nom, es fara més avant.
        return $task;
    }

    //La funció del request s'anomena injeccio de dependències.

    /**
     * @param StoreTask $request a mida, quines normes de validació a de complir aquella request.
     *
     * @return mixed
     */
    public function store(StoreTask $request)
    {
        $request->validate([

The call to Illuminate\Foundation\Http\FormRequest::validate() has too many arguments starting with array('name' => 'required', 'user_id' => 'required', 'description' => 'required').

            'name'        => 'required',
            'user_id'     => 'required',
            'description' => 'required',
        ]);

        $task = Task::create($request->only(['name', 'user_id', 'description']));

        return $task;
    }

    /**
     * @param DestroyTask $request
     * @param Task        $task
     *
     * @return Task
     */
    public function destroy(DestroyTask $request, Task $task)
    {
        //el Task $task és equivalent a $task = Task::findOrFail($id)

        $task->delete();

        return $task;
    }

    /**
     * @param UpdateTask $request
     * @param Task       $task
     *
     * @return Task
     */
    public function update(UpdateTask $request, Task $task)
    {
        $request->validate([

The call to Illuminate\Foundation\Http\FormRequest::validate() has too many arguments starting with array('name' => 'required').

            'name' => 'required',
        ]);

        $task->name = $request->name;
        $task->save();

        return $task;
    }
}