crocodicstudio\crudbooster\controllers\CBController\FormSubmitHandlers

Complexity

Total Complexity

21

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	sendResponseForSave()	0	13	5
A	postAddSave()	0	17	1
A	postEditSave()	0	17	1
A	setTimeStamps()	0	4	2
C	inputAssignment()	0	31	12

<?php

namespace crocodicstudio\crudbooster\controllers\CBController;

use crocodicstudio\crudbooster\CBCoreModule\RelationHandler;
use crocodicstudio\crudbooster\controllers\FormValidator;
use crocodicstudio\crudbooster\helpers\CRUDBooster;
use crocodicstudio\crudbooster\helpers\DbInspector;
use Illuminate\Support\Facades\Request;
use Illuminate\Support\Facades\Schema;

trait FormSubmitHandlers
{
    public $arr = [];

    public $return_url = null;

    public function inputAssignment($id = null)

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

    {
        $hide_form = (request('hide_form')) ? unserialize(request('hide_form')) : [];

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

        foreach ($this->form as $form) {
            $name = $form['name'];
            $type = $form['type'] ?: 'text';
            $inputdata = request($name);

            if (! $name || in_array($name, $hide_form) || $form['exception']) {
                continue;
            }

            $hookPath = CbComponentsPath($type).DIRECTORY_SEPARATOR.'hookInputAssignment.php';
            if (file_exists($hookPath)) {
                require_once($hookPath);
            }
            unset($hookPath);

            if (Request::hasFile($name)) {
                continue;
            }

            if ($inputdata == '' && DbInspector::isNullableColumn($this->table, $name)) {
                continue;
            }

            $this->arr[$name] = '';

            if ($inputdata != '') {
                $this->arr[$name] = $inputdata;
            }
        }
    }

    public function postAddSave()
    {
        $this->genericLoader();

It seems like genericLoader() must be provided by classes using this trait. How about adding it as abstract method to this trait?

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

        $this->setTimeStamps('created_at');

        $this->arr = $this->hookBeforeAdd($this->arr);

It seems like hookBeforeAdd() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $id = $this->table()->insertGetId($this->arr);

It seems like table() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        app(RelationHandler::class)->save($this->table, $id, $this->data_inputan);
        $this->hookAfterAdd($id);

It seems like hookAfterAdd() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->insertLog('log_add', $id. ' on ' . $this->table);

It seems like insertLog() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->sendResponseForSave('alert_add_data_success');
    }

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

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

        $this->setTimeStamps('updated_at');

        $this->arr = $this->hookBeforeEdit($this->arr, $id);

It seems like hookBeforeEdit() must be provided by classes using this trait. How about adding it as abstract method to this trait?

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

It seems like findRow() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        app(RelationHandler::class)->save($this->table, $id, $this->data_inputan);
        $this->hookAfterEdit($id);

It seems like hookAfterEdit() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->insertLog('log_update', $id. ' on ' . $this->table);

        $this->sendResponseForSave('alert_update_data_success');
    }

    private function sendResponseForSave($msg)
    {
        $this->return_url = $this->return_url ?: request('return_url');
        if ($this->return_url) {
            if (request('submit') == cbTrans('button_save_more')) {
                backWithMsg(cbTrans($msg), 'success');
            }
            CRUDBooster::redirect($this->return_url, cbTrans($msg), 'success');
        }
        if (request('submit') == cbTrans('button_save_more')) {
            CRUDBooster::redirect(CRUDBooster::mainpath('add'), cbTrans($msg), 'success');
        }
        CRUDBooster::redirect(CRUDBooster::mainpath(), cbTrans($msg), 'success');
    }

    private function setTimeStamps($col)
    {
        if (Schema::hasColumn($this->table, $col)) {
            $this->arr[$col] = YmdHis();
        }
    }
}