App\Http\Controllers\TaskController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	114
Duplicated Lines	9.65 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes	12
Bugs	0	Features	7

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	index()	0	6	1
A	create()	0	4	1
A	store()	0	11	4
A	show()	11	11	2
A	edit()	0	4	1
A	update()	0	14	2
A	destroy()	0	4	1

<?php

namespace App\Http\Controllers;

use App\Acme\Transformers\TaskTransformer;
use App\Task;
use Illuminate\Http\Request;

use App\Http\Requests;
use Illuminate\Support\Facades\Input;
use Response;

class TaskController extends ApiController
{
    protected $taskTranformer;

    function __construct(TaskTransformer $taskTransformer)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->taskTransformer = $taskTransformer;

The property taskTransformer does not seem to exist. Did you mean taskTranformer?

        // TODO: Post test not working with auth middlerare
        //$this->middleware('auth.basic', ['only' => 'store']);

75% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $tasks = Task::all();

        return $this->respond($this->taskTransformer->transformCollection($tasks))->setStatusCode(200);

The property taskTransformer does not seem to exist. Did you mean taskTranformer?

    }

    /**
     * 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

There is no parameter named $request. Was it maybe removed?

     * @return \Illuminate\Http\Response
     */
    public function store()
    {
        if (!Input::get('name') or !Input::get('priority') or !Input::get('done'))

Using logical operators such as or instead of || is generally not recommended.

        {
            return $this->setStatusCode(422)->respondWithError('Parameters failed validation for a task');
        }

        Task::create(Input::all());

        return $this->respondCreated('Task successfully created.');
    }

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

This method seems to be duplicated in your project.

    {
        $task = Task::find($id);

        if (!$task)
        {
            return $this->respondNotFound('Task does not exist');

The return type of return $this->respondNotFound('Task does not exist'); (Illuminate\Http\JsonResponse) is incompatible with the return type documented by App\Http\Controllers\TaskController::show of type Illuminate\Http\Response.

        }

        return $this->respond($this->taskTransformer->transform($task))->setStatusCode(200);

The property taskTransformer does not seem to exist. Did you mean taskTranformer?

    }

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

        if (!$task)
        {
            return $this->respondNotFound('Task does not exist!!');
        }

        $task->name = $request->name;

The property name does not seem to exist in Illuminate\Http\Request.

        $task->priority = $request->priority;

The property priority does not seem to exist in Illuminate\Http\Request.

        $task->done = $request->done;

The property done does not seem to exist in Illuminate\Http\Request.

        $task->save();
    }

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