FormSubmitHandlers::inputAssignment() - Code Metrics - Inspection of "fixes and refactors" - crocodic-studio/crudbooster - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1098)

by Iman

created 2018-03-26 16:35 UTC

FormSubmitHandlers::inputAssignment() C

↳ Parent: FormSubmitHandlers

Complexity

Conditions	12
Paths	20

Size

Total Lines	31
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	12
eloc	18
c	0
b	0
f	0
nc	20
nop	1
dl	0
loc	31
rs	5.1612

How to fix Complexity

<?php

namespace crocodicstudio\crudbooster\controllers\CBController;

use CB;
use crocodicstudio\crudbooster\controllers\FormValidator;
use Illuminate\Support\Facades\Request;
use crocodicstudio\crudbooster\CBCoreModule\RelationHandler;
use Illuminate\Support\Facades\Schema;

trait FormSubmitHandlers
{
    public function inputAssignment($id = null)
    public function inputAssignment(/** @scrutinizer ignore-unused */ $id = null)
    {
        $hide_form = (request('hide_form')) ? unserialize(request('hide_form')) : [];
        $hide_form = (request('hide_form')) ? unserialize(/** @scrutinizer ignore-type */ request('hide_form')) : [];

        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 = \CB::componentsPath($type).DIRECTORY_SEPARATOR.'hookInputAssignment.php';
            if (file_exists($hookPath)) {
                require_once($hookPath);
            }
            unset($hookPath);

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

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

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


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


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

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

        $this->setTimeStamps('created_at');

        $this->hookBeforeAdd($this->arr);
               hookBeforeAdd($this->arr);
        $id = $this->table()->insertGetId($this->arr);
        $id = $this->/** @scrutinizer ignore-call */ table()->insertGetId($this->arr);
        app(RelationHandler::class)->save($this->table, $id, $this->data_inputan);
        $this->hookAfterAdd($id);
               hookAfterAdd($id);

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

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

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

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

        $this->setTimeStamps('updated_at');

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

        $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')) {
                CB::redirect(Request::server('HTTP_REFERER'), cbTrans($msg), 'success');
            }
            CB::redirect($this->return_url, cbTrans($msg), 'success');
        }
        if (request('submit') == cbTrans('button_save_more')) {
            CB::redirect(CB::mainpath('add'), cbTrans($msg), 'success');
        }
        CB::redirect(CB::mainpath(), cbTrans($msg), 'success');
    }

    private function setTimeStamps($col)
    {
        if (Schema::hasColumn($this->table, $col)) {
            $this->arr[$col] = date('Y-m-d H:i:s');

        }
    }
}

crocodic-studio / crudbooster

Pull Request — master (#1098)

FormSubmitHandlers::inputAssignment() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like