crocodicstudio\crudbooster\Modules\ModuleGenerator\AdminModulesController

Complexity

Total Complexity

24

Size/Duplication

Total Lines	224
Duplicated Lines	0 %

Importance

Changes

0

18 Methods

Rating	Name	Duplication	Size	Complexity
A	postStep4()	0	5	1
A	getStep4()	0	5	1
A	getStep3()	0	5	1
A	getStep2()	0	5	1
A	postStepFinish()	0	5	1
B	cbInit()	0	35	1
A	getStep1()	0	5	1
A	getTableColumns()	0	5	1
A	getAdd()	0	5	1
A	makeColumns()	0	8	1
A	postStep2()	0	5	1
A	postStep3()	0	5	1
A	hookBeforeDelete()	0	9	2
A	getTypeInfo()	0	4	1
A	getCheckSlug()	0	6	1
A	grantFullAccess()	0	10	1
B	postAddSave()	0	35	5
A	postEditSave()	0	22	2

<?php

namespace crocodicstudio\crudbooster\Modules\ModuleGenerator;

use crocodicstudio\crudbooster\controllers\CBController;
use crocodicstudio\crudbooster\controllers\FormValidator;
use crocodicstudio\crudbooster\helpers\CRUDBooster;
use Illuminate\Support\Facades\DB;

class AdminModulesController extends CBController
{
    public function cbInit()
    {
        $this->table = 'cms_modules';
        $this->primaryKey = 'id';
        $this->titleField = 'name';
        $this->limit = 100;
        $this->buttonAdd = false;
        $this->buttonExport = false;
        $this->buttonImport = false;
        $this->buttonFilter = false;
        $this->buttonDetail = false;
        $this->buttonBulkAction = false;
        $this->buttonActionStyle = 'button_icon';
        $this->orderby = ['is_protected' => 'asc', 'name' => 'asc'];

        $this->makeColumns();

        $this->form = Form::makeForm($this->table);

        $this->scriptJs = "
 			$(function() {
 				$('#table_name').change(function() {
					var v = $(this).val();
					$('#path').val(v);
				})	
 			}) ";

        $this->addAction[] = [
            'label' => 'Module Wizard',
            'icon' => 'fa fa-wrench',
            'url' => CRUDBooster::mainpath('step1').'/[id]',
            "showIf" => "[is_protected] == 0",
        ];

        $this->indexButton[] = ['label' => 'Generate New Module', 'icon' => 'fa fa-plus', 'url' => CRUDBooster::mainpath('step1'), 'color' => 'success'];
    }
    // public function getIndex() {
    // 	$data['page_title'] = 'Module Generator';
    // 	$data['result'] = DB::table('cms_modules')->where('is_protected',0)->orderby('name','asc')->get();
    // 	$this->cbView('CbModulesGen::index',$data);
    // }	

    private function makeColumns()
    {
        $this->col = [
            ['label' => 'name', 'name' => 'name'],
            ['label' => "Table", 'name' => "table_name"],
            ['label' => "Path", 'name' => "path"],
            ['label' => "Controller", 'name' => "controller"],
            ['label' => "Protected", 'name' => "is_protected", "visible" => false],
        ];
    }

    function hookBeforeDelete($ids)

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

    {
        foreach ($ids as $id) {
            $controller = ModulesRepo::getControllerName($id);
            DB::table('cms_menus')->where('path', 'like', '%'.$controller.'%')->delete();
            @unlink(controller_path($controller));

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

        }

        return $ids;
    }

    public function getTableColumns($table)
    {
        $columns = \Schema::getColumnListing($table);

        return response()->json($columns);

The method json() does not exist on Symfony\Component\HttpFoundation\Response. It seems like you code against a sub-type of Symfony\Component\HttpFoundation\Response such as Illuminate\Http\Response or Illuminate\Http\JsonResponse or Illuminate\Http\RedirectResponse.

    }

    public function getCheckSlug($slug)
    {
        $check = ModulesRepo::countByPath($slug);
        $lastId = DB::table('cms_modules')->max('id') + 1;

        return response()->json(['total' => $check, 'lastid' => $lastId]);
    }

    public function getAdd()
    {
        $this->cbLoader();

        return redirect()->route("ModulesControllerGetStep1");
    }

    public function getStep1($id = 0, Step1Handler $handler)
    {
        $this->cbLoader();

        return $handler->showForm($id);
    }

    public function getStep2($id, Step2Handler $handler)
    {
        $this->cbLoader();

        return $handler->showForm($id);
    }

    public function postStep2(Step1Handler $handler)
    {
        $this->cbLoader();

        return $handler->handleFormSubmit();
    }

    public function postStep3(Step2Handler $handler)
    {
        $this->cbLoader();

        return $handler->handleFormSubmit();
    }

    public function getStep3($id, Step3Handler $step3)
    {
        $this->cbLoader();

        return $step3->showForm($id);
    }

    public function getTypeInfo($type = 'text')
    {
        header("Content-Type: application/json");
        echo file_get_contents(CbComponentsPath($type).'/info.json');
    }

    public function postStep4(Step3Handler $handler)
    {
        $this->cbLoader();

        return $handler->handleFormSubmit();
    }

    public function getStep4($id, Step4Handler $handler)
    {
        $this->cbLoader();

        return $handler->showForm($id);
    }

    public function postStepFinish(Step4Handler $handler)
    {
        $this->cbLoader();

        return $handler->handleFormSubmit();
    }

    public function postAddSave()
    {
        $this->cbLoader();
        app(FormValidator::class)->validate(null, $this->form, $this);
        $this->inputAssignment();

        //Generate Controller
        $route_basename = basename(request('path'));
        if ($this->arr['controller'] == '') {
            $this->arr['controller'] = ControllerGenerator::generateController(request('table_name'), $route_basename);
        }

        $this->arr['created_at'] = YmdHis();
        $this->table()->insert($this->arr);

        //Insert Menu
        if ($this->arr['controller']) {
            (new CreateMenuForNewModule())->execute($this->arr['controller'], $this->arr['name'], $this->arr['icon']);
        }

        $moduleId = $this->arr['id'];

        $this->grantFullAccess($moduleId);

        //Refresh Session Roles
        CRUDBooster::refreshSessionRoles();

        //$ref_parameter = Request::input('ref_parameter');
        if (request('return_url')) {
            CRUDBooster::redirect(request('return_url'), cbTrans("alert_add_data_success"), 'success');
        }
        if (request('submit') == cbTrans('button_save_more')) {
            CRUDBooster::redirect(CRUDBooster::mainpath('add'), cbTrans('alert_add_data_success'), 'success');
        }
        CRUDBooster::redirect(CRUDBooster::mainpath(), cbTrans('alert_add_data_success'), 'success');
    }

    public function postEditSave($id)
    {
        $this->cbLoader();

        //$row = $this->table()->where($this->primaryKey, $id)->first();

        app(FormValidator::class)->validate($id, $this->form, $this);

        $this->inputAssignment();

        //Generate Controller
        $route_basename = basename(request('path'));
        if ($this->arr['controller'] == '') {
            $this->arr['controller'] = ControllerGenerator::generateController(request('table_name'), $route_basename);
        }

        $this->findRow($id)->update($this->arr);

        //Refresh Session Roles
        CRUDBooster::refreshSessionRoles();

        backWithMsg(cbTrans('alert_update_data_success'));
    }

    /**
     * @param $module_id
     */
    private function grantFullAccess($module_id)
    {
        DB::table('cms_privileges_roles')->insert([
            'id_cms_modules' => $module_id,
            'id_cms_privileges' => auth('cbAdmin')->user()->id_cms_privileges,

Accessing id_cms_privileges on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            'can_see_module' => 1,
            'can_create' => 1,
            'can_read' => 1,
            'can_edit' => 1,
            'can_delete' => 1,
        ]);
    }
}