crocodicstudio\crudbooster\Modules\MenuModule\AdminMenusController

Complexity

Total Complexity

20

Size/Duplication

Total Lines	145
Duplicated Lines	0 %

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	setCols()	0	5	1
A	getMenuId()	0	14	3
A	postSaveMenu()	0	18	4
A	getMenuPath()	0	9	3
A	getIndex()	0	9	1
A	setButtons()	0	12	1
A	cbInit()	0	20	2
A	hookBeforeAdd()	0	13	2
A	hookBeforeEdit()	0	12	2
A	hookAfterDelete()	0	3	1

<?php

namespace crocodicstudio\crudbooster\Modules\MenuModule;

use crocodicstudio\crudbooster\controllers\CBController;
use Illuminate\Support\Facades\Request;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Facades\PDF;

The type Illuminate\Support\Facades\PDF was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Support\Facades\Excel;

The type Illuminate\Support\Facades\Excel was not found. Maybe you did not declare it correctly or list all dependencies?

use CRUDBooster;

The type CRUDBooster was not found. Maybe you did not declare it correctly or list all dependencies?

class AdminMenusController extends CBController
{
    public function cbInit()
    {
        $this->table = "cms_menus";
        $this->primaryKey = "id";
        $this->title_field = "name";
        $this->limit = 20;
        $this->orderby = ["id" => "desc"];

        $this->setButtons();

        $id = CRUDBooster::getCurrentId();
        $row = CRUDBooster::first($this->table, $id);
        $row = (Request::segment(3) == 'edit') ? $row : null;

        $this->script_js = view('CbMenu::js', ['id' => $id, 'type' => $row->type])->render();

        $this->setCols();

        list($statisticId, $moduleId) = $this->getMenuId($row);
        $this->form = MenusForm::makeForm($statisticId, $moduleId, $row);
    }

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

        $return_url = Request::fullUrl();

        $page_title = 'Menu Management';

        return view('CbMenu::menus_management', compact('return_url', 'page_title'));
    }

    public function hookBeforeAdd(&$postData)
    {
        $postData['parent_id'] = 0;

        $postData['path'] = $this->getMenuPath($postData);

        unset($postData['module_slug']);
        unset($postData['statistic_slug']);

        if ($postData['is_dashboard'] == 1) {
            //If set dashboard, so unset for first all dashboard
            $this->table()->where('is_dashboard', 1)->update(['is_dashboard' => 0]);
            Cache::forget('sidebarDashboard'.CRUDBooster::myPrivilegeId());
        }
    }

    public function hookBeforeEdit(&$postData, $id)
    {
        if ($postData['is_dashboard'] == 1) {
            //If set dashboard, so unset for first all dashboard
            $this->table()->where('is_dashboard', 1)->update(['is_dashboard' => 0]);
            Cache::forget('sidebarDashboard'.CRUDBooster::myPrivilegeId());
        }

        $postData['path'] = $this->getMenuPath($postData);

        unset($postData['module_slug']);
        unset($postData['statistic_slug']);
    }

    public function hookAfterDelete($id)
    {
        $this->table()->where('parent_id', $id)->delete();
    }

    public function postSaveMenu()
    {
        $this->cbInit();
        $isActive = request('isActive');
        $post = json_decode(request('menus'), true);

It seems like request('menus') can also be of type array; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

        foreach ($post[0] as $i => $menu) {
            $pid = $menu['id'];
            $children = $menu['children'][0] ?: [];

            foreach ($children as $index => $child) {
                $this->findRow($child['id'])->update(['sorting' => $index + 1, 'parent_id' => $pid, 'is_active' => $isActive]);
            }

            $this->findRow($pid)->update(['sorting' => $i + 1, 'parent_id' => 0, 'is_active' => $isActive]);
        }

        return response()->json(['success' => true]);

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.

    }

    /**
     * @param $postdata
     * @return string
     */
    private function getMenuPath($postdata)
    {
        if ($postdata['type'] == 'Statistic') {
            $stat = CRUDBooster::first('cms_statistics', ['id' => $postdata['statistic_slug']])->slug;
            return 'statistic-builder/show/'.$stat;
        }

        if ($postdata['type'] == 'Module') {
            return CRUDBooster::first('cms_moduls', ['id' => $postdata['module_slug']])->path;
        }
    }

    private function setButtons()
    {
        $this->buttonTableAction = true;
        $this->button_action_style = "FALSE";
        $this->button_add = false;

The property button_add does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->deleteBtn = true;
        $this->button_edit = true;
        $this->buttonDetail = true;
        $this->button_show = false;

The property button_show does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->button_filter = true;

The property button_filter does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->buttonExport = false;
        $this->button_import = false;
    }

    /**
     * @param $row
     * @return array
     */
    private function getMenuId($row)
    {
        $idModule = $idStatistic = 0;

        if ($row->type == 'Module') {
            $idModule = DB::table('cms_moduls')->where('path', $row->path)->first()->id;
        }

        if ($row->type == 'Statistic') {
            $row->path = str_replace('statistic-builder/show/', '', $row->path);
            $idStatistic = DB::table('cms_statistics')->where('slug', $row->path)->first()->id;
        }

        return [$idStatistic, $idModule];
    }

    private function setCols()
    {
        $this->col = [];
        $this->col[] = ["label" => "Name", "name" => "name"];
        $this->col[] = ["label" => "Is Active", "name" => "is_active"];
    }
}