crocodicstudio\crudbooster\controllers\CBController::postAddSave()

Complexity

Conditions	1
Paths	1

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	CBController::getEdit()	0	9	1
A	CBController::table()	0	4	2

<?php

namespace crocodicstudio\crudbooster\controllers;

error_reporting(E_ALL ^ E_NOTICE);

use crocodicstudio\crudbooster\CBCoreModule\Hooks;
use crocodicstudio\crudbooster\CBCoreModule\Index;
use crocodicstudio\crudbooster\controllers\CBController\CbFormLoader;
use crocodicstudio\crudbooster\controllers\CBController\CbIndexLoader;
use crocodicstudio\crudbooster\controllers\CBController\CbLayoutLoader;
use crocodicstudio\crudbooster\controllers\CBController\Deleter;
use crocodicstudio\crudbooster\controllers\CBController\ExportData;
use crocodicstudio\crudbooster\controllers\CBController\FormSubmitHandlers;
use crocodicstudio\crudbooster\controllers\CBController\ImportData;
use crocodicstudio\crudbooster\controllers\CBController\IndexAjax;
use Illuminate\Support\Facades\Request;
use Illuminate\Support\Facades\DB;
use CRUDBooster;

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

use CB;
use Schema;

class CBController extends Controller
{
    use Hooks;
    use Deleter, CbFormLoader, CbIndexLoader, CbLayoutLoader, IndexAjax, ImportData, ExportData, FormSubmitHandlers;

The trait crocodicstudio\crudbooster\controllers\CBController\ImportData requires the property $name which is not provided by crocodicstudio\crudbooster\controllers\CBController.

    public $data_inputan;

    public $module_name;

    public $table;

    public $title_field;

    public $primary_key = 'id';

    public $arr = [];

    public $col = [];

    public $form = [];

    public $data = [];

    public $addaction = [];

    //public $global_privilege = false;

    public $button_delete = true;

    public $button_action_style = 'button_icon';

    public $return_url = null;

    public $parent_field = null;

    public $parent_id = null;

    public $hide_form = [];

    public $index_return = false; //for export

    public function cbView($template, $data)
    {
        $this->cbLayoutLoader();
        view()->share($this->data);
        return view($template, $data);
    }

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

The method cbInit() does not exist on crocodicstudio\crudbooster\controllers\CBController. Since you implemented __call, consider adding a @method annotation.

        $this->cbLayoutLoader();
        $this->cbFormLoader();
        $this->cbIndexLoader();
        $this->checkHideForm();
        $this->genericLoader();

        view()->share($this->data);
    }

    private function checkHideForm()
    {
        if (! count($this->hide_form)) {
            return null;
        }
        foreach ($this->form as $i => $f) {
            if (in_array($f['name'], $this->hide_form)) {
                unset($this->form[$i]);
            }
        }
    }

    public function getIndex()
    {
        $index = app(Index::class);
        $this->cbIndexLoader();
        $this->genericLoader();
        $data = $index->index($this);

        return $this->cbView('crudbooster::default.index', $data);
    }

    public function getUpdateSingle()
    {
        $table = request('table');
        $column = request('column');
        $value = request('value');
        $id = request('id');
        DB::table($table)->where(CB::pk($table), $id)->update([$column => $value]);

It seems like $table can also be of type array; however, parameter $table of Illuminate\Database\Connection::table() does only seem to accept string, maybe add an additional type check?

        backWithMsg(cbTrans('alert_update_data_success'));
    }

    public function getAdd()
    {
        $page_title = cbTrans('add_data_page_title', ['module' => CB::getCurrentModule()->name]);
        $command = 'add';
        return $this->cbForm(compact('page_title', 'command'));
    }


    /**
     * @param null $tableName

Are you sure the doc-type for parameter $tableName is correct as it would always require null to be passed?

     * @return mixed
     */
    public function table($tableName = null)
    {
        $table = $tableName ?: $this->table;
        return \DB::table($table);
    }

    public function getEdit($id)
    {
        $row = $this->findRow($id)->first();

        $page_title = cbTrans("edit_data_page_title", ['module' => CB::getCurrentModule()->name, 'name' => $row->{$this->title_field}]);
        $command = 'edit';
        session()->put('current_row_id', $id);

        return $this->cbForm(compact('id', 'row', 'page_title', 'command'));
    }

    /**
     * @param $id
     * @return mixed
     */
    public function findRow($id)
    {
        return $this->table()->where($this->primary_key, $id);
    }

    public function getDetail($id)
    {
        $row = $this->findRow($id)->first();


        $page_title = cbTrans('detail_data_page_title', ['module' => CB::getCurrentModule()->name, 'name' => $row->{$this->title_field}]);
        $command = 'detail';

        session()->put('current_row_id', $id);

        return $this->cbForm(compact('row', 'page_title', 'command', 'id'));
    }

    public function actionButtonSelected($id_selected, $button_name)
    {
    }

    private function insertLog($msg, $name)
    {
        CB::insertLog(trans('logging.'.$msg, ['module' => CB::getCurrentModule()->name, 'name' => $name]));
    }

    /**
     * @param $data
     * @return mixed
     */
    private function cbForm($data)
    {
        $this->genericLoader();
        $this->cbFormLoader();
        return $this->cbView('crudbooster::default.form', $data);
    }

    private function genericLoader()
    {
        $this->primary_key = CB::pk($this->table);
        $this->columns_table = $this->col;
        $this->data_inputan = $this->form;
        $this->data['pk'] = $this->primary_key;
        $this->data['forms'] = $this->data_inputan;
        $this->data['hide_form'] = $this->hide_form;
        $this->data['table'] = $this->table;
        $this->data['title_field'] = $this->title_field;
        $this->data['appname'] = cbGetsetting('appname');
        $this->data['index_button'] = $this->index_button;
        $this->data['button_delete'] = $this->button_delete;
        $this->data['sub_module'] = $this->sub_module;
        $this->data['parent_field'] = (request('parent_field')) ?: $this->parent_field;
        $this->data['parent_id'] = (request('parent_id')) ?: $this->parent_id;

        if (CB::getCurrentMethod() == 'getProfile') {
            session()->put('current_row_id', CB::myId());
            $this->data['return_url'] = Request::fullUrl();
        }
    }
}