EloquentJs\Controllerless\GenericController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	86
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	index()	0	4	1
A	store()	0	4	1
A	show()	0	4	1
A	update()	0	12	2
A	destroy()	0	10	2

<?php

namespace EloquentJs\Controllerless;

use EloquentJs\Model\AcceptsEloquentJsQueries;
use Illuminate\Http\Request;
use Illuminate\Routing\Controller;

class GenericController extends Controller
{
    /**
     * @var AcceptsEloquentJsQueries
     */
    protected $model;

    /**
     * Create a new GenericController instance.
     *
     * @param AcceptsEloquentJsQueries $model
     */
    public function __construct(AcceptsEloquentJsQueries $model)
    {
        $this->model = $model;
    }

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        return $this->model->eloquentJs()->get();

The method eloquentJs() does not exist on EloquentJs\Model\AcceptsEloquentJsQueries. Did you maybe mean scopeEloquentJs()?

    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return Model
     */
    public function store(Request $request)
    {
        return $this->model->create($request->all());

The method create() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)
    {
        return $this->model->eloquentJs()->find($id);

The method eloquentJs() does not exist on EloquentJs\Model\AcceptsEloquentJsQueries. Did you maybe mean scopeEloquentJs()?

    }

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

The strict comparison === seems to always evaluate to false as the types of $id (integer) and '*' (string) can never be identical. Maybe you want to use a loose comparison == instead?

            return $this->model->newQuery()->eloquentJs()->update($request->all());

The method newQuery() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

        }

        $resource = $this->model->findOrFail($id);

The method findOrFail() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

        $resource->update($request->all());

        return $resource;
    }

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

The strict comparison === seems to always evaluate to false as the types of $id (integer) and '*' (string) can never be identical. Maybe you want to use a loose comparison == instead?

            return $this->model->newQuery()->eloquentJs()->delete();

The method newQuery() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

        }

        $resource = $this->model->findOrFail($id);

The method findOrFail() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

        return ['success' => $resource->delete()];

The return type of return array('success' => $resource->delete()); (array) is incompatible with the return type documented by EloquentJs\Controllerless\GenericController::destroy of type Illuminate\Http\Response.

    }
}