Distilleries\Expendable\Http\Controllers\Backend\PermissionController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getIndex()	0	4	1
B	postEdit()	0	30	4

<?php namespace Distilleries\Expendable\Http\Controllers\Backend;

use Distilleries\Expendable\Contracts\LayoutManagerContract;
use Distilleries\Expendable\Http\Forms\Permission\PermissionForm;
use Distilleries\Expendable\Http\Controllers\Backend\Base\ModelBaseController;
use Distilleries\Expendable\States\FormStateTrait;
use Illuminate\Http\Request;
use Distilleries\Expendable\Models\Permission;
use Distilleries\FormBuilder\Contracts\FormStateContract;
use \FormBuilder;

class PermissionController extends ModelBaseController  implements FormStateContract {

Expected 1 space before implements keyword; 2 found

    use FormStateTrait;

    public function __construct(PermissionForm $form, Permission $model, LayoutManagerContract $layoutManager)
    {
        parent::__construct($model, $layoutManager);
        $this->form = $form;
    }


    // ------------------------------------------------------------------------------------------------
    // ------------------------------------------------------------------------------------------------
    // ------------------------------------------------------------------------------------------------

    public function getIndex()
    {
        return redirect()->to(action('\\' . get_class($this) . '@getEdit'));
    }

    // ------------------------------------------------------------------------------------------------

    public function postEdit(Request $request)
    {

        $form = FormBuilder::create(get_class($this->form), [
            'model' => $this->model
        ]);

        if ($form->hasError())
        {
            return $form->validateAndRedirectBack();
        }

        $permissions = $request->get('permission');
        $this->model->truncate();

The method truncate does not exist on object<Distilleries\Expendable\Models\BaseModel>? Since you implemented __call, maybe consider adding a @method annotation.

        foreach ($permissions as $role_id=>$permission) {

            foreach ($permission as $service_id) {
                $permModel = new $this->model;
                $permModel->role_id = $role_id;
                $permModel->service_id = $service_id;
                $permModel->save();
            }
        }


        return redirect()->to(action('\\' . get_class($this) . '@getIndex'));

    }
}