NavJobs\Transmit\Templates\ResourceController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	14.52 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	4	1
A	store()	0	5	1
A	show()	0	6	1
A	destroy()	0	5	1
A	update()	9	9	1

<?php

namespace NavJobs\Transmit\Templates;

use NavJobs\Transmit\Controller;

class ResourceController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        return $this->respondWithPaginatedCollection();
    }
    /**
     * 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()
    {
        $item = $this->model->create(request()->all());

The property model does not exist. Did you maybe forget to declare it?

        return $this->respondWithItem($item);
    }
    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        return $this->respondWithItem($this->model, function ($model) use ($id) {
            return $model->findOrFail($id);
        });
    }
    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update($id, Request $request)

This method seems to be duplicated in your project.

    {
        return $this->respondWithItem($this->model, function ($model) use ($id, $request) {
            $item = $model->findOrFail($id);
            $item->fill($request->all());
            $item->save();
            return $item;
        });
    }
    /**
     * Remove the specified resource from storage.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function destroy($id)
    {
        $this->model->findOrFail($id)->delete();
        return $this->respondWithNoContent();
    }
}