FormConfigGenerator::generateFormConfig() - Code Metrics - Inspection of "refactor" - crocodic-studio/crudbooster - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1113)

by Iman

created 2018-03-30 14:28 UTC

FormConfigGenerator::generateFormConfig() B

↳ Parent: FormConfigGenerator

Complexity

Conditions	5
Paths	5

Size

Total Lines	80
Code Lines	56

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	56
nc	5
nop	2
dl	0
loc	80
rs	8.4041
c	0
b	0
f	0

How to fix Long Method

<?php

namespace crocodicstudio\crudbooster\Modules\ModuleGenerator\ControllerGenerator;

use crocodicstudio\crudbooster\helpers\DbInspector;
use crocodicstudio\crudbooster\Modules\ModuleGenerator\FileManipulator;
use Illuminate\Support\Facades\Schema;
use CB;

class FormConfigGenerator
{
    /**
     * @param $table
     * @param $coloms
     * @return array
     */
    static function generateFormConfig($table, $coloms)

    {
        $formArrayString = [];
        foreach ($coloms as $i => $colName) {
            //$attribute = [];
            $input = [
                'label' => self::getLabel($colName),
                'name' => $colName,
                'type' => '',
                'options' => '',
                'required' => true,
                'validation' => '',
                'help' => '',
                'placeholder' => '',
            ];

            if (FieldDetector::isExceptional($colName)) {
                continue;
            }

            $typeData = DbInspector::getFieldTypes($table, $colName);

            list($input['type'], $input['validation'], $input['options']) = self::parseFieldType($typeData);

            if (FieldDetector::isForeignKey($colName)) {
                list($input['type'], $input['options']) = self::handleForeignKey($colName);
            } elseif (substr($colName, 0, 3) == 'is_') {
                $input['type'] = 'radio_dataenum';
                $label = ucwords(substr($colName, 3));
                $input['options'] = [
                    'enum' => ['In '.$label, $label],
                    'value' => [0, 1],
                ];
            }

            $map = [
                'password' => [
                    'type' => 'password',
                    'validation' => 'min:5|max:32|required',
                    'help' => cbTrans("text_default_help_password"),
                ],
                'image' => [
                    'type' => 'upload',
                    'validation' => 'required|image',
                    'help' => cbTrans('text_default_help_upload'),
                ],
                'geo' => [
                    'type' => 'hidden',
                    'validation' => 'required|numeric',
                ],
                'phone' => [
                    'type' => 'number',
                    'validation' => 'required|numeric',
                    'placeholder' => cbTrans('text_default_help_number'),
                ],
                'email' => [
                    'type' => 'email',
                    'validation' => 'require|email|unique:'.$table,
                    'placeholder' => cbTrans('text_default_help_email'),
                ],
                'name' => [
                    'type' => 'text',
                    'validation' => 'required|string|min:3|max:70',
                    'placeholder' => cbTrans('text_default_help_text'),
                ],
                'url' => [
                    'type' => 'text',
                    'validation' => 'required|url',
                    'placeholder' => cbTrans('text_default_help_url'),
                ],
            ];
            
            $props = array_get($map, FieldDetector::detect($colName));
            unset($map);
            $input = array_merge($input, $props);

            $formArrayString[] = FileManipulator::stringify($input, str_repeat(" ", 12));
        }

        return $formArrayString;
    }

    /**
     * @param $c
     * @return mixed|string
     */
    private static function getLabel($c)
    {
        $label = str_replace("id_", "", $c);
        $label = ucwords(str_replace("_", " ", $label));

        return $label;
    }

    /**
     * @param $typeData
     *
     * @return array
     */
    private static function parseFieldType($typeData)
    {
        // if matched a key, overrides $typeData to corresponding values
        $typeData = array_get([
            'longtext' => 'text',
            'integer' => 'int',
            'timestamp' => 'datetime',
        ], $typeData, $typeData);

        $default = ["text", "min:1|max:255", []];

        return array_get([
            'text' => ['textarea', "string|min:5", []],
            'date' => ['date', "date", ['php_format' => 'M, d Y', 'datepicker_format' => 'M, dd YYYY',]],
            'datetime' => ['datetime', "date_format:Y-m-d H:i:s", ['php_format' => 'M, d Y H:i',]],
            'time' => ['time', 'date_format:H:i:s', []],
            'double' => ['money', "integer|min:0", []],
            'int' => ['number', 'integer|min:0', []],
        ], $typeData, $default);
        ], /** @scrutinizer ignore-type */ $typeData, $default);
    }

    /**
     * @param $field
     * @return array
     */
    private static function handleForeignKey($field)
    {
        $jointable = str_replace(['id_', '_id'], '', $field);
        if (! Schema::hasTable($jointable)) {
            return ['', ''];
        }
        $options = [
            'table' => $jointable,
            'field_label' => DbInspector::colName(DbInspector::getTableCols($jointable)),
            'field_value' => DbInspector::findPk($jointable),
        ];

        return ['select2_datatable', $options];
    }
}

crocodic-studio / crudbooster

Pull Request — master (#1113)

FormConfigGenerator::generateFormConfig() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like