IndexImport::readAndInsert() - 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

IndexImport::readAndInsert() C

↳ Parent: IndexImport

Complexity

Conditions	9
Paths	27

Size

Total Lines	51
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	9
eloc	31
c	0
b	0
f	0
nc	27
nop	3
dl	0
loc	51
rs	6.2727

How to fix Long Method

<?php

namespace crocodicstudio\crudbooster\controllers\Helpers;

use CRUDBooster;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Facades\Validator;
use Maatwebsite\Excel\Facades\Excel;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Cache;


class IndexImport
{
    /**
     * @return mixed
     */
    public function doneImport()
    {
        $data = [];
        $data['page_title'] = trans('crudbooster.import_page_title', ['module' => \CRUDBooster::getCurrentModule()->name]);
        session()->put('select_column', request('select_column'));

        return view('crudbooster::import', $data);
    }


    /**
     * @param $file_md5
     * @return mixed
     */
    function handleImportProgress($file_md5)

    {
        $total = session('total_data_import') * 100;
        $prog = intval(cache('success_'.$file_md5)) / $total;
        $prog = round($prog, 2);

        if ($prog >= 100) {
            cache()->forget('success_'.$file_md5);
        }

        return response()->json(['progress' => $prog, 'last_error' => cache('error_'.$file_md5)]);
        return response()->/** @scrutinizer ignore-call */ json(['progress' => $prog, 'last_error' => cache('error_'.$file_md5)]);
    }

    /**
     * @param $file
     * @return string
     */
    function uploadImportData($file)

    {
        $dir = 'uploads/'.date('Y-m');
        $filename = md5(str_random(5)).'.'. $file->getClientOriginalExtension();

        //Create Directory Monthly
        Storage::makeDirectory($dir);

        //Move file to storage
        Storage::putFileAs($dir, $file, $filename);

        return CRUDBooster::mainpath('import-data').'?file='.base64_encode($dir.'/'.$filename);
    }

    /**
     * @param $file
     * @return array
     */
    function validateForImport($file)

    {
        return Validator::make(['extension' => $file->getClientOriginalExtension(),], ['extension' => 'in:xls,xlsx,csv']);

    }


    /**
     * @param $file_md5
     * @param $table
     * @param $titleField
     */
    function InsertToDB($file_md5, $table, $titleField)

    {
        $select_column = array_filter(session('select_column'));
        $select_column = array_filter(/** @scrutinizer ignore-type */ session('select_column'));
        $table_columns = DB::getSchemaBuilder()->getColumnListing($table);

        $file = base64_decode(request('file'));
        $file = base64_decode(/** @scrutinizer ignore-type */ request('file'));
        $file = 'storage'.DIRECTORY_SEPARATOR.'app'.DIRECTORY_SEPARATOR.$file;
        $rows = Excel::load($file, function ($reader) {
        $rows = Excel::load($file, function (/** @scrutinizer ignore-unused */ $reader) {
        })->get();

        //$data_import_column = [];
        foreach ($rows as $value) {
            $a = $this->readAndInsert($select_column, $table_columns, $value);

            $has_title_field = true;
            foreach ($a as $k => $v) {
                if ($k == $titleField && $v == '') {
                    $has_title_field = false;
                    break;
                }
            }

            if ($has_title_field == false) {
                continue;
            }

            try {
                if (\Schema::hasColumn($table, 'created_at')) {
                    $a['created_at'] = date('Y-m-d H:i:s');
                }

                \DB::table($table)->insert($a);
                Cache::increment('success_'.$file_md5);
            } catch (\Exception $e) {
                $e = (string) $e;
                Cache::put('error_'.$file_md5, $e, 500);
            }
        }
    }

    /**
     * @param $select_column
     * @param $table_columns
     * @param $value
     * @return array
     */
    private function readAndInsert($select_column, $table_columns, $value)
    {
        $a = [];
        foreach ($select_column as $sk => $s) {
            $colname = $table_columns[$sk];

            if (! CRUDBooster::isForeignKey($colname) || intval($value->$s)) {
                $a[$colname] = $value->$s;
                continue;
            }

            //Skip if value is empty
            if ($value->$s == '') {
                continue;
            }

            $relation_table = CRUDBooster::getTableForeignKey($colname);
            $relation_moduls = DB::table('cms_moduls')->where('table_name', $relation_table)->first();

            $relation_class = __NAMESPACE__.'\\'.$relation_moduls->controller;
            if (! class_exists($relation_class)) {
                $relation_class = '\App\Http\Controllers\\'.$relation_moduls->controller;
            }
            $relation_class = new $relation_class;
            $relation_class->genericLoader();

            $title_field = $relation_class->title_field;

            $relation_insert_data = [];
            $relation_insert_data[$title_field] = $value->$s;

            if (\Schema::hasColumn($relation_table, 'created_at')) {
                $relation_insert_data['created_at'] = date('Y-m-d H:i:s');
            }

            try {
                $relation_exists = DB::table($relation_table)->where($title_field, $value->$s)->first();
                if ($relation_exists) {
                    $relation_primary_key = $relation_class->primary_key;
                    $relation_id = $relation_exists->$relation_primary_key;
                } else {
                    $relation_id = DB::table($relation_table)->insertGetId($relation_insert_data);
                }

                $a[$colname] = $relation_id;
            } catch (\Exception $e) {
                exit($e);

            }
        }

        return $a;
    }
}

crocodic-studio / crudbooster

Pull Request — master (#1098)

IndexImport::readAndInsert() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like