Mascame\Artificer\Controllers\ModelController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	157
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	4
Bugs	0	Features	1

9 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	11	1
B	filter()	0	22	4
A	show()	0	6	1
A	all()	0	9	1
A	edit()	0	15	1
A	field()	0	8	1
A	handleAjaxResponse()	0	12	1
A	updateOrCreate()	0	11	1
A	destroy()	0	10	3

<?php namespace Mascame\Artificer\Controllers;

use Input;
use Mascame\Artificer\Artificer;
use Mascame\Artificer\Options\AdminOption;
use Mascame\Artificer\Requests\ArtificerFormRequest;
use Redirect;
use Request;
use Response;
use URL;
use View;

class ModelController extends BaseModelController
{

    /**
     * Show the form for creating a new resource.
     *
     * @return Response
     */
    public function create()
    {
        $this->handleData($this->modelObject->schema->getInstance());

        $form = array(
            'form_action_route' => 'admin.model.store',
            'form_method' => 'post'
        );

        return View::make($this->getView('edit'))->with('items', $this->data)->with($form);
    }

    /**
     * @param $modelName
     * @return $this
     */
    public function filter($modelName)
    {
        $this->handleData($this->model->firstOrFail());

        $sort = $this->getSort();

        $data = $this->model->where(function ($query) {

            foreach (\Request::all() as $name => $value) {
                if ($value != '' && isset($this->fields[$name])) {
                    $this->fields[$name]->filter($query, $value);
                }
            }

            return null;
        })
            ->with($this->modelObject->getRelations())
            ->orderBy($sort['column'], $sort['direction'])
            ->get();

        return parent::all($modelName, $data, $sort);

It seems like you call parent on a different method (all() instead of filter()). Are you sure this is correct? If so, you might want to change this to $this->all().

    }

    /**
     * Display the specified resource.
     *
     * @param  int $id
     * @return Response
     */
    public function show($modelName, $id)

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

    {
        $this->handleData($this->model->findOrFail($id));

        return View::make($this->getView('show'))->with('item', $this->data);
    }

    /**
     * Display the specified post.
     *
     * @return Response
     */
    public function all($modelName, $data = null, $sort = null)
    {
        $sort = $this->getSort();

        $data = $this->model->with($this->modelObject->getRelations())->orderBy($sort['column'],
            $sort['direction'])->get();

        return parent::all($modelName, $data, $sort);
    }

    /**
     * Show the form for editing the specified post.
     *
     * @param  int $id
     * @return Response
     */
    public function edit($modelName, $id)

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

    {
        $this->handleData(
            $this->model->with($this->modelObject->getRelations())->findOrFail($id)
        );

        $form = array(
            'form_action_route' => 'admin.model.update',
            'form_method' => 'put'
        );

        return View::make($this->getView('edit'))
            ->with('items', $this->data)
            ->with($form);
    }

    public function field($modelName, $id, $field)

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

    {
        $this->handleData($this->model->with($this->modelObject->getRelations())->findOrFail($id));

        $this->fields[$field]->showFullField = true;

        return \HTML::field($this->fields[$field], AdminOption::get('icons'));
    }

    protected function handleAjaxResponse($item)
    {
        return Response::json(array(
                'item' => $item->toArray(),
                'refresh' => URL::route('admin.model.field.edit', array(
                    'slug' => Input::get('_standalone_origin'),
                    'id' => Input::get('_standalone_origin_id'),
                    'field' => ':fieldName:'
                ))
            )
        );
    }

    /**
     * Update or create the specified resource in storage.
     *
     * @param  int $id
     * @return Response

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

     */

    public function updateOrCreate(ArtificerFormRequest $request)
    {
        $request->persist();

        // Todo
        //if (Request::ajax()) {
        //    return $this->handleAjaxResponse($model);
        //}

        return Redirect::route('admin.model.all', array('slug' => $this->modelObject->getRouteName()));

The method route() does not seem to exist on object<redirect>.

    }

    /**
     * Remove the specified resource from storage.
     *
     * @param $modelName
     * @param $id
     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
     */
    public function destroy($modelName, $id)

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

    {
        if ($this->model->destroy($id)) {
            // Todo Notify::success('<b>Success!</b> The record has been deleted!', true);
        } else {
            // Todo Notify::danger('<b>Failed!</b> The record could not be deleted!');
        }

        return Request::ajax() ? \Response::json([]) : Redirect::back();

The method back() does not seem to exist on object<redirect>.

    }

}